Skip to content

Fixes for metadata credential fetching#32

Open
contradict wants to merge 2 commits intoJuliaCloud:masterfrom
russelhowe:master
Open

Fixes for metadata credential fetching#32
contradict wants to merge 2 commits intoJuliaCloud:masterfrom
russelhowe:master

Conversation

@contradict
Copy link

The first change is for a change in the Julia HTTP library, data is now body.

The second is for a change it the Google API, the format of the Content-Type header has changed.

With these two changes, MetadataCredentials works on GCE instances.

There are now two values in one header:
    application/json; charset=utf-8
Split on the semicolon and check the first part.
@mattBrzezinski
Copy link
Member

This package doesn't look like it gets too much attention sadly. When I have some spare time I can look into the CI error and fix it.

Would you mind possibly adding some tests to this? Although looking at what exists currently there doesn't seem to be much there.

@mattBrzezinski mattBrzezinski self-assigned this Sep 10, 2020
@JockLawrie
Copy link
Contributor

Hi Matt, I am no longer using GCP and therefore am no longer maintaining this package. If you'd like to fork this repo I'll link to the fork from here and archive this repo.

@mattBrzezinski
Copy link
Member

Hi Matt, I am no longer using GCP and therefore am no longer maintaining this package. If you'd like to fork this repo I'll link to the fork from here and archive this repo.

I think it can continue to live in the JuliaCloud org, it seems like an appropriate place.

@Alexander-Barth
Copy link

I can confirm that this PR is still necessary for MetadataCredentials().
Would it be possible of somebody from the JuliaCloud org would merge this PR?

Thanks for this great package (and PR :-)) !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants