Skip to content

benchmark: increase crypto DSA keygen params - #40416

Merged
mscdex merged 1 commit into
nodejs:masterfrom
mscdex:benchmark-crypto-fix-error
Oct 16, 2021
Merged

benchmark: increase crypto DSA keygen params#40416
mscdex merged 1 commit into
nodejs:masterfrom
mscdex:benchmark-crypto-fix-error

Conversation

@mscdex

Copy link
Copy Markdown
Contributor

OpenSSL 3.0 increased the minimum values for these parameters.

Fixes: #40410

@mscdexmscdex added crypto Issues and PRs related to the crypto subsystem. benchmark Issues and PRs related to the benchmark subsystem. labels Oct 11, 2021
@MesteeryMesteery added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Oct 12, 2021
@nodejs-github-bot

This comment has been minimized.

@nodejs-github-bot

This comment has been minimized.

@mscdex
mscdexforce-pushed the benchmark-crypto-fix-error branch from 9a616c1 to 282d1e1CompareOctober 16, 2021 01:44
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

OpenSSL 3.0 increased the minimum values for these parameters.
PR-URL: nodejs#40416Fixes: nodejs#40410
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Zijian Liu <lxxyxzj@gmail.com>
@mscdex
mscdexforce-pushed the benchmark-crypto-fix-error branch from 282d1e1 to ed01811CompareOctober 16, 2021 10:09
@mscdex
mscdex merged commit ed01811 into nodejs:masterOct 16, 2021
@mscdex
mscdex deleted the benchmark-crypto-fix-error branch October 16, 2021 10:10
@BethGriggsBethGriggs mentioned this pull request Oct 18, 2021
2 tasks
targos pushed a commit that referenced this pull request Nov 4, 2021
OpenSSL 3.0 increased the minimum values for these parameters.
PR-URL: #40416Fixes: #40410
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Zijian Liu <lxxyxzj@gmail.com>
@BethGriggsBethGriggs mentioned this pull request Nov 26, 2021
1 task
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.benchmarkIssues and PRs related to the benchmark subsystem.cryptoIssues and PRs related to the crypto subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

crypto keygen benchmark failing after OpenSSL 3.0 upgrade

8 participants

@mscdex@nodejs-github-bot@lpinca@targos@tniessen@richardlau@Lxxyx@Mesteery