Skip to content

log: adjust level and message(s) slightly for four messages#2532

Merged
georglauterbach merged 4 commits intomasterfrom
scripts/log-adjustents
Apr 5, 2022
Merged

log: adjust level and message(s) slightly for four messages#2532
georglauterbach merged 4 commits intomasterfrom
scripts/log-adjustents

Conversation

@georglauterbach
Copy link
Copy Markdown
Member

Description

Small follow-up where I found some log levels that I thought should be changed. Disabling ClamAV and SA is perfectly fine, and should not be logged with the same level as something like Welcome to docker-mailserver <VERSION>. Another small adjustment to the SSL insecure message, nothing big though. I swear this is the last log adjustment PR in a while :D

Type of change

  • Improvement (non-breaking change that does improve existing functionality)

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (README.md or the documentation under docs/)
  • If necessary I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

@georglauterbach georglauterbach added area/scripts kind/improvement Improve an existing feature, configuration file or the documentation labels Apr 4, 2022
@georglauterbach georglauterbach added this to the v11.0.0 milestone Apr 4, 2022
@georglauterbach georglauterbach self-assigned this Apr 4, 2022
@georglauterbach georglauterbach changed the title log: adjust level and message(s) slightly log: adjust level and message(s) slightly for four messages Apr 4, 2022
wernerfred
wernerfred previously approved these changes Apr 4, 2022
Comment thread target/scripts/helpers/ssl.sh Outdated
Comment thread target/scripts/startup/setup-stack.sh Outdated
Comment thread target/scripts/startup/setup-stack.sh Outdated
@georglauterbach georglauterbach merged commit c7b16a5 into master Apr 5, 2022
@georglauterbach georglauterbach deleted the scripts/log-adjustents branch April 5, 2022 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/scripts kind/improvement Improve an existing feature, configuration file or the documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants