Skip to content

Conversation

@Elenetta17
Copy link
Member

  • Replace fixed logging.basicConfig(level=logging.INFO) in startup event with dynamic configuration based on LOG_LEVEL environment variable
  • Default log level is now WARNING, reducing noisy logs in production
  • Allows easy adjustment of logging verbosity per environment without code changes

Tested on the production server

- Replace fixed logging.basicConfig(level=logging.INFO) in startup event with dynamic configuration based on LOG_LEVEL environment variable
- Default log level is now WARNING, reducing noisy logs in production
- Allows easy adjustment of logging verbosity per environment without code changes
Copy link
Member

@SaiedKazemi SaiedKazemi left a comment

Choose a reason for hiding this comment

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

LGTM

@Elenetta17 Elenetta17 merged commit 1fa7924 into main Oct 17, 2025
2 checks passed
@Elenetta17 Elenetta17 deleted the adjust_logging branch October 17, 2025 08:30
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.

3 participants