Skip to content

feat: customize log level#26

Merged
wiggin77 merged 1 commit intowiggin77:masterfrom
filipgolonka:feat/customize-log-level
Oct 5, 2025
Merged

feat: customize log level#26
wiggin77 merged 1 commit intowiggin77:masterfrom
filipgolonka:feat/customize-log-level

Conversation

@filipgolonka
Copy link
Contributor

@wiggin77 🙏

@filipgolonka filipgolonka requested a review from wiggin77 as a code owner October 3, 2025 09:43
@wiggin77 wiggin77 requested a review from Copilot October 5, 2025 17:08
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds configurable log level support to the mail relay application. Instead of hardcoding the log level to "info", users can now specify their desired log level in the configuration file.

  • Added log_level field to the configuration structure and JSON schema
  • Modified logger setup to use the configured log level instead of hardcoded "info"
  • Set "info" as the default log level in configuration defaults

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
testdata/valid.json Added example log_level configuration field
main.go Added LogLevel field to config struct, updated setupLogger function signature, and set default value
server.go Updated to use configured log level instead of hardcoded value

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Owner

@wiggin77 wiggin77 left a comment

Choose a reason for hiding this comment

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

LGTM 👍 Thanks @filipgolonka

@wiggin77 wiggin77 merged commit 9cc54f8 into wiggin77:master Oct 5, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants