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

fix: update dep gcp-metadata#334

Merged
ofrobots merged 1 commit intogoogleapis:masterfrom
ofrobots:update-gcp-metadata
Sep 27, 2017
Merged

fix: update dep gcp-metadata#334
ofrobots merged 1 commit intogoogleapis:masterfrom
ofrobots:update-gcp-metadata

Conversation

@ofrobots
Copy link
Copy Markdown
Contributor

@ofrobots ofrobots commented Sep 27, 2017

The latest version of gcp-metadata provides an error upon a 404 reply from metadata service. Previously we were leaking the 404 html as the service context.

Partially addresses https://stackoverflow.com/questions/46411341/stackdriver-debugger-on-google-compute-engine.

This bug also meant that we would incorrectly identify App Engine apps as GKE.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Sep 27, 2017
@ofrobots ofrobots merged commit be0f2fd into googleapis:master Sep 27, 2017
@ofrobots ofrobots deleted the update-gcp-metadata branch September 27, 2017 23:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants