Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

auth: reduce logging for "GSS-TSIG request but feature not compiled in" #14841

Open
patrickpoortman opened this issue Nov 11, 2024 · 2 comments · May be fixed by #14842
Open

auth: reduce logging for "GSS-TSIG request but feature not compiled in" #14841

patrickpoortman opened this issue Nov 11, 2024 · 2 comments · May be fixed by #14842
Assignees
Labels

Comments

@patrickpoortman
Copy link

patrickpoortman commented Nov 11, 2024

  • Program: auth
  • Issue type: Feature request

Short description

Reduce logging output when GSS-TSIG requests are made to auth when the GSS-TSIG feature was not compiled in PowerDNS to prevent unnecessary logging. Or make this log message optional.

Usecase

For each GSS-TSIG request made there is a log entry made at the error level, when it is not part of the compile options.

Nov 11 15:22:52 ns1 pdns_server[3681960]: GSS-TSIG request but not feature not compiled in

Description

Reduce the loglevel to info or implement a way to filter the log output being written to the logfiles.
Currently the error is defined here;

g_log<<Logger::Error<<"GSS-TSIG request but feature not compiled in"<<endl;

This issue has been created based on the comment in #14839 (comment)

@omoerbeek
Copy link
Member

Two lines above the same. Level should probably be "Debug". I'll make a PR (I am the author iirc).

@omoerbeek
Copy link
Member

Actually, this is an auth issue, not a dnsdist issue, I'll edit it a bit.

@omoerbeek omoerbeek changed the title dnsdist: reduce logging for "GSS-TSIG request but feature not compiled in" auth: reduce logging for "GSS-TSIG request but feature not compiled in" Nov 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants