Skip to content

Add support for bearer auth header for private subgraph endpoints#4348

Closed
AryanGodara wants to merge 1 commit intomainfrom
aryan/enable-goldsky-private-endpoints
Closed

Add support for bearer auth header for private subgraph endpoints#4348
AryanGodara wants to merge 1 commit intomainfrom
aryan/enable-goldsky-private-endpoints

Conversation

@AryanGodara
Copy link
Copy Markdown
Member

@AryanGodara AryanGodara commented Apr 20, 2026

Changes

Added option to send bearer auth header for goldsky subgraphs.
Enabling private endpoints and tags on the subgraph endpoints
This enables rotation for subgraph endpoints

How to test

  • run the usual unit/e2e test suit

@AryanGodara AryanGodara marked this pull request as ready for review April 20, 2026 23:07
@AryanGodara AryanGodara requested a review from a team as a code owner April 20, 2026 23:07
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces support for optional bearer token authentication when accessing private subgraph endpoints for Uniswap V3 and Balancer V2 liquidity sources. The changes include updating configuration files, internal data structures, and the SubgraphClient to handle and transmit the api-key via the Authorization header. I have no feedback to provide.

@fafk
Copy link
Copy Markdown
Contributor

fafk commented Apr 21, 2026

Should be obsoleted by #4349

@AryanGodara AryanGodara changed the title Add support for bearer auth header for private subgraph endpoints… Add support for bearer auth header for private subgraph endpoints Apr 21, 2026
@AryanGodara
Copy link
Copy Markdown
Member Author

Should be obsoleted by #4349
Closing this in favor of this PR ^^
🙇🏼

@github-actions github-actions Bot locked and limited conversation to collaborators Apr 21, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants