Skip to content
This repository was archived by the owner on Aug 23, 2023. It is now read-only.

Enable transport and client settings even in single mode #1000

Merged
merged 1 commit into from
Aug 21, 2018

Conversation

shanson7
Copy link
Collaborator

When requests are canceled, transport is used to cancel the downstream. This causes a panic in single mode because the transport is never instantiated.

Note that HTTP timeout must now be either blank or a valid value even in single mode.

@shanson7
Copy link
Collaborator Author

I was able to reproduce the crash, and can confirm that the above fixes it. I don't think there would be any large ramifications of instantiating the transport in single mode.

@deniszh
Copy link

deniszh commented Aug 21, 2018

Could this issue be addressed, pretty please? Thanks!

@Dieterbe
Copy link
Contributor

@deniszh sorry for the delay. i've decided to merge this PR but would still like to implement a more proper fix later.

@Dieterbe Dieterbe self-requested a review August 21, 2018 10:56
@Dieterbe Dieterbe merged commit 6ea2e60 into grafana:master Aug 21, 2018
@Dieterbe
Copy link
Contributor

@deniszh please let us know how latest master works for you, thanks!

@deniszh
Copy link

deniszh commented Aug 21, 2018

👍
It works, thanks a lot!

@shanson7 shanson7 deleted the fixSingleMode branch August 21, 2018 20:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants