Skip to content

Propagate request IDs to responses and error bodies#300

Closed
zhangsiqiang519 wants to merge 1 commit into
BountyOnChain:mainfrom
zhangsiqiang519:codex/stellar-bounty-request-id-178
Closed

Propagate request IDs to responses and error bodies#300
zhangsiqiang519 wants to merge 1 commit into
BountyOnChain:mainfrom
zhangsiqiang519:codex/stellar-bounty-request-id-178

Conversation

@zhangsiqiang519

Copy link
Copy Markdown

Summary

  • Return x-request-id on HTTP responses when a request id is present.
  • Include the request id in HTTP error response bodies for support/debug correlation.
  • Add focused tests for the middleware and exception filter behavior.

Closes #178

Verification

  • npm test --workspace=apps/backend -- --runTestsByPath src/common/middleware/logger.middleware.spec.ts src/common/filters/http-exception.filter.spec.ts src/common/json-logger.service.spec.ts --runInBand
  • npm run lint --workspace=apps/backend
  • npm run build --workspace=apps/backend
  • npm run test --workspace=apps/backend -- --runInBand
  • git diff --check

@GBOYEE

GBOYEE commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

I'm interested in tackling this! I have strong experience with full-stack and can deliver a clean solution.

@GBOYEE

GBOYEE commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Can I take this? backend fix, straightforward scope. Will submit PR in ~1hr. Closes #300

@GBOYEE

GBOYEE commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

I'd like to work on this.

Approach:

  • I'll write the test cases and ensure they pass with full coverage
  • Verify with existing tests + add new ones if needed

Estimated effort: ~1-2 hours. PR incoming shortly.
Closes #300

Copy link
Copy Markdown
Contributor

Thanks for this. Issue #178 is assigned to @egwujiohaifesinachiperpetual-max, so I'm holding off. Worth coordinating in the issue thread before opening or landing.

Copy link
Copy Markdown
Contributor

Closing this — #178 is assigned to @egwujiohaifesinachiperpetual-max. Thanks for the work, feel free to coordinate on the issue.

@Xuccessor Xuccessor closed this Jun 23, 2026
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.

[OBSERVABILITY] Inconsistent request ID propagation in logs

3 participants