Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DOCS: EVP_PKEY_get_default_digest_{name,nid}() - add params description #20430

Closed
wants to merge 4 commits into from

Conversation

levitte
Copy link
Member

@levitte levitte commented Mar 3, 2023

This includes both OSSL_PARAM(3) parameter keys and legacy pkey_ctrl()
control commands.

Fixes #20428

This includes both OSSL_PARAM(3) parameter keys and legacy pkey_ctrl()
control commands.

Fixes openssl#20428
@levitte levitte added branch: master Merge to master branch approval: review pending This pull request needs review by a committer approval: otc review pending This pull request needs review by an OTC member branch: 3.0 Merge to openssl-3.0 branch branch: 3.1 Merge to openssl-3.1 labels Mar 3, 2023
…escription

New message:

DOCS: provider-keymgmt(7) - params for EVP_PKEY_get_default_digest_{name,nid}()

This describes them in detail in provider-keymgmt(7), and adds references
in EVP_PKEY_get_default_digest_name(3).  It also adds some information about
the legacy controls used by EVP_PKEY_get_default_digest_nid(3).
doc/man7/provider-keymgmt.pod Outdated Show resolved Hide resolved
doc/man7/provider-keymgmt.pod Outdated Show resolved Hide resolved
@paulidale paulidale 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 Mar 6, 2023
@t8m t8m added triaged: documentation The issue/pr deals with documentation (errors) tests: exempted The PR is exempt from requirements for testing labels Mar 6, 2023
@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 Mar 7, 2023
@openssl-machine
Copy link
Collaborator

This pull request is ready to merge

openssl-machine pushed a commit that referenced this pull request Mar 7, 2023
…ame,nid}()

This describes them in detail in provider-keymgmt(7).

Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Paul Dale <pauli@openssl.org>
(Merged from #20430)
@levitte
Copy link
Member Author

levitte commented Mar 7, 2023

Merged

master:
ac57336 DOCS: provider-keymgmt(7) - params for EVP_PKEY_get_default_digest_{name,nid}()

3.1:
5b2fe0b DOCS: provider-keymgmt(7) - params for EVP_PKEY_get_default_digest_{name,nid}()

3.0:
4e56028 DOCS: provider-keymgmt(7) - params for EVP_PKEY_get_default_digest_{name,nid}()

@levitte levitte closed this Mar 7, 2023
@levitte levitte deleted the fix-20428 branch March 7, 2023 06:59
openssl-machine pushed a commit that referenced this pull request Mar 7, 2023
…ame,nid}()

This describes them in detail in provider-keymgmt(7).

Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Paul Dale <pauli@openssl.org>
(Merged from #20430)

(cherry picked from commit ac57336)
openssl-machine pushed a commit that referenced this pull request Mar 7, 2023
…ame,nid}()

This describes them in detail in provider-keymgmt(7).

Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Paul Dale <pauli@openssl.org>
(Merged from #20430)

(cherry picked from commit ac57336)
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 Merge to master branch branch: 3.0 Merge to openssl-3.0 branch branch: 3.1 Merge to openssl-3.1 tests: exempted The PR is exempt from requirements for testing triaged: documentation The issue/pr deals with documentation (errors)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Default and mandatory digest for provider-based signature algorithms
5 participants