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.

Sign up for GitHub

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

Jump to bottom

fix: allows user-agent header with header provider #871

Merged
merged 2 commits into from Feb 17, 2021
Merged

fix: allows user-agent header with header provider #871

merged 2 commits into from Feb 17, 2021

Commits on Feb 15, 2021

  1. fix: allows user-agent header with header provider 

    A bug was introduced, where if the caller tried to set a custom user
agent with a header provider an exception would be thrown (for duplicate
keys). Here, we merge the user agent set by the client along with the
one set by the library, instead of throwing such exception.
    @thiagotnunes
    thiagotnunes committed Feb 15, 2021
    Copy the full SHA
    4835d85 View commit details
    Browse the repository at this point in the history

Commits on Feb 17, 2021

  1. test: adds test for default user agent 

    Tests if the default user agent is present in the user-agent header set
in the GapicSpannerRpc class.
    @thiagotnunes
    thiagotnunes committed Feb 17, 2021
    Copy the full SHA
    6ff8f44 View commit details
    Browse the repository at this point in the history