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

Add HTTPS and Basic auth #79

Open
roidelapluie opened this issue Jan 12, 2021 · 3 comments
Open

Add HTTPS and Basic auth #79

roidelapluie opened this issue Jan 12, 2021 · 3 comments

Comments

@roidelapluie
Copy link
Member

I would like to add HTTPS and Basic auth here, but I have some questions that influx experts might help grasp

Would this exporter still work with https?
Should we disable udp when we enable https?
Do you have further concerns about influx compat if we add tls here?

@matthiasr
Copy link
Contributor

InfluxDB supports TLS so there should be no problem there. As far as I can tell, when you enable it it becomes the only endpoint, so I think disabling UDP in that case makes sense. Maybe we should, for simplicity, disable it by default?

@matthiasr
Copy link
Contributor

As of 2.0, InfluxDB also dropped the UDP support.

@matthiasr
Copy link
Contributor

Also, 2.0 only supports token authentication. We can do that later (#80) but I think it would be good if we could support that in general.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants