Skip to content

fix: pass headers to transport#928

Merged
jackwotherspoon merged 2 commits intomainfrom
set-headers
Jan 30, 2025
Merged

fix: pass headers to transport#928
jackwotherspoon merged 2 commits intomainfrom
set-headers

Conversation

@jackwotherspoon
Copy link
Contributor

In #920 we set the headers for quota project and user agent but failed to actually pass the headers to the underlying transport.

This PR corrects that silly mistake.

@jackwotherspoon jackwotherspoon self-assigned this Jan 30, 2025
@jackwotherspoon jackwotherspoon requested a review from a team as a code owner January 30, 2025 16:00
@jackwotherspoon jackwotherspoon merged commit 08be3ab into main Jan 30, 2025
14 checks passed
@jackwotherspoon jackwotherspoon deleted the set-headers branch January 30, 2025 17:54
Yuan325 added a commit to googleapis/genai-toolbox that referenced this pull request Feb 25, 2025
Bump cloud sql go connector to v1.15.0 to adapt the fix for user agent
(GoogleCloudPlatform/cloud-sql-go-connector#928).
jeffreyrubi pushed a commit to jeffreyrubi/genai-toolbox that referenced this pull request Jun 7, 2025
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.

2 participants