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 gcp provider, disable custom http client #405

Merged
merged 2 commits into from
Jan 8, 2023
Merged

fix gcp provider, disable custom http client #405

merged 2 commits into from
Jan 8, 2023

Conversation

orbatschow
Copy link
Contributor

Fixes #404

@qdm12 qdm12 merged commit 8a18aec into qdm12:master Jan 8, 2023
@qdm12
Copy link
Owner

qdm12 commented Jan 8, 2023

Awesome thanks @orbatschow 🎉 🎖️
Sorry this took forever to get merged.
Not like a big fan of not using the same shared http client in the application, but oh well ain't got time to debunk why 😄

@qdm12
Copy link
Owner

qdm12 commented Jun 16, 2024

I don't know why, I must had overridden your changes since the WithHTTPClient option got re-introduced some time ago a84a1d6#diff-0a4e705a31ab1c68c5c035ad1bd0d37db732015d6370f8c6e54a9ee50870547cL25

Anyway now it's back to removed...

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.

Bug: GCP provider seems to be broken
2 participants