Skip to content

fix: replace console.error with Nest Logger in rate-limit.guard.ts#865

Merged
nanaf6203-bit merged 1 commit into
MettaChain:mainfrom
Danielobito009:fix/rate-limit-logger
Jul 1, 2026
Merged

fix: replace console.error with Nest Logger in rate-limit.guard.ts#865
nanaf6203-bit merged 1 commit into
MettaChain:mainfrom
Danielobito009:fix/rate-limit-logger

Conversation

@Danielobito009

Copy link
Copy Markdown
Contributor

Summary

Closes #779. Replaces console.error with NestJS structured Logger
in rate-limit.guard.ts for consistent log formatting.

Changes

  • src/auth/guards/rate-limit.guard.ts
    • Imported Logger from @nestjs/common
    • Added private readonly logger = new Logger(RateLimitGuard.name)
    • Replaced console.error(...) with this.logger.error(...)

Why

NestJS Logger provides structured output with timestamps, context
labels, and log levels — making errors easier to trace in production
vs raw console.error output.

closes #779

@drips-wave

drips-wave Bot commented Jun 30, 2026

Copy link
Copy Markdown

@Danielobito009 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

@nanaf6203-bit nanaf6203-bit 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

@nanaf6203-bit nanaf6203-bit merged commit 3870714 into MettaChain:main Jul 1, 2026
5 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.

[F11] Replace console.error with Nest Logger in rate-limit.guard.ts

3 participants