Skip to content

dnsdist: Log listening addresses and version at the 'info' level #11711

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jun 21, 2022
Merged

dnsdist: Log listening addresses and version at the 'info' level #11711

merged 1 commit into from
Jun 21, 2022

Conversation

rgacogne
Copy link
Member

Short description

These are really information, not warnings.

Checklist

I have:

  • read the CONTRIBUTING.md document
  • compiled this code
  • tested this code
  • included documentation (including possible behaviour changes)
  • documented the code
  • added or modified regression test(s)
  • added or modified unit test(s)

@rgacogne
dnsdist: Log listening addresses and version at the 'info' level
51bd35d 
These are really informations, not warnings.
@rgacogne rgacogne added this to the dnsdist-1.8.0 milestone Jun 20, 2022
omoerbeek
omoerbeek approved these changes Jun 20, 2022
View reviewed changes
Copy link
Member

@omoerbeek omoerbeek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Right. Some day we should review all log levels, as the code tends to be sloppy.

@rgacogne rgacogne merged commit ce42f89 into PowerDNS:master Jun 21, 2022
@rgacogne rgacogne deleted the ddist-startup-infos branch June 21, 2022 07:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@omoerbeek omoerbeek omoerbeek approved these changes

Assignees
No one assigned
Labels
dnsdist
Projects
None yet
Milestone
dnsdist-1.8.0
Development

Successfully merging this pull request may close these issues.
2 participants
@rgacogne @omoerbeek