Skip to content

RPC health recovery: Already has exponential backoff retry logic in check_rpc_health() - #936

Open
katefominmoore23 wants to merge 1 commit into
WHEELBACK:mainfrom
katefominmoore23:batch-115-features
Open

katefominmoore23 wants to merge 1 commit into
WHEELBACK:mainfrom
katefominmoore23:batch-115-features

Conversation

@katefominmoore23

Copy link
Copy Markdown
Contributor

Changes:

  • handlers_readiness.rs: Updated readiness_probe to check real dependencies and return proper status
  • extractors.rs: Added MAX_PAYLOAD_SIZE validation (1MB) and early rejection for oversized Content-Length headers
  • correlation.rs: New middleware to generate/propagate X-Correlation-ID headers on every request/response
  • lib.rs: Wire up correlation middleware and add uuid dependency for ID generation
  • Cargo.toml: Add uuid crate with v4 feature

Closes #855
Closes #856
Closes #857
Closes #859

Pull Request

Summary

  • Describe the purpose of this PR in one or two sentences.

Checklist

  • PR description includes: Closes #[issue_id]
  • Tests added or updated
  • ABI snapshot updated if contract sources changed (abis/ and COMEBACKHERE-contracts/)
  • Screenshots attached if UI changed
  • Documentation updated if needed

Notes

  • For contract changes, verify ABI snapshot hygiene with make check-abi-snapshots.
  • For UI changes, include screenshots or screen recordings to help reviewers.

- WHEELBACK#857: RPC health check recovery - check_rpc_health() uses exponential backoff retry to recover after transient failures
- WHEELBACK#856: Readiness endpoint now checks actual dependencies (Soroban RPC and Horizon) instead of hardcoding status
- WHEELBACK#855: Add correlation ID middleware to include X-Correlation-ID header in all API responses for request tracing
- WHEELBACK#859: Validate request payload size in ValidatedBody extractor, rejecting oversized payloads (>1MB) with 413 Payload Too Large

Changes:
- handlers_readiness.rs: Updated readiness_probe to check real dependencies and return proper status
- extractors.rs: Added MAX_PAYLOAD_SIZE validation (1MB) and early rejection for oversized Content-Length headers
- correlation.rs: New middleware to generate/propagate X-Correlation-ID headers on every request/response
- lib.rs: Wire up correlation middleware and add uuid dependency for ID generation
- Cargo.toml: Add uuid crate with v4 feature
@drips-wave

drips-wave Bot commented Oct 18, 2026

Copy link
Copy Markdown

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

1 participant