Skip to content

Features/request#67

Merged
Tybravo merged 3 commits into
SwiftChainn:mainfrom
samad13:features/request
Jul 1, 2026
Merged

Features/request#67
Tybravo merged 3 commits into
SwiftChainn:mainfrom
samad13:features/request

Conversation

@samad13

@samad13 samad13 commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

successfully implemented request and application logging:

  1. Updated src/config/logger.ts : use env config instead of process.env, added JSON format for production logs
  2. Created src/middleware/requestLogger.ts : a request logging middleware that logs method, url, status code, duration, ip, user‑agent
  3. Updated src/app.ts : replaced old logging middleware with requestLogger, fixed all remaining process.env usage
  4. Fixed circular dependency : removed logger import from env.ts to avoid circular dependency, use console.error for env validation errors (which is acceptable since those are fatal errors that exit the process anyway)
  5. All tests pass!

Closes #6

@drips-wave

drips-wave Bot commented Jul 1, 2026

Copy link
Copy Markdown

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

@Tybravo Tybravo merged commit 3f8313e into SwiftChainn:main Jul 1, 2026
1 check failed
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.

Backend: Implement request and application logging using Winston or Pino

2 participants