Skip to content

fix(ts-sdk): send auth headers for execution price details#997

Open
realfishsam wants to merge 1 commit into
mainfrom
fix/issue-931-ts-execution-price-auth-1781073223
Open

fix(ts-sdk): send auth headers for execution price details#997
realfishsam wants to merge 1 commit into
mainfrom
fix/issue-931-ts-execution-price-auth-1781073223

Conversation

@realfishsam

Copy link
Copy Markdown
Contributor

Summary

  • Update TypeScript getExecutionPriceDetailed to use getAuthHeaders(), matching other authenticated SDK calls
  • Add a hosted dispatch regression test asserting the bearer auth header is sent

Fixes #931

Test Plan

  • RED: added focused Jest test and confirmed it failed because Authorization was missing
  • GREEN: npm test -- --runInBand tests/hosted-dispatch.test.ts -t 'getExecutionPriceDetailed sends hosted bearer auth header'
  • npm test -- --runInBand tests/hosted-dispatch.test.ts

Note: local test bootstrap required temporary ignored generated OpenAPI stubs because this runner lacks Java for the generator; committed changes are only client/test files.

@realfishsam

Copy link
Copy Markdown
Contributor Author

CI note from launcher: the focused regression test passes locally, but three generated-sync checks are red. I inspected the failed logs; the generator diffs remove existing hosted-mode sections from generated client outputs and update broad API reference/model docs unrelated to this two-line auth fix. I did not commit those broad generated diffs into this PR. Local environment also lacks Java, so full OpenAPI generation cannot run here.

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.

sdk-drift: getExecutionPriceDetailed sends this.config.headers instead of getAuthHeaders() in TypeScript — auth tokens omitted

1 participant