Skip to content

Make the SM2 group be the default group for SM2 algorithm#14684

Closed
t8m wants to merge 1 commit intoopenssl:masterfrom
t8m:sm2-curve-default
Closed

Make the SM2 group be the default group for SM2 algorithm#14684
t8m wants to merge 1 commit intoopenssl:masterfrom
t8m:sm2-curve-default

Conversation

@t8m
Copy link
Copy Markdown
Member

@t8m t8m commented Mar 25, 2021

Fixes #14481

Checklist
  • tests are added or updated

@t8m t8m added approval: review pending This pull request needs review by a committer branch: master Applies to master branch labels Mar 25, 2021
@t8m t8m force-pushed the sm2-curve-default branch from 6955508 to 4d782dc Compare March 25, 2021 14:25
@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 Mar 25, 2021
@levitte
Copy link
Copy Markdown
Member

levitte commented Mar 25, 2021

My approval is subject to the CI succeeding, of course. But, I like what I see :-)

Copy link
Copy Markdown
Member

@mattcaswell mattcaswell left a comment

Choose a reason for hiding this comment

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

Approved assuming CIs pass

@openssl-machine
Copy link
Copy Markdown
Collaborator

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.

@t8m t8m 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 26, 2021
openssl-machine pushed a commit that referenced this pull request Mar 26, 2021
Fixes #14481

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

t8m commented Mar 26, 2021

Merged to master. Thank you for the reviews.

@t8m t8m closed this Mar 26, 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

You must set a group name when generating an SM2 key

5 participants