Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(v2): add BuildHeaders and InsertMetadata to header #290

Merged
merged 12 commits into from
Jun 26, 2023

Conversation

quartzmo
Copy link
Member

replaces: #286

refs: googleapis/gapic-generator-go#1300
refs: googleapis/gapic-generator-go#1301

This PR moves the generated insertMetadata and buildHeaders functions to new, relocated gax header functions.

@quartzmo quartzmo requested a review from a team as a code owner June 22, 2023 21:43
codyoss added a commit to codyoss/gax-go that referenced this pull request Jun 23, 2023
Add a new callctx package to gax to facilitate adding and
retrieving values from context.Context that our libraries can use
to share information throughout the call stack. The first addition
to this package will be two APIs to allow users to set RPC headers
that will be added to API requests our clients make.

A follow up PR will be made to add using these after these changes
are merged with googleapis#290.
codyoss added a commit to codyoss/gax-go that referenced this pull request Jun 23, 2023
Add a new callctx package to gax to facilitate adding and
retrieving values from context.Context that our libraries can use
to share information throughout the call stack. The first addition
to this package will be two APIs to allow users to set RPC headers
that will be added to API requests our clients make.

A follow up PR will be made to add using these after these changes
are merged with googleapis#290.
codyoss added a commit that referenced this pull request Jun 23, 2023
Add a new callctx package to gax to facilitate adding and
retrieving values from context.Context that our libraries can use
to share information throughout the call stack. The first addition
to this package will be two APIs to allow users to set RPC headers
that will be added to API requests our clients make.

A follow up PR will be made to add using these after these changes
are merged with #290.
v2/header.go Outdated Show resolved Hide resolved
v2/header.go Outdated Show resolved Hide resolved
v2/header.go Show resolved Hide resolved
v2/header_test.go Outdated Show resolved Hide resolved
v2/header.go Outdated Show resolved Hide resolved
Copy link
Member

@codyoss codyoss left a comment

Choose a reason for hiding this comment

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

Thanks!

@quartzmo quartzmo merged commit 6a4b89f into googleapis:main Jun 26, 2023
@quartzmo quartzmo deleted the header-helpers branch June 26, 2023 20:38
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