Skip to content

feat: implement comprehensive logging standards (#536)#962

Open
soterika wants to merge 1 commit into
rinafcode:mainfrom
soterika:feat/536-logging-standards
Open

feat: implement comprehensive logging standards (#536)#962
soterika wants to merge 1 commit into
rinafcode:mainfrom
soterika:feat/536-logging-standards

Conversation

@soterika

@soterika soterika commented Jul 1, 2026

Copy link
Copy Markdown
  • Add AppLoggerService: structured JSON logs, correlation IDs, NestJS ConsoleLogger extension, convenience methods (logRequest, logEvent, logQuery)
  • Add redaction.util: recursive sensitive-data redaction for 30+ key patterns (password, token, secret, authorization, jwt, pin, cvv, etc.)
  • Add LoggingInterceptor: global HTTP request/response logger with duration, status code, and correlation ID echo header
  • Add LoggingMiddleware: lightweight middleware to propagate correlation ID to response headers
  • Add LoggingModule (@global): registers AppLoggerService, interceptor, and middleware; imported once in AppModule
  • Add 87 unit tests across 3 suites covering all acceptance criteria
  • Add docs/logging.md: usage guide, log level table, configuration, integration examples

Closes #536

- Add AppLoggerService: structured JSON logs, correlation IDs, NestJS
  ConsoleLogger extension, convenience methods (logRequest, logEvent,
  logQuery)
- Add redaction.util: recursive sensitive-data redaction for 30+ key
  patterns (password, token, secret, authorization, jwt, pin, cvv, etc.)
- Add LoggingInterceptor: global HTTP request/response logger with
  duration, status code, and correlation ID echo header
- Add LoggingMiddleware: lightweight middleware to propagate correlation
  ID to response headers
- Add LoggingModule (@global): registers AppLoggerService, interceptor,
  and middleware; imported once in AppModule
- Add 87 unit tests across 3 suites covering all acceptance criteria
- Add docs/logging.md: usage guide, log level table, configuration,
  integration examples

Closes rinafcode#536
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.

Implement comprehensive logging standards

1 participant