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

Add headers for requesting and receiving the server version #57

Merged
merged 1 commit into from Apr 28, 2022

Conversation

josephschorr
Copy link
Member

No description provided.


// AddRequestHeaders returns a new context with the given values as request headers.
func AddRequestHeaders(ctx context.Context, keys ...BoolRequestMetadataHeaderKey) context.Context {
values := map[RequestMetadataHeaderKey]string{}
Copy link
Member

Choose a reason for hiding this comment

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

values := make(map[RequestMetadataHeaderKey]string, len(keys))

pkg/requestmeta/requestmeta.go Show resolved Hide resolved
@josephschorr josephschorr force-pushed the server-version-headers branch 3 times, most recently from fd904b0 to cae9cf7 Compare April 27, 2022 20:36
@josephschorr josephschorr changed the title Add headers for requesting and receiving the server version Add headers for requesting and receiving the server version and other debug info Apr 27, 2022
@josephschorr josephschorr force-pushed the server-version-headers branch 2 times, most recently from 03c6df6 to 278411a Compare April 27, 2022 20:47
@josephschorr josephschorr changed the title Add headers for requesting and receiving the server version and other debug info Add headers for requesting and receiving the server version Apr 28, 2022
@josephschorr josephschorr merged commit fe11c14 into authzed:main Apr 28, 2022
@josephschorr josephschorr deleted the server-version-headers branch April 28, 2022 17:26
@github-actions github-actions bot locked and limited conversation to collaborators Apr 28, 2022
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.

None yet

2 participants