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

Use ssl settings in the haproxy module #14579

Closed
cafuego opened this issue Nov 18, 2019 · 5 comments
Closed

Use ssl settings in the haproxy module #14579

cafuego opened this issue Nov 18, 2019 · 5 comments
Assignees
Labels
enhancement Team:Integrations Label for the Integrations team

Comments

@cafuego
Copy link

cafuego commented Nov 18, 2019

Describe the enhancement:
Honour ssl settings in the haproxy module. The documentation suggests it is possible to override ssl verification and other settings for the haproxy module, but this is not the case; these override settings are ignored.

Describe a specific use case for the enhancement or feature:
In cases where haproxy is configured to only allow access to the stats backend via https using a self-signed certificate, the haproxy module throws errors. By allowing users to specify that ssl validation errors should be ignored, stats may be fetched and processed.

@cafuego
Copy link
Author

cafuego commented Nov 18, 2019

Is this simply a matter of including settings: ["ssl"] in the metricbeat/module/haproxy/_meta/fields.yml file?

@exekias
Copy link
Contributor

exekias commented Nov 18, 2019

Thanks for opening this @cafuego, I'm afraid it's not that simple, as that would only update docs. This will require some changes in haproxy code to honor SSL settings

@cafuego
Copy link
Author

cafuego commented Nov 19, 2019

Fair enough, I won't make a useless pull request then ;-)

@mtojek
Copy link
Contributor

mtojek commented Feb 18, 2020

All PRs merged. Resolving.

@cafuego
Copy link
Author

cafuego commented Mar 5, 2020

Thank you for doing this :-) Has it made it into a release yet?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Team:Integrations Label for the Integrations team
Projects
None yet
Development

No branches or pull requests

4 participants