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

Concat request path to Base URL path. #4

Merged
merged 1 commit into from
Nov 17, 2017
Merged

Concat request path to Base URL path. #4

merged 1 commit into from
Nov 17, 2017

Conversation

sl1pm4t
Copy link
Contributor

@sl1pm4t sl1pm4t commented Nov 17, 2017

If the base URL includes a path component, don’t overwrite this with the
request path.
This is necessary when Grafana is running behind a reverse
proxy that is doing path based routing.

request path. This is necessary when Grafana is running behind a reverse
proxy that is doing path based routing.
@sl1pm4t
Copy link
Contributor Author

sl1pm4t commented Nov 17, 2017

Hi @apparentlymart
I'm sending this PR to your repo instead of the raintank repo, because you're vendoring this in the Terraform grafana provider - which is really where I need this change.

@apparentlymart
Copy link
Contributor

Seems reasonable to me, @sl1pm4t! Thanks.

This was forked for the Terraform provider in order to get some new functionality I needed to have Terraform manage data sources. At some point it would probably be a good idea to try to upstream all of these changes to raintank/grafana-api-golang-client, but unfortunately I don't have time to work on that right now. 😖

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.

2 participants