Skip to content

fix(security): fix ExtraHeaders bug and add HTTP client timeout#14

Open
lakhansamani wants to merge 1 commit intomainfrom
fix/extra-headers-and-http-timeout
Open

fix(security): fix ExtraHeaders bug and add HTTP client timeout#14
lakhansamani wants to merge 1 commit intomainfrom
fix/extra-headers-and-http-timeout

Conversation

@lakhansamani
Copy link
Copy Markdown
Contributor

Summary

  • ExtraHeaders bug (HIGH): Fixed NewAuthorizerClient to use the enriched headers map (with x-authorizer-url and x-authorizer-client-id) instead of the raw extraHeaders parameter. Previously, default headers were never sent with requests.
  • HTTP timeout (MEDIUM): Added 30-second timeout to http.Client in GraphQL requests to prevent indefinite hangs.

Files Changed

  • client.go — ExtraHeaders assignment fix
  • graphql.go — HTTP client timeout

Test plan

  • Verify SDK requests include x-authorizer-url and x-authorizer-client-id headers
  • Verify requests timeout after 30 seconds when server is unresponsive

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