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.

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

Adding options for basic auth to API calls #262

Closed
reinhard-brandstaedter opened this issue May 26, 2021 · 1 comment · Fixed by #263
Closed

Adding options for basic auth to API calls #262

reinhard-brandstaedter opened this issue May 26, 2021 · 1 comment · Fixed by #263
Labels
enhancement New feature or request
Milestone

Comments

@reinhard-brandstaedter
Copy link
Contributor

Proposal:
I'm seeing implementations where additional (basic) authentication proxies are being placed in front of InfluxDBs (mainly 1.8 versions where auth is disabled). This makes it difficult to use the compatibility API as basic auth headers cannot be added, only the Authorization: Token {user}:{pwd} would be added. As described here.
Would it be possible to also pass a Basic Auth header so that a auth proxy would be able to process the API requests in such cases?

Current behavior:
Writing to the compatibility API to a InfluxDb 1.8 which has auth disabled but uses a proxy with basic auth is not possible

Desired behavior:
It would be good to allow these special setups with authenticating proxies and influxdbs with not auth enabled

Alternatives considered:
Reverting to either not use the new influxdb-client or manual line protocol posting with basic auth (not very efficient)

Use case:
It would help in migrating solutions that need to write to both 1.8 (with special setups) and 2.0 influxDBs

@bednar
Copy link
Contributor

bednar commented May 26, 2021

@reinhard-brandstaedter, thanks for suggestion, We will take a look.

@bednar bednar added the enhancement New feature or request label May 26, 2021
@bednar bednar added this to the 1.18.0 milestone May 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants