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

[Improvement] Support TLS Expiry alerts also for CA certs in cert chain #2593

Closed
1 task done
skaempfe opened this issue Jan 12, 2023 · 0 comments · Fixed by #2594
Closed
1 task done

[Improvement] Support TLS Expiry alerts also for CA certs in cert chain #2593

skaempfe opened this issue Jan 12, 2023 · 0 comments · Fixed by #2594
Labels
feature-request Request for new features to be added

Comments

@skaempfe
Copy link
Contributor

⚠️ Please verify that this feature request has NOT been suggested before.

  • I checked and didn't find similar feature request

🏷️ Feature Request Type

Other

🔖 Feature description

Currently only the first certificate within the certificate chain is checked and alerted for expiry but not the other (signing) certificates within the chain.
When monitoring the certificate expiry days, not only the first (server) certificate should be evaluated but also all certificates within the certificate chain (each issuer certificate) and then alerted separately.
This raises the awareness for sys admins for required ca-chain modifications/replacements.

✔️ Solution

See provided pull request.

❓ Alternatives

No response

📝 Additional Context

Last november we had a situation where an intermediate CA certificate expired (QuoVadis Global SSL ICA G3) despite the fact that the signed server certificate still was valid for 10 more month. This is a rather unusual situation and even more complicated to analyze and understand.

@skaempfe skaempfe added the feature-request Request for new features to be added label Jan 12, 2023
skaempfe added a commit to skaempfe/uptime-kuma that referenced this issue Jan 12, 2023
skaempfe added a commit to skaempfe/uptime-kuma that referenced this issue Jan 12, 2023
…epresent what id does. Evaluate certificate expiry from all certs in chain. Send a separate notification for every cert in chain, including cert type and CN.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request Request for new features to be added
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant