Skip to content

Comments

doc: fix macro name#17125

Closed
paulidale wants to merge 1 commit intoopenssl:masterfrom
paulidale:doc-fix-x509-cert
Closed

doc: fix macro name#17125
paulidale wants to merge 1 commit intoopenssl:masterfrom
paulidale:doc-fix-x509-cert

Conversation

@paulidale
Copy link
Contributor

@paulidale paulidale commented Nov 24, 2021

OSSL_STORE_INFO_X509 doesn't exist. It should be OSSL_STORE_INFO_CERT.

Fixes #17121

  • documentation is added or updated
  • tests are added or updated

@paulidale
doc: fix macro name
8799cea 
OSSL_STORE_INFO_X509 doesn't exist.  It should be OSSL_STORE_INFO_CERT.

Fixes openssl#17121
@paulidale paulidale self-assigned this Nov 24, 2021
@paulidale paulidale added approval: review pending This pull request needs review by a committer branch: 3.0 Applies to openssl-3.0 branch branch: master Applies to master branch triaged: documentation The issue/pr deals with documentation (errors) labels Nov 24, 2021
@levitte levitte added approval: done This pull request has the required number of approvals and removed approval: review pending This pull request needs review by a committer labels Nov 24, 2021
@levitte levitte mentioned this pull request Nov 24, 2021
Closed
@levitte levitte added the branch: 1.1.1 Applies to OpenSSL_1_1_1-stable branch (EOL) label Nov 24, 2021
@levitte
Copy link
Member

levitte commented Nov 24, 2021

This should be applied to 1.1.1 as well. I've labeled accordingly

@openssl-machine
Copy link
Collaborator

openssl-machine commented Nov 25, 2021

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set. Please review the updates and set the label manually.

openssl-machine pushed a commit that referenced this pull request Nov 25, 2021
@paulidale
doc: fix macro name
01fde90 
OSSL_STORE_INFO_X509 doesn't exist.  It should be OSSL_STORE_INFO_CERT.

Fixes #17121

Reviewed-by: Richard Levitte <[email protected]>
(Merged from #17125)
@paulidale
Copy link
Contributor Author

paulidale commented Nov 25, 2021

Merged to 1.1.1, 3.0 and master. Thanks for the review and the extra branch suggestion.

@paulidale paulidale closed this Nov 25, 2021
@paulidale paulidale deleted the doc-fix-x509-cert branch November 25, 2021 06:06
openssl-machine pushed a commit that referenced this pull request Nov 25, 2021
@paulidale
doc: fix macro name
7f32117 
OSSL_STORE_INFO_X509 doesn't exist.  It should be OSSL_STORE_INFO_CERT.

Fixes #17121

Reviewed-by: Richard Levitte <[email protected]>
(Merged from #17125)

(cherry picked from commit 01fde90)
openssl-machine pushed a commit that referenced this pull request Nov 25, 2021
@paulidale
doc: fix macro name
5f42292 
OSSL_STORE_INFO_X509 doesn't exist.  It should be OSSL_STORE_INFO_CERT.

Fixes #17121

Reviewed-by: Richard Levitte <[email protected]>
(Merged from #17125)

(cherry picked from commit 01fde90)
baentsch added a commit to open-quantum-safe/openssl that referenced this pull request Jun 23, 2022
@baentsch @luyahan
@devnexen @davidben @t8m @paulidale @mattcaswell @levitte @ueno @ischwarze @bbbrumley @tmshort @ifranzki @romen @bernd-edlinger @uudiin @zhaozg @Dani-Hub @ellert @lprimak @mspncp @beldmit @PeiweiHu @x2018 @allanjude @DDvO @dongbeiouba @KanPlus @roussosalex @pkubaj @jmgdjgpp @Thomas1664 @JiangJias @faramir-dev @hlandau @juergenchrist @reaperhulk @basavesh @frasertweedale
Upstream 111p merge (#375