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

fix: update @google-cloud/common dependency to 0.25.3 #871

Merged
merged 1 commit into from
Sep 27, 2018
Merged

Conversation

kjin
Copy link
Contributor

@kjin kjin commented Sep 27, 2018

This PR upgrades @g-c/c to avoid an issue with authentication in gcp-metadata that was fixed in a newer version of the module.

In order to make this change, I've changed and moved tests that deal with keyFilename/credentials to be more modular (it's a test of the trace writer, not the module as a whole).

Also closes #855

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Sep 27, 2018
@ghost ghost assigned kjin Sep 27, 2018
Copy link

@jcao219 jcao219 left a comment

Choose a reason for hiding this comment

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

This update fixes an issue where google-auth-library falsely believes that it is not running in a GCE instance, when in fact it is, because the API call to retrieve the instance metadata might fail with ECONNRESET on the first try.

@ghost ghost assigned JustinBeckwith Sep 27, 2018
@JustinBeckwith
Copy link
Contributor

@kjin nudges

@kjin kjin merged commit 23a0616 into master Sep 27, 2018
@kjin kjin deleted the common-0.25 branch September 27, 2018 23:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants