- Apr 23, 2020
-
-
Xu Meng authored
The API clock_getres with attribute API CLOCK_THREAD_CPUTIME_ID is not supported on IBM i PASE. PR-URL: https://github.com/nodejs/node/pull/32459 Reviewed-By:
Richard Lau <riclau@uk.ibm.com> Reviewed-By:
Colin Ihrig <cjihrig@gmail.com> Reviewed-By:
Anna Henningsen <anna@addaleax.net>
-
Robert Nagy authored
The http client should not know anything about free sockets. Let the agent handle its pool of sockets. PR-URL: https://github.com/nodejs/node/pull/32003 Reviewed-By:
Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By:
Matteo Collina <matteo.collina@gmail.com>
-
Harshitha KP authored
Under peculiar system load conditions, the profiler thread does not get enough CPU slices to perform the sampling. Improve the interaction between worker and parent thread by performing a large disc read, which is a better blend of CPU and I/O bound work, than earlier versions. This produces x10 more samples than the existing one, in 10 iterations, as opposed to 1024. Also capture worker error situations to improve debugging Refs: https://github.com/nodejs/node/issues/26401#issuecomment-597438516 PR-URL: https://github.com/nodejs/node/pull/32190 Reviewed-By:
James M Snell <jasnell@gmail.com>
-
Bartlomiej Brzozowski authored
PR-URL: https://github.com/nodejs/node/pull/32205 Fixes: https://github.com/nodejs/node/issues/31573 Reviewed-By:
-
Robert Nagy authored
PR-URL: https://github.com/nodejs/node/pull/32277 Refs: https://github.com/nodejs/node/issues/28710 Reviewed-By:
-
Antoine du HAMEL authored
PR-URL: https://github.com/nodejs/node/pull/32359 Reviewed-By:
Rich Trott <rtrott@gmail.com>
-
Xu Meng authored
IBMi PASE Node.js always links to shared openssl and zlib libraries. So skip the static binding tests. PR-URL: https://github.com/nodejs/node/pull/31967 Reviewed-By:
Michael Dawson <michael_dawson@ca.ibm.com>
-
Gus Caplan authored
Fixes: https://github.com/nodejs/node/issues/31889 PR-URL: https://github.com/nodejs/node/pull/31938 Reviewed-By:
-
Christian Niederer authored
The Aliased*Array overflow check test introduced a dependency to a compiled artifact. Add this artifact to the build process in a similar fashion as test/addons and test/js-native-api do. PR-URL: https://github.com/nodejs/node/pull/31740 Reviewed-By:
David Carlier <devnexen@gmail.com> Reviewed-By:
-
Christian Niederer authored
Added native extension similar to test/addons/stringbytes-external-exceeded-max. Added an abort test to regression test the non overflow behaviour. PR-URL: https://github.com/nodejs/node/pull/31740 Reviewed-By:
-
Christian Niederer authored
When resizing an aliased_buffer check if the new size will overflow. PR-URL: https://github.com/nodejs/node/pull/31740 Reviewed-By:
-
Rich Trott authored
Isolate the two test cases here a bit more and refactor code. PR-URL: https://github.com/nodejs/node/pull/32526 Reviewed-By:
-
Harshitha KP authored
this was influenced by https://github.com/nodejs/node/pull/32269 PR-URL: https://github.com/nodejs/node/pull/32298 Reviewed-By:
-
Syohei YOSHIDA authored
V8 CPU profile -> V8 heap profile PR-URL: https://github.com/nodejs/node/pull/32404 Reviewed-By:
Michaël Zasso <targos@protonmail.com> Reviewed-By:
-
himself65 authored
PR-URL: https://github.com/nodejs/node/pull/32294 Reviewed-By:
Bartosz Sosnowski <bartosz@janeasystems.com> Reviewed-By:
Luigi Pinca <luigipinca@gmail.com> Reviewed-By:
Trivikram Kamat <trivikr.dev@gmail.com> Reviewed-By:
-
Ruben Bridgewater authored
Also clean up the plugins package.json files. PR-URL: https://github.com/nodejs/node/pull/32310 Reviewed-By:
Jiawen Geng <technicalcute@gmail.com> Reviewed-By:
-
himself65 authored
PR-URL: https://github.com/nodejs/node/pull/32322 Reviewed-By:
Gus Caplan <me@gus.host>
-
Filip Skokan authored
PR-URL: https://github.com/nodejs/node/pull/32340 Reviewed-By:
Tobias Nießen <tniessen@tnie.de>
-
himself65 authored
PR-URL: https://github.com/nodejs/node/pull/32453 Reviewed-By:
Beth Griggs <Bethany.Griggs@uk.ibm.com> Reviewed-By:
Yongsheng Zhang <zyszys98@gmail.com> Reviewed-By:
Anto Aravinth <anto.aravinth.cse@gmail.com>
-
Richard Lau authored
For notarization on macOS all packaged binaries must be signed. Add a regression test to check that known binaries from our dependencies (at the time of this commit term-size via npm) are signed. Signed-off-by:
-
Bartosz Sosnowski authored
On Windows 2016 under high load further change events can be emitted after writing the 5 bytes is reported. Updating the mtime of the file can be reported as a separate change. This will increase the "before" count, but not the "w1HookCount" since we removed the listener. This makes the test keep the listeners until the end of the test. Fixes: https://github.com/nodejs/node/issues/21425 PR-URL: https://github.com/nodejs/node/pull/32484 Reviewed-By:
-
Ling Samuel authored
PR-URL: https://github.com/nodejs/node/pull/32456 Reviewed-By:
Sam Roberts <vieuxtech@gmail.com> Reviewed-By:
-
Ben Noordhuis authored
Disable the following compiler warning: warning: visibility attribute not supported in this configuration; ignored [-Wattributes] This is gcc complaining about `__attribute((visibility("default"))` in static library builds. Legitimate but harmless (and uninteresting) and it drowns out more relevant warnings. PR-URL: https://github.com/nodejs/node/pull/32419 Reviewed-By: -
James M Snell authored
Use "options.inspectOptions" instead of just "inspectOptions" Signed-off-by:
-
James M Snell authored
Use "options.recursive" instead of just "recursive" Signed-off-by:
-
himself65 authored
PR-URL: https://github.com/nodejs/node/pull/32191 Reviewed-By:
-
Rich Trott authored
Use Promise.all() instead of countdown in test-cluster-net-listen-ipv6only-false. Signed-off-by:
-
Rich Trott authored
Signed-off-by:
-
Rich Trott authored
* Remove assert.strictEqual where assert.ok suffices * Replace countdown with Promise.all() PR-URL: https://github.com/nodejs/node/pull/32493 Reviewed-By:
-
Alba Mendez authored
PR-URL: https://github.com/nodejs/node/pull/32525 Reviewed-By:
-
Juan José Arboleda authored
PR-URL: https://github.com/nodejs/node/pull/32374 Reviewed-By:
-
Xavier Stouder authored
Using ToLocalChecked on MaybeLocal without verifying it's empty can lead to unattempted crash. PR-URL: https://github.com/nodejs/node/pull/32339 Reviewed-By:
-
Anna Henningsen authored
Otherwise, this leaks memory if the weak callback is never called. PR-URL: https://github.com/nodejs/node/pull/32421 Reviewed-By:
-
James M Snell authored
Use `options.method` instead of just `method` Signed-off-by:
Evan Lucas <evanlucas@me.com> Reviewed-By:
-
James M Snell authored
Use "cmd.action" instead of just "action" for ERR_INVALID_ARG_TYPE Signed-off-by:
-
unknown authored
PR-URL: https://github.com/nodejs/node/pull/32228 Reviewed-By:
-
Rich Trott authored
Since the previous sentence describes `10` as a default, and the following sentence explains how to modify that default, it is unnecessary to explain that "Obviously, not all events should be limited to just 10 listeners." PR-URL: https://github.com/nodejs/node/pull/32457 Reviewed-By:
Myles Borins <myles.borins@gmail.com>
-
Rich Trott authored
PR-URL: https://github.com/nodejs/node/pull/32457 Reviewed-By:
-
Rich Trott authored
Instead of indicating that examples show lines to change in previous examples, present the examples as standalone items. They suffice on their own. In the first of these, it says to change "the second line" of a previous example, but if it were literally changed to the provided line, it would result in a syntax error. In the second of these, it gives the wrong line to change. All of this is unnecessary and probably makes the examples harder to follow. So simplify and treat each one as a separate example. PR-URL: https://github.com/nodejs/node/pull/32451 Reviewed-By:
-
himself65 authored
PR-URL: https://github.com/nodejs/node/pull/32385 Reviewed-By:
Gabriel Schulhof <gabriel.schulhof@intel.com>
-
