Merge branch 'main' into actions/tools-update-undici

nodejs · Feb 13, 2023 · 1f7ae10 · 1f7ae10
2 parents 2b4bb00 + ecd385e
commit 1f7ae10
Show file tree

Hide file tree

Showing 999 changed files with 10,377 additions and 4,976 deletions.
diff --git a/.github/workflows/tools.yml b/.github/workflows/tools.yml
@@ -78,7 +78,7 @@ jobs:
                 ./tools/update-undici.sh
               fi
           - id: postject
-            subsystem: test
+            subsystem: deps,test
             label: test
             run: |
               NEW_VERSION=$(npm view postject dist-tags.latest)
@@ -144,6 +144,16 @@ jobs:
                 echo "NEW_VERSION=$NEW_VERSION" >> $GITHUB_ENV
                 ./tools/dep_updaters/update-simdutf.sh "$NEW_VERSION"
               fi
+          - id: ada
+            subsystem: deps
+            label: dependencies
+            run: |
+              NEW_VERSION=$(gh api repos/ada-url/ada/releases/latest -q '.tag_name|ltrimstr("v")')
+              CURRENT_VERSION=$(grep "#define ADA_VERSION" ./deps/ada/ada.h | sed -n "s/^.*VERSION \(.*\)/\1/p")
+              if [ "$NEW_VERSION" != "$CURRENT_VERSION" ]; then
+                echo "NEW_VERSION=$NEW_VERSION" >> $GITHUB_ENV
+                ./tools/dep_updaters/update-ada.sh "$NEW_VERSION"
+              fi
     steps:
       - uses: actions/checkout@v3
         with:

diff --git a/.mailmap b/.mailmap
@@ -521,6 +521,7 @@ Tierney Cyren <[email protected]> <[email protected]>
 Tierney Cyren <[email protected]> <[email protected]>
 Tim Caswell <[email protected]> <[email protected]>
 Tim Costa <[email protected]> <[email protected]>
+Tim Perry <[email protected]> <[email protected]>
 Tim Price <[email protected]>
 Tim Ruffles <[email protected]> <[email protected]>
 Tim Smart <[email protected]>

diff --git a/AUTHORS b/AUTHORS
@@ -3605,5 +3605,8 @@ Suyash Nayan <[email protected]>
 Alec Mev <[email protected]>
 Miguel Teixeira <[email protected]>
 José Dapena Paz <[email protected]>
+Mohamed Akram <[email protected]>
+三咲智子 Kevin Deng <[email protected]>
+Nicolas DUBIEN <[email protected]>
 
 # Generated by tools/update-authors.mjs
diff --git a/BUILDING.md b/BUILDING.md
@@ -342,12 +342,17 @@ the `--help` option:
 $ tools/test.py --help
 ```
 
+> Note: On Windows you should use `python3` executable.
+> Example: `python3 tools/test.py test/message`
+
 You can usually run tests directly with node:
 
 ```text
-$ ./node ./test/parallel/test-stream2-transform.js
+$ ./node test/parallel/test-stream2-transform.js
 ```
 
+> Info: `./node` points to your local Node.js build.
+
 Remember to recompile with `make -j4` in between test runs if you change code in
 the `lib` or `src` directories.
 

diff --git a/LICENSE b/LICENSE
@@ -1338,6 +1338,28 @@ The externally maintained libraries used by Node.js are:
     CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
   """
 
+- ada, located at deps/ada, is licensed as follows:
+  """
+    Copyright 2023 Ada authors
+
+    Permission is hereby granted, free of charge, to any person obtaining a copy of
+    this software and associated documentation files (the "Software"), to deal in
+    the Software without restriction, including without limitation the rights to
+    use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of
+    the Software, and to permit persons to whom the Software is furnished to do so,
+    subject to the following conditions:
+
+    The above copyright notice and this permission notice shall be included in all
+    copies or substantial portions of the Software.
+
+    THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+    IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS
+    FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR
+    COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER
+    IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
+    CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
+  """
+
 - npm, located at deps/npm, is licensed as follows:
   """
     The npm application

diff --git a/Makefile b/Makefile
@@ -170,7 +170,7 @@ with-code-cache test-code-cache:
 
 out/Makefile: config.gypi common.gypi node.gyp \
 	deps/uv/uv.gyp deps/llhttp/llhttp.gyp deps/zlib/zlib.gyp \
-	deps/simdutf/simdutf.gyp \
+	deps/simdutf/simdutf.gyp deps/ada/ada.gyp \
 	tools/v8_gypfiles/toolchain.gypi tools/v8_gypfiles/features.gypi \
 	tools/v8_gypfiles/inspector.gypi tools/v8_gypfiles/v8.gyp
 	$(PYTHON) tools/gyp_node.py -f make

diff --git a/README.md b/README.md
@@ -432,8 +432,6 @@ For information about the governance of the Node.js project, see
   **Paolo Insogna** <<[email protected]>> (he/him)
 * [srl295](https://github.com/srl295) -
   **Steven R Loomis** <<[email protected]>>
-* [starkwang](https://github.com/starkwang) -
-  **Weijia Wang** <<[email protected]>>
 * [sxa](https://github.com/sxa) -
   **Stewart X Addison** <<[email protected]>> (he/him)
 * [targos](https://github.com/targos) -
@@ -638,6 +636,8 @@ For information about the governance of the Node.js project, see
   **Shigeki Ohtsu** <<[email protected]>> (he/him)
 * [silverwind](https://github.com/silverwind) -
   **Roman Reiss** <<[email protected]>>
+* [starkwang](https://github.com/starkwang) -
+  **Weijia Wang** <<[email protected]>>
 * [stefanmb](https://github.com/stefanmb) -
   **Stefan Budeanu** <<[email protected]>>
 * [tellnes](https://github.com/tellnes) -

diff --git a/benchmark/.eslintrc.yaml b/benchmark/.eslintrc.yaml
@@ -9,12 +9,7 @@ rules:
 
 overrides:
   - files:
-      - crypto/*.js
-      - http/*.js
-      - http2/*.js
       - path/*.js
-      - process/*.js
-      - url/*.js
     rules:
       comma-dangle: [error, {
         arrays: always-multiline,

diff --git a/benchmark/crypto/aes-gcm-throughput.js b/benchmark/crypto/aes-gcm-throughput.js
@@ -5,7 +5,7 @@ const keylen = { 'aes-128-gcm': 16, 'aes-192-gcm': 24, 'aes-256-gcm': 32 };
 const bench = common.createBenchmark(main, {
   n: [500],
   cipher: ['aes-128-gcm', 'aes-192-gcm', 'aes-256-gcm'],
-  len: [1024, 4 * 1024, 16 * 1024, 64 * 1024, 256 * 1024, 1024 * 1024]
+  len: [1024, 4 * 1024, 16 * 1024, 64 * 1024, 256 * 1024, 1024 * 1024],
 });
 
 function main({ n, len, cipher }) {

diff --git a/benchmark/crypto/cipher-stream.js b/benchmark/crypto/cipher-stream.js
@@ -6,9 +6,9 @@ const bench = common.createBenchmark(main, {
   cipher: ['AES192', 'AES256'],
   type: ['asc', 'utf', 'buf'],
   len: [2, 1024, 102400, 1024 * 1024],
-  api: ['legacy', 'stream']
+  api: ['legacy', 'stream'],
 }, {
-  flags: ['--no-warnings']
+  flags: ['--no-warnings'],
 });
 
 function main({ api, cipher, type, len, writes }) {

diff --git a/benchmark/crypto/get-ciphers.js b/benchmark/crypto/get-ciphers.js
@@ -4,7 +4,7 @@ const common = require('../common.js');
 
 const bench = common.createBenchmark(main, {
   n: [1, 5000],
-  v: ['crypto', 'tls']
+  v: ['crypto', 'tls'],
 });
 
 function main({ n, v }) {

diff --git a/benchmark/crypto/hash-stream-creation.js b/benchmark/crypto/hash-stream-creation.js
@@ -10,7 +10,7 @@ const bench = common.createBenchmark(main, {
   type: ['asc', 'utf', 'buf'],
   out: ['hex', 'binary', 'buffer'],
   len: [2, 1024, 102400, 1024 * 1024],
-  api: ['legacy', 'stream']
+  api: ['legacy', 'stream'],
 });
 
 function main({ api, type, len, out, writes, algo }) {

diff --git a/benchmark/crypto/hash-stream-throughput.js b/benchmark/crypto/hash-stream-throughput.js
@@ -9,7 +9,7 @@ const bench = common.createBenchmark(main, {
   algo: ['sha1', 'sha256', 'sha512'],
   type: ['asc', 'utf', 'buf'],
   len: [2, 1024, 102400, 1024 * 1024],
-  api: ['legacy', 'stream']
+  api: ['legacy', 'stream'],
 });
 
 function main({ api, type, len, algo, writes }) {

diff --git a/benchmark/crypto/hkdf.js b/benchmark/crypto/hkdf.js
@@ -4,7 +4,7 @@ const common = require('../common.js');
 const assert = require('assert');
 const {
   hkdf,
-  hkdfSync
+  hkdfSync,
 } = require('crypto');
 
 const bench = common.createBenchmark(main, {

diff --git a/benchmark/crypto/keygen.js b/benchmark/crypto/keygen.js
@@ -4,7 +4,7 @@ const common = require('../common.js');
 const assert = require('assert');
 const {
   generateKeyPair,
-  generateKeyPairSync
+  generateKeyPairSync,
 } = require('crypto');
 
 const bench = common.createBenchmark(main, {
@@ -18,7 +18,7 @@ const methods = {
     for (let i = 0; i < n; ++i) {
       generateKeyPairSync('rsa', {
         modulusLength: 1024,
-        publicExponent: 0x10001
+        publicExponent: 0x10001,
       });
     }
     bench.end(n);
@@ -35,7 +35,7 @@ const methods = {
     for (let i = 0; i < n; ++i)
       generateKeyPair('rsa', {
         modulusLength: 512,
-        publicExponent: 0x10001
+        publicExponent: 0x10001,
       }, done);
   },
 

diff --git a/benchmark/crypto/oneshot-sign-verify.js b/benchmark/crypto/oneshot-sign-verify.js
@@ -9,7 +9,7 @@ const keyFixtures = {
   publicKey: fs.readFileSync(`${fixtures_keydir}/ec_p256_public.pem`)
     .toString(),
   privateKey: fs.readFileSync(`${fixtures_keydir}/ec_p256_private.pem`)
-    .toString()
+    .toString(),
 };
 
 const data = crypto.randomBytes(256);
@@ -20,7 +20,7 @@ let keyObjects;
 function getKeyObject({ privateKey, publicKey }) {
   return {
     privateKey: crypto.createPrivateKey(privateKey),
-    publicKey: crypto.createPublicKey(publicKey)
+    publicKey: crypto.createPublicKey(publicKey),
   };
 }
 
@@ -60,15 +60,15 @@ function measureAsyncSerial(n, privateKey, publicKey, keys) {
       data,
       {
         key: privateKey || keys[n - remaining].privateKey,
-        dsaEncoding: 'ieee-p1363'
+        dsaEncoding: 'ieee-p1363',
       },
       (err, signature) => {
         crypto.verify(
           'sha256',
           data,
           {
             key: publicKey || keys[n - remaining].publicKey,
-            dsaEncoding: 'ieee-p1363'
+            dsaEncoding: 'ieee-p1363',
           },
           signature,
           done);
@@ -104,7 +104,7 @@ function measureAsyncParallel(n, privateKey, publicKey, keys) {
 function main({ n, mode, keyFormat }) {
   pems ||= [...Buffer.alloc(n)].map(() => ({
     privateKey: keyFixtures.privateKey,
-    publicKey: keyFixtures.publicKey
+    publicKey: keyFixtures.publicKey,
   }));
   keyObjects ||= pems.map(getKeyObject);
 

diff --git a/benchmark/crypto/randomInt.js b/benchmark/crypto/randomInt.js
@@ -7,7 +7,7 @@ const bench = common.createBenchmark(main, {
   mode: ['sync', 'async-sequential', 'async-parallel'],
   min: [-(2 ** 47) + 1, -10_000, -100],
   max: [100, 10_000, 2 ** 47],
-  n: [1e3, 1e5]
+  n: [1e3, 1e5],
 });
 
 function main({ mode, min, max, n }) {

diff --git a/benchmark/crypto/rsa-encrypt-decrypt-throughput.js b/benchmark/crypto/rsa-encrypt-decrypt-throughput.js
@@ -19,7 +19,7 @@ keylen_list.forEach((key) => {
 const bench = common.createBenchmark(main, {
   n: [500],
   keylen: keylen_list,
-  len: [16, 32, 64]
+  len: [16, 32, 64],
 });
 
 function main({ len, algo, keylen, n }) {

diff --git a/benchmark/crypto/rsa-sign-verify-throughput.js b/benchmark/crypto/rsa-sign-verify-throughput.js
@@ -20,7 +20,7 @@ const bench = common.createBenchmark(main, {
   writes: [500],
   algo: ['SHA1', 'SHA224', 'SHA256', 'SHA384', 'SHA512'],
   keylen: keylen_list,
-  len: [1024, 102400, 2 * 102400, 3 * 102400, 1024 * 1024]
+  len: [1024, 102400, 2 * 102400, 3 * 102400, 1024 * 1024],
 });
 
 function main({ len, algo, keylen, writes }) {

diff --git a/benchmark/crypto/webcrypto-digest.js b/benchmark/crypto/webcrypto-digest.js
@@ -15,7 +15,7 @@ const kMethods = {
   'SHA-1': 'sha1',
   'SHA-256': 'sha256',
   'SHA-384': 'sha384',
-  'SHA-512': 'sha512'
+  'SHA-512': 'sha512',
 };
 
 // This benchmark only looks at clock time and ignores factors

diff --git a/benchmark/http/bench-parser.js b/benchmark/http/bench-parser.js
@@ -4,9 +4,9 @@ const common = require('../common');
 
 const bench = common.createBenchmark(main, {
   len: [4, 8, 16, 32],
-  n: [1e5]
+  n: [1e5],
 }, {
-  flags: ['--expose-internals', '--no-warnings']
+  flags: ['--expose-internals', '--no-warnings'],
 });
 
 function main({ len, n }) {

diff --git a/benchmark/http/check_invalid_header_char.js b/benchmark/http/check_invalid_header_char.js
@@ -23,7 +23,7 @@ const groupedInputs = {
 
   // Put it here so the benchmark result lines will not be super long.
   LONG_AND_INVALID: ['Here is a value that is really a folded header ' +
-    'value\r\n  this should be supported, but it is not currently']
+    'value\r\n  this should be supported, but it is not currently'],
 };
 
 const inputs = [

diff --git a/benchmark/http/chunked.js b/benchmark/http/chunked.js
@@ -14,7 +14,7 @@ const bench = common.createBenchmark(main, {
   n: [1, 4, 8, 16],
   len: [1, 64, 256],
   c: [100],
-  duration: 5
+  duration: 5,
 });
 
 function main({ len, n, c, duration }) {

diff --git a/benchmark/http/client-request-body.js b/benchmark/http/client-request-body.js
@@ -8,7 +8,7 @@ const bench = common.createBenchmark(main, {
   dur: [5],
   type: ['asc', 'utf', 'buf'],
   len: [32, 256, 1024],
-  method: ['write', 'end']
+  method: ['write', 'end'],
 });
 
 function main({ dur, len, type, method }) {
@@ -33,7 +33,7 @@ function main({ dur, len, type, method }) {
     agent: new http.Agent({ maxSockets: 1 }),
     host: '127.0.0.1',
     path: '/',
-    method: 'POST'
+    method: 'POST',
   };
 
   const server = http.createServer((req, res) => {

diff --git a/benchmark/http/cluster.js b/benchmark/http/cluster.js
@@ -34,7 +34,7 @@ function main({ type, len, c, duration }) {
       bench.http({
         path: path,
         connections: c,
-        duration
+        duration,
       }, () => {
         w1.destroy();
         w2.destroy();

diff --git a/benchmark/http/create-clientrequest.js b/benchmark/http/create-clientrequest.js
@@ -11,7 +11,7 @@ const bench = common.createBenchmark(main, {
   // Use 'url' to avoid name clash with other http benchmark
   url: types.concat(['wpt']),
   arg: ['URL', 'string', 'options'],
-  e: [1]
+  e: [1],
 });
 
 function noop() {}
@@ -25,7 +25,7 @@ function main({ url: type, arg, e }) {
   switch (arg) {
     case 'options': {
       const options = data.map((i) => ({
-        path: new URL(i).path, createConnection: noop
+        path: new URL(i).path, createConnection: noop,
       }));
       bench.start();
       for (let i = 0; i < len; i++) {

diff --git a/benchmark/http/end-vs-write-end.js b/benchmark/http/end-vs-write-end.js
@@ -15,7 +15,7 @@ const bench = common.createBenchmark(main, {
   len: [64 * 1024, 128 * 1024, 256 * 1024, 1024 * 1024],
   c: [100],
   method: ['write', 'end'],
-  duration: 5
+  duration: 5,
 });
 
 function main({ len, type, method, c, duration }) {

diff --git a/benchmark/http/headers.js b/benchmark/http/headers.js
@@ -6,7 +6,7 @@ const http = require('http');
 const bench = common.createBenchmark(main, {
   n: [10, 600],
   len: [1, 100],
-  duration: 5
+  duration: 5,
 });
 
 function main({ len, n, duration }) {