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

crypto: use compatible version of EVP_CIPHER_name #38925

Closed
wants to merge 1 commit into from

Conversation

codebytere
Copy link
Member

EVP_CIPHER_name is only exposed via OpenSSL, but it's defined in the source code as OBJ_nid2sn(EVP_CIPHER_nid(e)), both of which are available in BoringSSL and OpenSSL. This PR switches to use that instead for better compatibility.

@codebytere codebytere requested a review from jasnell June 4, 2021 08:39
@github-actions github-actions bot added c++ Issues and PRs that require attention from people who are familiar with C++. crypto Issues and PRs related to the crypto subsystem. needs-ci PRs that need a full CI run. labels Jun 4, 2021
@bl-ue
Copy link
Contributor

bl-ue commented Jun 4, 2021

Great catch 😄

Copy link
Member

@richardlau richardlau left a comment

Choose a reason for hiding this comment

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

Do we need a comment in the code to stop someone changing it back?

@codebytere codebytere force-pushed the use-compatible-fn branch 2 times, most recently from 61057ac to 9c4a94b Compare June 4, 2021 15:39
@nodejs nodejs deleted a comment from nodejs-github-bot Jun 7, 2021
@nodejs-github-bot
Copy link
Collaborator

@nodejs nodejs deleted a comment from nodejs-github-bot Jun 7, 2021
@nodejs nodejs deleted a comment from nodejs-github-bot Jun 7, 2021
@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

codebytere added a commit that referenced this pull request Jun 8, 2021
PR-URL: #38925
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
@codebytere
Copy link
Member Author

Landed in 911ff34

@codebytere codebytere closed this Jun 8, 2021
@codebytere codebytere deleted the use-compatible-fn branch June 8, 2021 08:59
targos pushed a commit that referenced this pull request Jun 11, 2021
PR-URL: #38925
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
@danielleadams danielleadams mentioned this pull request Jun 14, 2021
@richardlau
Copy link
Member

Doesn't land cleanly on v14.x-staging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c++ Issues and PRs that require attention from people who are familiar with C++. crypto Issues and PRs related to the crypto subsystem. needs-ci PRs that need a full CI run.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants