crypto: move DEP0113 to End-of-Life by tniessen · Pull Request #26249 · nodejs/node (original) (raw)

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 andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Conversation9 Commits3 Checks0 Files changed

Conversation

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters

[ Show hidden characters]({{ revealButtonHref }})

tniessen

Cipher.setAuthTag and Decipher.getAuthTag have been runtime deprecated in node 11 and their existence still does not make sense. It is not urgent to remove them, but it also should do no harm. (Calling them will always throw anyway, and that behavior will still be the same after removal, except that the reason will be different.)

Checklist

@tniessen

@tniessen tniessen added the semver-major

PRs that contain breaking changes and should be released in the next major version.

label

Feb 21, 2019

@nodejs-github-bot

cjihrig

sam-github

@tniessen

@tniessen

cjihrig

Trott

Trott

Trott

@Trott

I've added the notable change label. However, I'm not sure if this is really a notable change. But we do try to mark all deprecations as notable. Feel free to remove if you disagree.

@tniessen

@tniessen

Thanks @Trott, I used your second suggestion! 😃

@tniessen

@tniessen tniessen added the author ready

PRs that have at least one approval, no pending requests for changes, and a CI started.

label

Mar 1, 2019

tniessen added a commit that referenced this pull request

Mar 2, 2019

@tniessen

PR-URL: #26249 Reviewed-By: Sam Roberts vieuxtech@gmail.com Reviewed-By: Colin Ihrig cjihrig@gmail.com Reviewed-By: Rich Trott rtrott@gmail.com

@tniessen

Thanks for reviewing, landed in b8018f4.

@tniessen tniessen removed the author ready

PRs that have at least one approval, no pending requests for changes, and a CI started.

label

Mar 2, 2019

BethGriggs added a commit that referenced this pull request

Apr 22, 2019

@BethGriggs

Notable changes:

PR-URL: #26930

BethGriggs added a commit that referenced this pull request

Apr 23, 2019

@BethGriggs

Notable changes:

PR-URL: #26930

BethGriggs added a commit that referenced this pull request

Apr 23, 2019

@BethGriggs

Notable changes:

PR-URL: #26930

This was referenced

Apr 23, 2019

This was referenced

Apr 23, 2019

Labels

crypto

Issues and PRs related to the crypto subsystem.

deprecations

Issues and PRs related to deprecations.

notable-change

PRs with changes that should be highlighted in changelogs.

semver-major

PRs that contain breaking changes and should be released in the next major version.