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

feat: expose des-ede3 cipher #27992

Merged

Conversation

trop[bot]
Copy link
Contributor

@trop trop bot commented Mar 4, 2021

Backport of #27897

See that PR for details.

Notes: Added support for the des-ede3 cipher in node crypto.

@trop trop bot requested a review from a team as a code owner March 4, 2021 00:20
@electron-cation electron-cation bot added the new-pr 🌱 PR opened in the last 24 hours label Mar 4, 2021
@trop trop bot requested a review from nornagon March 4, 2021 00:20
@trop trop bot mentioned this pull request Mar 4, 2021
2 tasks
@trop trop bot added 12-x-y backport This is a backport PR backport/requested 🗳 semver/minor backwards-compatible functionality labels Mar 4, 2021
@electron-cation electron-cation bot removed the new-pr 🌱 PR opened in the last 24 hours label Mar 4, 2021
@jkleinsc
Copy link
Contributor

jkleinsc commented Mar 4, 2021

Merging as CI failures are known flakes unrelated to this PR.

@jkleinsc jkleinsc merged commit 189f7e0 into 12-x-y Mar 4, 2021
@release-clerk
Copy link

release-clerk bot commented Mar 4, 2021

Release Notes Persisted

Added support for the des-ede3 cipher in node crypto.

@trop trop bot deleted the trop/12-x-y-bp-feat-expose-des-ede3-cipher-1614817224701 branch March 4, 2021 20:52
@ChALkeR
Copy link
Contributor

ChALkeR commented Mar 10, 2021

This is labeled semver/minor but ended up in 12.0.1?
Wasn't this supposed to be 12.1.0, or am I missing something?

This was referenced Mar 15, 2021
@nornagon
Copy link
Member

nornagon commented Mar 22, 2021

This is labeled semver/minor but ended up in 12.0.1?
Wasn't this supposed to be 12.1.0, or am I missing something?

12.0.1 was mislabelled due to an internal tooling error and should have been 12.1.0. We've fixed the issue so it shouldn't happen again :)

@ChALkeR
Copy link
Contributor

ChALkeR commented Mar 22, 2021

Thanks! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
12-x-y backport/requested 🗳 backport This is a backport PR semver/minor backwards-compatible functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants