boynux / squid-exporter

Squid Prometheus Exporter
https://www.boynux.com/squid-exporter
MIT License
134 stars 53 forks source link

Add Basic authentication #76

Closed techstudio-project closed 1 year ago

techstudio-project commented 1 year ago

Considering adding Basic certification?

For security reasons, /metrics endpoint Access should be authenticated before it can be accessed

boynux commented 1 year ago

Hi,

thank for making this request, do you know where I can find more information about squids support for cert authentication? Note that the current implementation of the exporter can use basic auth credentials, but adding cert auths wouldn't be that hard I think.

Also if you want to contribute please send me a PR

techstudio-project commented 1 year ago

sorry, I wrote it wrong, I mean add basic authentication on squid-exporter

boynux commented 1 year ago

Basic auth is already supported. Please check that out and let me know if you need further help.