Skip to content

Correct information about the default value of the -md parameter of the openssl cms command. #29175

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

Correct information about the default value of the -md parameter of the openssl cms command. #29175

wants to merge 1 commit into from

Conversation

@igus68
Copy link
Contributor

@igus68 igus68 commented Nov 19, 2025

SHA1 was still mentioned there.

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

@igus68 igus68 added branch: master Applies to master branch branch: 3.0 Applies to openssl-3.0 branch branch: 3.2 Applies to openssl-3.2 (EOL) branch: 3.3 Applies to openssl-3.3 branch: 3.4 Applies to openssl-3.4 branch: 3.5 Applies to openssl-3.5 branch: 3.6 Applies to openssl-3.6 triaged: documentation The issue/pr deals with documentation (errors) labels Nov 19, 2025
@igus68 igus68 closed this Nov 19, 2025
@igus68 igus68 reopened this Nov 19, 2025
@t8m t8m added approval: review pending This pull request needs review by a committer tests: exempted The PR is exempt from requirements for testing labels Nov 19, 2025
@t8m t8m requested a review from a team November 19, 2025 17:53
beldmit
beldmit approved these changes Nov 19, 2025
View reviewed changes
Copy link
Member

@beldmit beldmit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@beldmit beldmit 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 19, 2025
@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 Nov 20, 2025
@openssl-machine
Copy link
Collaborator

openssl-machine commented Nov 20, 2025

This pull request is ready to merge

openssl-machine pushed a commit that referenced this pull request Nov 21, 2025
@igus68 @nhorman
Correct information about the default value of the -md parameter
2a7b058 
of the openssl cms command.

Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Dmitry Belyavskiy <beldmit@gmail.com>
Reviewed-by: Paul Dale <paul.dale@oracle.com>
Reviewed-by: Neil Horman <nhorman@openssl.org>
(Merged from #29175)
openssl-machine pushed a commit that referenced this pull request Nov 21, 2025
@igus68 @nhorman
Correct information about the default value of the -md parameter
894e46f 
of the openssl cms command.

Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Dmitry Belyavskiy <beldmit@gmail.com>
Reviewed-by: Paul Dale <paul.dale@oracle.com>
Reviewed-by: Neil Horman <nhorman@openssl.org>
(Merged from #29175)

(cherry picked from commit 2a7b058)
openssl-machine pushed a commit that referenced this pull request Nov 21, 2025
@igus68 @nhorman
Correct information about the default value of the -md parameter
662362c 
of the openssl cms command.

Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Dmitry Belyavskiy <beldmit@gmail.com>
Reviewed-by: Paul Dale <paul.dale@oracle.com>
Reviewed-by: Neil Horman <nhorman@openssl.org>
(Merged from #29175)

(cherry picked from commit 2a7b058)
openssl-machine pushed a commit that referenced this pull request Nov 21, 2025
@igus68 @nhorman
Correct information about the default value of the -md parameter
47a47e0 
of the openssl cms command.

Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Dmitry Belyavskiy <beldmit@gmail.com>
Reviewed-by: Paul Dale <paul.dale@oracle.com>
Reviewed-by: Neil Horman <nhorman@openssl.org>
(Merged from #29175)

(cherry picked from commit 2a7b058)
openssl-machine pushed a commit that referenced this pull request Nov 21, 2025
@igus68 @nhorman
Correct information about the default value of the -md parameter
0e98b51 
of the openssl cms command.

Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Dmitry Belyavskiy <beldmit@gmail.com>
Reviewed-by: Paul Dale <paul.dale@oracle.com>
Reviewed-by: Neil Horman <nhorman@openssl.org>
(Merged from #29175)

(cherry picked from commit 2a7b058)
@nhorman
Copy link
Contributor

nhorman commented Nov 21, 2025

merged to all active branches, thank you!

@nhorman nhorman closed this Nov 21, 2025
openssl-machine pushed a commit that referenced this pull request Nov 21, 2025
@igus68 @nhorman
Correct information about the default value of the -md parameter
4c05d28 
of the openssl cms command.

Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Dmitry Belyavskiy <beldmit@gmail.com>
Reviewed-by: Paul Dale <paul.dale@oracle.com>
Reviewed-by: Neil Horman <nhorman@openssl.org>
(Merged from #29175)

(cherry picked from commit 2a7b058)
openssl-machine pushed a commit that referenced this pull request Nov 21, 2025
@igus68 @nhorman
Correct information about the default value of the -md parameter
088803f 
of the openssl cms command.

Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Dmitry Belyavskiy <beldmit@gmail.com>
Reviewed-by: Paul Dale <paul.dale@oracle.com>
Reviewed-by: Neil Horman <nhorman@openssl.org>
(Merged from #29175)

(cherry picked from commit 2a7b058)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@t8m t8m t8m approved these changes

@simo5 simo5 simo5 approved these changes

@beldmit beldmit beldmit approved these changes

@paulidale paulidale paulidale approved these changes

Assignees

No one assigned

Labels

approval: ready to merge The 24 hour grace period has passed, ready to merge branch: master Applies to master branch branch: 3.0 Applies to openssl-3.0 branch branch: 3.2 Applies to openssl-3.2 (EOL) branch: 3.3 Applies to openssl-3.3 branch: 3.4 Applies to openssl-3.4 branch: 3.5 Applies to openssl-3.5 branch: 3.6 Applies to openssl-3.6 tests: exempted The PR is exempt from requirements for testing triaged: documentation The issue/pr deals with documentation (errors)

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@igus68 @openssl-machine @nhorman @t8m @simo5 @beldmit @paulidale