Skip to content

Add slow request logging (#754) - #1203

Merged
Chucks1093 merged 1 commit into
accesslayerorg:mainfrom
DavisVT:feature/slow-request-logging
Oct 22, 2026
Merged

Chucks1093 merged 1 commit into
accesslayerorg:mainfrom
DavisVT:feature/slow-request-logging

Conversation

@DavisVT

@DavisVT DavisVT commented Oct 21, 2026

Copy link
Copy Markdown
Contributor
  • Add SLOW_REQUEST_THRESHOLD_MS environment variable (default 2000ms)
  • Implement slowRequestLoggerMiddleware to log requests exceeding threshold
  • Log includes method, path, status_code, duration_ms, wallet (if authenticated), slow_threshold_ms
  • Log emitted after response is sent, no request/response bodies logged
  • Register middleware in app.ts

closes #754

- Add SLOW_REQUEST_THRESHOLD_MS environment variable (default 2000ms)
- Implement slowRequestLoggerMiddleware to log requests exceeding threshold
- Log includes method, path, status_code, duration_ms, wallet (if authenticated), slow_threshold_ms
- Log emitted after response is sent, no request/response bodies logged
- Register middleware in app.ts
@Chucks1093

Copy link
Copy Markdown
Member

✅ PR Ready for Review

Due to a recent malware incident affecting our repositories, all PRs are required to pass automated security checks before being reviewed for merging.

This PR has passed all automated checks:

  • ✅ Security scan — no malicious code detected
  • ✅ No merge conflicts
  • ✅ CI checks passed

Next step: Please share this PR link in our Telegram group so a maintainer can review and merge it: https://t.me/c/accesslayerorg/8514

1 similar comment
@Chucks1093

Copy link
Copy Markdown
Member

✅ PR Ready for Review

Due to a recent malware incident affecting our repositories, all PRs are required to pass automated security checks before being reviewed for merging.

This PR has passed all automated checks:

  • ✅ Security scan — no malicious code detected
  • ✅ No merge conflicts
  • ✅ CI checks passed

Next step: Please share this PR link in our Telegram group so a maintainer can review and merge it: https://t.me/c/accesslayerorg/8514

@Chucks1093
Chucks1093 merged commit 599fcf1 into accesslayerorg:main Oct 22, 2026
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

Development

Successfully merging this pull request may close these issues.

Add structured log for each request that exceeds the configured response time threshold

2 participants