-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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 AIA (Authority Information Access) #1350
Comments
can you share more on how it would be useful to the users? and the possible use-cases it will solve? |
Sure hopefully this helps Monitoring endpoints secured by https certificate but dont have the intimedatory certs within the chain |
can you share some examples of websites that blackbox_exporter fails on? maybe share a check config? |
Sure thing so if we use this site as the example https://incomplete-chain.badssl.com/ From a browser like Chrome this loads fine as AIA is enabled If we curl or a wget without AIA we get
Due to the missing intermediate cert NOTE mac has AIA now on curl so curl will work off mac but from a linux container etc it will fail as above then in blackbox exporter we have
then a job in prometheus for
The the test for https://badssl.com/ which is fine works as expected but https://incomplete-chain.badssl.com fails as blackbox doesnt support AIA to fill in the incomplete cert It would make sense to add a flag I think to enable it if its wanted for that module as I can see the argument that just setup certs properly and its not and issue but so many are missing the intermidate certs Hope thats helpful |
I read up more on AIA, and I think it makes sense for browsers to have the support it, but I can't think of the reasons to have a certificate that don't have the intimidatory certs (in cases where you control the target you are probing) From what I read online, it's generally not recommended to have a cert chain with missing intimidatory certs, but I also understand that you don't always control the targets you probe. we can consider this, feel free to send a PR but it should be behind the flag, and disabled by default. |
Can we please look at adding support for AIA (Authority Information Access)
https://www.rfc-editor.org/rfc/rfc3280#section-4.2.2.1
The text was updated successfully, but these errors were encountered: