Skip to content

Comments

Make more use of X509_add_certs(); minor related code & comments cleanup#14436

Merged
openssl-machine merged 1 commit intoopenssl:masterfrom
siemens:more_use_X509_add_certs
Mar 6, 2021
Merged

Make more use of X509_add_certs(); minor related code & comments cleanup#14436
openssl-machine merged 1 commit intoopenssl:masterfrom
siemens:more_use_X509_add_certs

Conversation

@DDvO
Copy link
Contributor

@DDvO DDvO commented Mar 5, 2021

This is a minor follow-up on #12615.

@paulidale paulidale 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 Mar 5, 2021
@openssl-machine
Copy link
Collaborator

24 hours has passed since 'approval: done' was set, but this PR has failing CI tests. Once the tests pass it will get moved to 'approval: ready to merge' automatically, alternatively please review and set the label manually.

@DDvO DDvO force-pushed the more_use_X509_add_certs branch from e2e26ee to 0dca5ed Compare March 6, 2021 12:09
@openssl-machine openssl-machine merged commit 0dca5ed into openssl:master Mar 6, 2021
@DDvO
Copy link
Contributor Author

DDvO commented Mar 6, 2021

Merged after rebasing to solve merge conflict - thanks @paulidale

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approval: done This pull request has the required number of approvals branch: master Applies to master branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants