Skip to content

feat(logging): make log level configurable via LOG_LEVEL - #907

Merged
MaryammAli merged 1 commit into
BlockDash-Studios:mainfrom
Husten150:be-004-log-level
Sep 27, 2026
Merged

MaryammAli merged 1 commit into
BlockDash-Studios:mainfrom
Husten150:be-004-log-level

Conversation

@Husten150

Copy link
Copy Markdown
Contributor

Refs #757 (BE-004). nest-winston JSON logging and x-request-id correlation were already wired via correlation-id.middleware.ts and logging.interceptor.ts. The remaining acceptance criterion was LOG_LEVEL configurability: winstonConfig set no level, so the level was winston's default and could not vary per environment. Defaults to 'info' when unset.

closes #759
closes #760
close #758
closes #757

Refs BlockDash-Studios#757 (BE-004). nest-winston JSON logging and x-request-id correlation were already wired via correlation-id.middleware.ts and logging.interceptor.ts. The remaining acceptance criterion was LOG_LEVEL configurability: winstonConfig set no level, so the level was winston's default and could not vary per environment. Defaults to 'info' when unset.
@drips-wave

drips-wave Bot commented Sep 27, 2026

Copy link
Copy Markdown

@Husten150 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@MaryammAli MaryammAli left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@MaryammAli
MaryammAli merged commit 3a913b0 into BlockDash-Studios:main Sep 27, 2026
1 of 2 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

2 participants