Skip to content

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

Open
Danielobito009 wants to merge 1 commit into
MettaChain:mainfrom
Danielobito009:fix/rate-limit-logger
Open

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

Conversation

@Danielobito009

Copy link
Copy Markdown

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

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

2 participants