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 support for '--web.config.file' flag and TLS authentication #130

Open
chillriver opened this issue Mar 16, 2022 · 5 comments
Open

Add support for '--web.config.file' flag and TLS authentication #130

chillriver opened this issue Mar 16, 2022 · 5 comments

Comments

@chillriver
Copy link

Hello,

how does the topic already say. I have that error when I started the service from systemd an read in the log from journalctl:

apache_exporter: error: unknown long flag '--web.config.file', try --help

I want to add a configfile in order to use TLS and define the certspath. Here is my configfragment in the unitfile from systemd.

ExecStart=/usr/local/bin/apache_exporter
--web.config.file="/etc/apache_exporter/web.yml"
--scrape_uri="http://localhost/server-status?auto"
--telemetry.address="0.0.0.0:9117"

Version of my apache_exporter v0.11.0

@Lusitaniae
Copy link
Owner

hi @chillriver, that's currently not a supported option

Feel free to submit a PR in order to add that feature

@Lusitaniae Lusitaniae changed the title error: unknown long flag '--web.config.file' Add support for '--web.config.file' flag and TLS authentication Mar 16, 2022
@chillriver
Copy link
Author

hi @Lusitaniae,

thanks for your reply. I tried something to test and made lucky result. It works!!

I changed follow: --web.config.file=.....

to : --web.config=.....

The configfile is loading and now I can pull from my webbrowser with TLS. Can you confirm this setting?

@Lusitaniae
Copy link
Owner

That's good to hear

Seems I rushed in my reply and give you wrong information!

Would you like to make a quick PR to update the parameter mentioned in the README at https://github.com/Lusitaniae/apache_exporter/blob/aee05657dae4661965fbd40df3fb99fc43b5c016/README.md#tls-and-basic-authentication and anything else you think relevant?

@chillriver
Copy link
Author

chillriver commented Mar 18, 2022

Yes I can edit the README and make a PR. It should be change the TLS chapter and also the help on flags.

@Lusitaniae
Copy link
Owner

Much appreciated!

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