Skip to content

fix(auth): honor funder address in CLOB auth headers#13

Merged
SebastianBoehler merged 1 commit into
mainfrom
codex/poly1271-funder-auth
May 21, 2026
Merged

fix(auth): honor funder address in CLOB auth headers#13
SebastianBoehler merged 1 commit into
mainfrom
codex/poly1271-funder-auth

Conversation

@SebastianBoehler
Copy link
Copy Markdown
Owner

@SebastianBoehler SebastianBoehler commented May 21, 2026

Summary

  • use the supplied funder/proxy address when building L1 CLOB auth headers for API-key create/derive
  • use the same supplied funder/proxy address in L2 HMAC headers when provided
  • add V2 signer tests covering funder-aware L1 and L2 header addresses
  • document the auth behavior in the V2 migration notes

Context

Konstantin reported that TypeScript API-key derivation worked when funderAddress was set to the portfolio/proxy address, while this C++ client could not derive credentials. The C++ path accepted a funder argument but ignored it in auth header generation.

Validation

  • cmake -S . -B /tmp/polymarket-cpp-client-auth-verify -DPOLYMARKET_CLIENT_BUILD_EXAMPLES=ON -DPOLYMARKET_CLIENT_BUILD_TESTS=ON -DPOLYMARKET_CLIENT_BUILD_BENCHMARKS=ON
  • cmake --build /tmp/polymarket-cpp-client-auth-verify -- -j4
  • ctest --test-dir /tmp/polymarket-cpp-client-auth-verify --output-on-failure
  • git diff --check

@SebastianBoehler SebastianBoehler merged commit de273d6 into main May 21, 2026
8 checks passed
@SebastianBoehler SebastianBoehler deleted the codex/poly1271-funder-auth branch May 21, 2026 18:31
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.

1 participant