Skip to content

[PART 10] fix: forward query parameters through proxy#10

Merged
andreabadesso merged 1 commit into
masterfrom
fix/proxy-query-params
Apr 1, 2026
Merged

[PART 10] fix: forward query parameters through proxy#10
andreabadesso merged 1 commit into
masterfrom
fix/proxy-query-params

Conversation

@andreabadesso

Copy link
Copy Markdown
Contributor

Summary

  • Query parameters were silently dropped when proxying to wallet-headless containers

Acceptance Criteria

  • Query extractor is added to proxy_handler function signature
  • When query params are present, they are serialized as ?key=value&... in the forwarded URL
  • When no query params, URL is unchanged (no trailing ?)
  • Token balance queries (/wallet/balance?token=xxx) work correctly through proxy

🤖 Generated with Claude Code

@andreabadesso andreabadesso changed the title fix: forward query parameters through proxy [PART 10] fix: forward query parameters through proxy Mar 31, 2026
@andreabadesso andreabadesso force-pushed the fix/proxy-query-params branch from 8e81c31 to 7a594fe Compare March 31, 2026 13:23
@andreabadesso andreabadesso changed the base branch from master to feat/tests-lifecycle-nano-contracts March 31, 2026 13:24
@andreabadesso andreabadesso added the enhancement New feature or request label Mar 31, 2026
@andreabadesso andreabadesso self-assigned this Mar 31, 2026
@andreabadesso andreabadesso moved this from Todo to In Progress (Done) in Hathor Network Mar 31, 2026
@github-project-automation github-project-automation Bot moved this from In Progress (Done) to In Review (WIP) in Hathor Network Mar 31, 2026
@andreabadesso andreabadesso changed the base branch from feat/tests-lifecycle-nano-contracts to master April 1, 2026 14:22
Query parameters (e.g., ?token=xxx for balance checks) were being
dropped by the proxy handler. Extract Query params and rebuild the
URL with the query string before forwarding.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@andreabadesso andreabadesso force-pushed the fix/proxy-query-params branch from 7a594fe to c7017a2 Compare April 1, 2026 14:26
@andreabadesso andreabadesso merged commit c8dcac1 into master Apr 1, 2026
1 check passed
@github-project-automation github-project-automation Bot moved this from In Review (WIP) to Waiting to be deployed in Hathor Network Apr 1, 2026
@andreabadesso andreabadesso deleted the fix/proxy-query-params branch April 1, 2026 14:26
@andreabadesso andreabadesso moved this from Waiting to be deployed to Done in Hathor Network Apr 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants