search

influxdata / influx-cli

CLI for managing resources in InfluxDB v2
MIT License
61 stars 22 forks source link

feat: init api server configs with path of host  #511

Open DonCamillo72 opened 1 year ago

DonCamillo72 commented 1 year ago

Reason: If the influxdb server runs behind a reverse proxy with url e.g. "https://anyhost/influxdb" the influx-cli could not access the url because the host base path of the api url is always empty.

Idea: The information of the base can be passed in the cli with the host parameter like this:

influx backup --host "https://anyhost/influxdb"

Implementation: The base path is available in the parameter params.Host.Path and could be forwarded to the OperationServers of the generated client API

albrecht-j commented 11 months ago

Hi @jeffreyssmith2nd i saw you created the last release. Maybe you can take a look at this PR?

Thank you :)

albrecht-j commented 11 months ago

Hi @srebhan can you support us here? I think it is less effort then the last one ;-)

If not, whom can we contact here? Best Regards Johannes

srebhan commented 11 months ago

@albrecht-j sorry but this is not my field of expertise. ;-)

jeffreyssmith2nd commented 11 months ago

Hey @albrecht-j, I'm the right person for this. My apologies, I just lost track of this PR. I will get it reviewed before the next release.

albrecht-j commented 8 months ago

Hey @jeffreyssmith2nd any updates to this PR? If we need to change something please let us know :)

DonCamillo72 commented 7 months ago

Hi @jeffreyssmith2nd, have you already reviewed the PR? Please let us know if something need to be changed. Thank you!

albrecht-j commented 4 months ago

Hi @jeffreyssmith2nd,

do you have a schedule for the next release?

tisonkun commented 3 months ago

Seems a function duplicatie to https://github.com/influxdata/influx-cli/pull/530.

But the InfluxDB team doesn't care. I can have my fork but perhaps switch to some SDK.

I remember Python SDK v1 has such path. Not sure if v2 has.