Skip to content
This repository was archived by the owner on Apr 7, 2026. It is now read-only.

chore: remove double client lib token header#2681

Merged
arpan14 merged 4 commits intomainfrom
improve-custom-client-lib-token-implementation
Oct 19, 2023
Merged

chore: remove double client lib token header#2681
arpan14 merged 4 commits intomainfrom
improve-custom-client-lib-token-implementation

Conversation

@olavloite
Copy link
Copy Markdown
Collaborator

Removes the double inclusion of the standard client lib token when no custom client lib token has been set. Also reuses more of the standard feature for setting the client lib token, instaed of appending the standard token at the moment that we are creating the gRPC stub. This should reduce the probability that anyone who might be using a custom header or custom client in some way will get it wrong.

Removes the double inclusion of the standard client lib token when no
custom client lib token has been set. Also reuses more of the standard
feature for setting the client lib token, instaed of appending the
standard token at the moment that we are creating the gRPC stub. This
should reduce the probability that anyone who might be using a custom
header or custom client in some way will get it wrong.
@olavloite olavloite requested a review from a team October 16, 2023 06:10
@product-auto-label product-auto-label Bot added size: m Pull request size is medium. api: spanner Issues related to the googleapis/java-spanner API. labels Oct 16, 2023
@gcf-owl-bot gcf-owl-bot Bot requested a review from a team October 16, 2023 06:13
@olavloite olavloite requested a review from arpan14 October 17, 2023 05:40
@arpan14 arpan14 merged commit 712c65f into main Oct 19, 2023
@arpan14 arpan14 deleted the improve-custom-client-lib-token-implementation branch October 19, 2023 07:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: spanner Issues related to the googleapis/java-spanner API. size: m Pull request size is medium.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants