-
Notifications
You must be signed in to change notification settings - Fork 0
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 Root
validation check option
#124
Comments
Perhaps a |
has root cert
validation check optionRoot
validation check option
No. While memorable, this implies a specific expected state and doesn't fit well with the other neutral check names. I think just |
Refs: I need to dig further into this, specifically whether best practices have changed and whether this RFC has been "clarified" or deprecated. |
From https://www.rfc-editor.org/rfc/rfc4346#section-7.4.2:
|
An initial implementation made it to the development branch (2024-12?), but after testing builds with this support alongside "intermediates" validation support I am leaning towards removing it. The specific detection this offers can be folded into another validation check and/or be covered by the generated payload and perfdata metrics. |
Food for thought regarding outgoing root certificates:
In this case, the From the latter FAQ link:
Not sure this requires any action on the part of the plugin, but it's something to consider. |
Follow-up item:
The The The replacement intermediate is not listed within an AIA URL for the previously issued certificates. |
This validation check would look for the presence of a root certificate in the chain and note any found.
I should also lookup current best practices and add a reference link both here in the documentation. If best practices indicate that root certificates are allowed, but discouraged, then a WARNING could be issued. If best practices indicate that root certificates are disallowed (entirely), then a CRITICAL state could be flagged.
It may be best to have this validation check set as ignored by default.
See also:
The text was updated successfully, but these errors were encountered: