Commits · 867ff41d87e292997ff2ceb8d77e715145aed7b7 · github.com / v8-riscv / node

Apr 01, 2020

src: handle report options on fatalerror · 867ff41d

Sam Roberts authored Mar 25, 2020

Follow on to https://github.com/nodejs/node/pull/32207, 3 other options
are also not respected under situations that the isolate is not
available.

PR-URL: https://github.com/nodejs/node/pull/32497


Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>

867ff41d

doc: update context-aware section of addon doc · e158218f

Gabriel Schulhof authored Jul 12, 2019



Replace the portion of the context-aware addon example that advocates
for managing the life cycle of the per-addon-instance data using a
weak reference with code that illustrates how to manage its life cycle
using an environment cleanup hook.

Signed-off-by: Gabriel Schulhof <gabriel.schulhof@intel.com>
PR-URL: https://github.com/nodejs/node/pull/28659


Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>

e158218f

Mar 31, 2020

src: refactoring and cleanup of node_i18n · accc984c

James M Snell authored Mar 22, 2020



Signed-off-by: James M Snell <jasnell@gmail.com>

PR-URL: https://github.com/nodejs/node/pull/32438


Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>

accc984c

Mar 30, 2020

wasi: clean up options validation · fd7e5d40

Denys Otrishko authored Feb 14, 2020

PR-URL: https://github.com/nodejs/node/pull/31797


Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Anna Henningsen <anna@addaleax.net>

fd7e5d40

dns: add dns.ALL hints flag constant · 18c3ff99

murgatroid99 authored Mar 09, 2020

PR-URL: https://github.com/nodejs/node/pull/32183


Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>

18c3ff99

doc: update AUTHORS list · bbf3fb8a

Luigi Pinca authored Mar 12, 2020

PR-URL: https://github.com/nodejs/node/pull/32222


Reviewed-By: Anna Henningsen <anna@addaleax.net>

bbf3fb8a

doc: deprecate process.mainModule · f571b294

Antoine du HAMEL authored Mar 12, 2020

PR-URL: https://github.com/nodejs/node/pull/32232


Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Gus Caplan <me@gus.host>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>

f571b294

doc: stream.end(cb) cb can be invoked with error · 6ade42bb

Pranshu Srivastava authored Mar 13, 2020

Update docs that the optional callback passed to
`writable.end` can be invoked in case of an error as well.

PR-URL: https://github.com/nodejs/node/pull/32238
Fixes: https://github.com/nodejs/node/issues/31220


Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>

6ade42bb

crypto: clear openssl error stack after en/decrypt · c37d4cce

Ben Noordhuis authored Mar 13, 2020

The publicEncrypt/privateDecrypt/etc. family of functions didn't clear
OpenSSL's error stack on early return.

Notably, trying to use an encrypted key with the wrong passphrase left
an error on the stack that made subsequent encrypt or decrypt operations
fail, even with an unencrypted key.

Fixes: https://github.com/nodejs/node/issues/32240

PR-URL: https://github.com/nodejs/node/pull/32248


Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>

c37d4cce

src: remove unnecessary 'Local.As' operation · 2d881298

himself65 authored Mar 16, 2020

PR-URL: https://github.com/nodejs/node/pull/32286


Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Gus Caplan <me@gus.host>
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: James M Snell <jasnell@gmail.com>

2d881298

fs: fix fs.read when passing null value · defbc2ed

himself65 authored Mar 25, 2020

PR-URL: https://github.com/nodejs/node/pull/32479


Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>

defbc2ed

test: fix a typo on test-fs-read-optional-params · fb254d2a

himself65 authored Mar 25, 2020

PR-URL: https://github.com/nodejs/node/pull/32461


Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Rich Trott <rtrott@gmail.com>

fb254d2a

test: skip a wasi test on IBMi PASE · 4898c1d4

Xu Meng authored Mar 24, 2020

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>

4898c1d4

worker: runtime error on pthread creation · b6459ec4

Harshitha KP authored Mar 18, 2020

With large number of worker threads pthread
fails with hard assertion.
Instead of hard assertion throw a runtime error.

PR-URL: https://github.com/nodejs/node/pull/32344
Fixes: https://github.com/nodejs/node/issues/32319


Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>

b6459ec4

lib: removed unused error code · a5e81e72

Yash Ladha authored Mar 25, 2020

PR-URL: https://github.com/nodejs/node/pull/32481


Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>

a5e81e72

stream: change var to let/const in stream files · 4fc13b01

Saajan authored Mar 12, 2020

PR-URL: https://github.com/nodejs/node/pull/32214


Reviewed-By: Robert Nagy <ronagy@icloud.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>

4fc13b01

http: move free socket error handling to agent · 2bf02285

Robert Nagy authored Feb 28, 2020

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: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>

