Skip to content

Commit

Permalink
fix: add warning message for non-revokable certificate (notaryproject…
Browse files Browse the repository at this point in the history
…#479)

Fix:
- added warning message for non-revokable certificate

---------

Signed-off-by: Junjie Gao <[email protected]>
Signed-off-by: Patrick Zheng <[email protected]>
  • Loading branch information
JeyJeyGao authored and Two-Hearts committed Dec 9, 2024
1 parent ffc04d8 commit 0a9ff2a
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions verifier/verifier.go
Original file line number Diff line number Diff line change
Expand Up @@ -704,6 +704,10 @@ func revocationFinalResult(certResults []*revocationresult.CertRevocationResult,
revokedCertSubject = problematicCertSubject
}
}

if i < len(certResults)-1 && certResult.Result == revocationresult.ResultNonRevokable {
logger.Warnf("Certificate #%d in the chain with subject %v neither has an OCSP nor a CRL revocation method.", (i + 1), cert.Subject.String())
}
}
if revokedFound {
problematicCertSubject = revokedCertSubject
Expand Down

0 comments on commit 0a9ff2a

Please sign in to comment.