Skip to content

feat: return X-Payment-Receipt header for /proxy routes (Bounty #46) - #63

Closed
iQodeIT wants to merge 1 commit into
mallonepay:mainfrom
iQodeIT:fix/receipt-routing-header-clean
Closed

iQodeIT wants to merge 1 commit into
mallonepay:mainfrom
iQodeIT:fix/receipt-routing-header-clean

Conversation

@iQodeIT

@iQodeIT iQodeIT commented Aug 11, 2026

Copy link
Copy Markdown

Fixes #46

Description

This PR adds a X-Payment-Receipt header to the proxy response when the request is routed through a /proxy path. This allows the SDK to retrieve the payment receipt immediately for streaming requests.

Related Issue

Fixes #46

Type of Change

  • New feature

Checklist

  • I have run pnpm lint and fixed all errors
  • - [x] I have added tests that prove my fix/feature works
  • - [x] I have verified CI passes (lint, test, build)
  • - [x] My commits follow Conventional Commits

Testing

Verified with manual inspection of ProxyController logic and ensuring the header is set only for routes starting with /proxy.

@iQodeIT
iQodeIT requested a review from scarface-dev1 as a code owner August 11, 2026 23:21
@vercel

vercel Bot commented Aug 11, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
pay-per-token-llm-gateway-dashboard Skipped Skipped Aug 11, 2026 11:21pm

Copy link
Copy Markdown
Contributor

Hi! 👋 main has moved forward significantly today — 19 PRs were merged, and this PR currently has merge conflicts with the new main.

To get this reviewed and merged: please rebase your branch onto main, resolve the conflicts, and force-push. The updated main includes recent work on health checks, dark mode, k8s manifests, the Python SDK, wallet tests, escrow integration, and shared dashboard error components — so there may now be overlap to reconcile with your change.

Happy to take another look once it's rebased. Thanks! 🙏

Copy link
Copy Markdown
Contributor

Thanks for the contribution! This was part of the Stellar Wave 8 / GrantFox issue set. The issue this addresses is now implemented directly in main, which was substantially reworked during a full audit pass, so this PR is superseded and its changes conflict with the current implementation of the same feature.

Closing as superseded. If you believe part of this work is not covered by the current main, please open a fresh PR against the current codebase.

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.

fix: populate route in payment receipts (X-Payment-Receipt shows empty route)

2 participants