2bf02285

test: harden the tick sampling logic · bffc9324

Harshitha KP authored Mar 11, 2020

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: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>

bffc9324

win,build: set exit_code on configure failure · 5e05f8ac

Bartlomiej Brzozowski authored Mar 09, 2020

PR-URL: https://github.com/nodejs/node/pull/32205
Fixes: https://github.com/nodejs/node/issues/31573


Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

5e05f8ac

lib: replace Array to ArrayIsArray by primordials · 1bbd679a

himself65 authored Mar 14, 2020

PR-URL: https://github.com/nodejs/node/pull/32258


Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>

1bbd679a

http: don't emit 'readable' after 'close' · e034f5c3

Robert Nagy authored Mar 15, 2020

PR-URL: https://github.com/nodejs/node/pull/32277
Refs: https://github.com/nodejs/node/issues/28710


Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>

e034f5c3

doc: tests local links in markdown documents · 19db0e60

Antoine du HAMEL authored Mar 19, 2020

PR-URL: https://github.com/nodejs/node/pull/32359


Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Rich Trott <rtrott@gmail.com>

19db0e60

test: als variant of test-timers-clearImmediate · bac401c4

Harshitha KP authored Mar 16, 2020

Refs: https://github.com/nodejs/node/issues/31978

PR-URL: https://github.com/nodejs/node/pull/32303


Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>

bac401c4

test: skip some binding tests on IBMi PASE · b416993c

Xu Meng authored Feb 26, 2020

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: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>

b416993c

fs: add fs.readv() · 97ef4d76

Sk Sajidul Kadir authored Mar 17, 2020

Fixes: https://github.com/nodejs/node/issues/2298
PR-URL: https://github.com/nodejs/node/pull/32356


Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>

97ef4d76

util: fix inspecting document.all · 89ae1f1b

Gus Caplan authored Feb 24, 2020

Fixes: https://github.com/nodejs/node/issues/31889

PR-URL: https://github.com/nodejs/node/pull/31938


Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>

89ae1f1b

benchmark: use let instead of var in zlib · 75aaf749

Daniele Belardi authored Feb 14, 2020

PR-URL: https://github.com/nodejs/node/pull/31794


Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>

75aaf749

benchmark: use let instead of var in worker · 806899fd

Daniele Belardi authored Feb 14, 2020

PR-URL: https://github.com/nodejs/node/pull/31794


Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>

806899fd

benchmark: use let instead of var in util · d9a77c64

Daniele Belardi authored Feb 14, 2020

PR-URL: https://github.com/nodejs/node/pull/31794


Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>

d9a77c64

benchmark: use let instead of var in url · a33e9c9d

Daniele Belardi authored Feb 14, 2020

PR-URL: https://github.com/nodejs/node/pull/31794


Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>

a33e9c9d

benchmark: use let instead of var in tls · 8ec533d8

Daniele Belardi authored Feb 14, 2020

PR-URL: https://github.com/nodejs/node/pull/31794


Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>

8ec533d8

benchmark: use let instead of var in timers · de877c57

Daniele Belardi authored Feb 14, 2020

PR-URL: https://github.com/nodejs/node/pull/31794


Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>

de877c57

benchmark: use let instead of var in run.js · 13ad05ce

Daniele Belardi authored Feb 14, 2020

PR-URL: https://github.com/nodejs/node/pull/31794


Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>

13ad05ce

benchmark: use let instead of var in dns · a2e70157

Daniele Belardi authored Feb 14, 2020

PR-URL: https://github.com/nodejs/node/pull/31794


Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>

a2e70157

benchmark: use let instead of var in common.js · e18f5af2

Daniele Belardi authored Feb 14, 2020

PR-URL: https://github.com/nodejs/node/pull/31794


Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>

e18f5af2

benchmark: use const instead of var in async_hooks · 435fe977

Daniele Belardi authored Feb 14, 2020

PR-URL: https://github.com/nodejs/node/pull/31794


Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>

435fe977

benchmark: add `no-var` rule in .eslintrc.yaml · d8f44bc2

Daniele Belardi authored Feb 14, 2020

PR-URL: https://github.com/nodejs/node/pull/31794


Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>

d8f44bc2

src: add test/abort build tasks · e08ac095

Christian Niederer authored Feb 14, 2020

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: James M Snell <jasnell@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>

e08ac095

src: add aliased-buffer-overflow abort test · babeb58c

Christian Niederer authored Feb 12, 2020

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: James M Snell <jasnell@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>

babeb58c

src: check for overflow when extending AliasedBufferBase · 8245a781

Christian Niederer authored Feb 11, 2020

When resizing an aliased_buffer check if the new size will overflow.

PR-URL: https://github.com/nodejs/node/pull/31740


Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>

8245a781