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

Feature Request: Support specifying path prefix for InfluxDB URL #452

Open
yoshigev opened this issue Oct 23, 2022 · 5 comments · May be fixed by #530
Open

Feature Request: Support specifying path prefix for InfluxDB URL #452

yoshigev opened this issue Oct 23, 2022 · 5 comments · May be fixed by #530
Labels
enhancement New feature or request

Comments

@yoshigev
Copy link

I'm opening a feature request for feature that was already done on 1.8: influxdata/influxdb#10294
It was done by @bthesorceror on influxdata/influxdb#16599.

The idea is to allow specifying a sub-path on the URL, so the client could work when an HTTP reverse proxy is routing the requests based on the path. For example, using a URL: https://example.com/influxdb/.

Apparently, this support was removed on version 2, as the flags no longer exists.

Is it possible to support this?

Thanks,
Yehoshua
@jeffreyssmith2nd jeffreyssmith2nd added the enhancement New feature or request label Oct 24, 2022
@Fuseteam
Copy link

Seems related to influxdata/influxdb#15721

@jdeus
Copy link

jdeus commented Aug 30, 2023

Any news?

@tisonkun tisonkun mentioned this issue Mar 11, 2024
Closed
@tisonkun
Copy link

Even we change the apiConfig.Host = params.Host.Host to use whatever user passes in.

@tisonkun
Copy link

But it related to some generated code logic. So I can hardly help. Basically we pass the "host" as "https://example.com/influxdb/" and the path part should be conveyed to the final client config and it should work.

@tisonkun tisonkun linked a pull request Mar 11, 2024 that will close this issue
Open
@tisonkun
Copy link

But I anyway made it. Here is a patch and I verify it locally.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: support switching basepath tisonkun/influx-cli
5 participants
@yoshigev @Fuseteam @tisonkun @jeffreyssmith2nd @jdeus