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.

Sign up for GitHub

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

Jump to bottom

example of how to prevent URI losing #2751

Closed
wants to merge 4 commits into from
Closed

example of how to prevent URI losing #2751

wants to merge 4 commits into from

Conversation

caio2k
Copy link

@caio2k caio2k commented May 2, 2017

This is an example as I'm not a GO expert!
It tries to fix #2750

@danielnelson
Copy link
Contributor

I think the tests should pass if you add the import for path.

@lpic10
Copy link
Contributor

lpic10 commented Jun 7, 2017

This probably fails because changing u.Path changes the underlying url object, thus it maybe be adding "/query/write/... etc" to requests.

@danielnelson danielnelson added the feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin label Aug 24, 2017
@dimrozakis dimrozakis mentioned this pull request Sep 13, 2017
Merged
3 tasks
@danielnelson
Copy link
Contributor

Closing in favor of #3224

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

influxdb - URLS' URI is ignored when sending HTTP request
3 participants
@caio2k @danielnelson @lpic10