-
-
Notifications
You must be signed in to change notification settings - Fork 601
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
Patch cert-checker hard-coded log levels and handle nil mariadb response #6066
Conversation
- Fix cert-checker to use the syslog and stdout logging facilities it reads from the config file instead of having them hard-coded to zero. - Fix cert-checker to handle a nil response from mariadb if no records are found. - Fix comment in log.go to correctly describe when the initialize function and therefore default values would be used.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM with two optional nits
Co-authored-by: Aaron Gable <aaron@aarongable.com>
Co-authored-by: Aaron Gable <aaron@aarongable.com>
Great suggestions, thanks. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR! Just some small formatting and comment nits. I also believe that your tests are currently failing.
Co-authored-by: Samantha <hello@entropy.cat>
Thanks for the comments feedback. I think I've addressed it all. |
Yep I've filed issues for the unit test flakes. Will try to get those fixed ASAP, even if it means reverting recent changes. |
reads from the config file instead of having them hard-coded to zero.
are found.
and therefore default values would be used.
Fixes #6067