Skip to content

ee-self-signed.pem: Restore original version, adding -attime to 25-test_verify.t#15499

Closed
DDvO wants to merge 2 commits intoopenssl:masterfrom
siemens:fix-ee-self-signed.pem
Closed

ee-self-signed.pem: Restore original version, adding -attime to 25-test_verify.t#15499
DDvO wants to merge 2 commits intoopenssl:masterfrom
siemens:fix-ee-self-signed.pem

Conversation

@DDvO
Copy link
Contributor

@DDvO DDvO commented May 27, 2021

Fixup for #12549, which updated and prolonged the validity of the test certificate but turned it into a CA cert,
which defeated the purpose of the test case "accept trusted self-signed EE cert excluding key usage keyCertSign".
See also #12549 (comment).

Checklist
  • tests are added or updated

@DDvO DDvO added approval: otc review pending triaged: bug The issue/pr is/fixes a bug labels May 27, 2021
@t8m t8m added approval: done This pull request has the required number of approvals branch: master Applies to master branch and removed approval: otc review pending labels May 27, 2021
@openssl-machine openssl-machine added approval: ready to merge The 24 hour grace period has passed, ready to merge and removed approval: done This pull request has the required number of approvals labels May 28, 2021
@openssl-machine
Copy link
Collaborator

This pull request is ready to merge

openssl-machine pushed a commit that referenced this pull request Jun 2, 2021
openssl-machine pushed a commit that referenced this pull request Jun 2, 2021
@DDvO
Copy link
Contributor Author

DDvO commented Jun 2, 2021

Merged - thanks @t8m

@DDvO DDvO closed this Jun 2, 2021
devnexen pushed a commit to devnexen/openssl that referenced this pull request Jul 7, 2021
devnexen pushed a commit to devnexen/openssl that referenced this pull request Jul 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approval: ready to merge The 24 hour grace period has passed, ready to merge branch: master Applies to master branch triaged: bug The issue/pr is/fixes a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants