Skip to content

More CRL test cases#439

Closed
tnytown wants to merge 1 commit intoC2SP:mainfrom
trail-of-forks:ap/crl-cases
Closed

More CRL test cases#439
tnytown wants to merge 1 commit intoC2SP:mainfrom
trail-of-forks:ap/crl-cases

Conversation

@tnytown
Copy link
Copy Markdown
Contributor

@tnytown tnytown commented May 21, 2025

Some CRL cases from #174.

  • xfail: CRL has wrong version (not 2, i.e. literal 1)
  • xfail: thisUpdate and nextUpdate must use the same date encoding rules as the rest of RFC 5280
  • xfail: revokedCertificates must not be present if empty (i.e. must not be an empty SEQUENCE OF)
  • pass: CRL contains revocation but not active yet (revocation date in the future)
  • xfail: CRL root is missing cRLSign: TRUE
  • pass: CRL revokes a similar-but-differ cert (same except for serial number, which is all that matters)
  • xfail: CRL has the wrong issuer (signed by root but issuer name does not match)

@tnytown tnytown marked this pull request as draft May 21, 2025 19:28
@tnytown tnytown force-pushed the ap/crl-cases branch 13 times, most recently from c5cbf23 to 00c0057 Compare May 22, 2025 15:13
@tnytown tnytown marked this pull request as ready for review May 22, 2025 15:17
@woodruffw
Copy link
Copy Markdown
Collaborator

I'm going to reopen this from an upstream branch to push the regression automation through.

@woodruffw
Copy link
Copy Markdown
Collaborator

#440.

@woodruffw woodruffw closed this May 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants