Skip to content
This repository was archived by the owner on Mar 17, 2021. It is now read-only.

Set new HttpClient via cleanhttp #19

Merged
merged 1 commit into from
Oct 17, 2019
Merged

Set new HttpClient via cleanhttp #19

merged 1 commit into from
Oct 17, 2019

Conversation

radeksimko
Copy link
Contributor

@radeksimko radeksimko commented Feb 23, 2019

The default http.Client when initialized like this does not have any Transport/RoundTripper, which prevents users of this library downstream from adding custom transports on top, e.g. logging transport we have in Terraform.

We would like to use it in the Grafana provider, but any request then blows up due to Transport being nil and I think it's better to avoid just blindly setting default transport when adding a new one.

@mlclmj mlclmj added enhancement size/XS Denotes a PR that changes 0-9 lines labels Oct 10, 2019
Copy link
Contributor

@mlclmj mlclmj left a comment

Choose a reason for hiding this comment

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

This is great, thanks for making the change!

@mlclmj mlclmj merged commit c88dfea into nytm:master Oct 17, 2019
@radeksimko radeksimko deleted the cleanhttp branch October 18, 2019 17:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement size/XS Denotes a PR that changes 0-9 lines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants