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 ValidationStatus to CertChainValidationResult #1146

Conversation

atc0005
Copy link
Owner

@atc0005 atc0005 commented Dec 7, 2024

Extend the CertChainValidationResult interface to require an implementation of ValidationStatus method.

Add/Update implementation for each of:

  • ExpirationValidationResult
  • SANsListValidationResult
  • HostnameValidationResult

Update the ExpirationValidationResult implementation to use existing helper methods to determine validation status instead of repeating expiration checks.

Add ValidationStatus* keyword constants for use with all CertChainValidationResult implementations.

Extend the CertChainValidationResult interface to require
an implementation of ValidationStatus method.

Add/Update implementation for each of:

- ExpirationValidationResult
- SANsListValidationResult
- HostnameValidationResult

Update the ExpirationValidationResult implementation to use
existing helper methods to determine validation status instead
of repeating expiration checks.

Add `ValidationStatus*` keyword constants for use with all
CertChainValidationResult implementations.
@atc0005 atc0005 added this to the v0.25.3-alpha.3 milestone Dec 7, 2024
@atc0005 atc0005 self-assigned this Dec 7, 2024
@atc0005 atc0005 merged commit 15aceaa into development Dec 7, 2024
28 of 29 checks passed
@atc0005 atc0005 deleted the extend-certchainvalidationresult-to-include-validationstatus-method branch December 7, 2024 13:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant