Home Original page

crypto: remove arbitrary UTF16 restriction by addaleax · Pull Request #29795 · nodejs/node

@addaleax

@nodejs-github-bot nodejs-github-bot added c++

Issues and PRs that require attention from people who are familiar with C++.

errors

Issues and PRs related to JavaScript errors originated in Node.js core.

labels

Oct 1, 2019

@addaleax addaleax added the author ready

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

label

Oct 1, 2019

BridgeAR

@addaleax @cjihrig 

Co-Authored-By: Colin Ihrig <cjihrig@gmail.com>

devnexen

Trott pushed a commit that referenced this pull request

Oct 3, 2019 
Since 71f633a, this is no longer necessary.

Refs: #22622
Fixes: #29793

PR-URL: #29795
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: David Carlier <devnexen@gmail.com>

@addaleax addaleax deleted the no-crypto-utf16-restrictions branch

October 3, 2019 21:03

BridgeAR pushed a commit that referenced this pull request

Oct 9, 2019 
Since 71f633a, this is no longer necessary.

Refs: #22622
Fixes: #29793

PR-URL: #29795
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: David Carlier <devnexen@gmail.com>