Ecosyste.ms: OpenCollective
An open API service for software projects hosted on Open Collective.
github.com/electron/node
Node fork to make it suitable for embedding in Electron
https://github.com/electron/node
Currently when building the following warning is emitted:
../test_symbol.c:4:19:
warning: unuse...
This is unused inside Node core, so nothing good can
come from keeping it around.
PR-URL: https...
44fdd36b96a55938d11112965be1bbaaec597695 authored over 6 years ago by Anna Henningsen <[email protected]>Remove a superfluos persistent from test_constructor_name.c.
PR-URL: https://github.com/nodejs/...
b5c1c146f5c5ba39435cf9488488803dd401607a authored over 6 years ago by Gabriel Schulhof <[email protected]>
The libuv 1.20.2 update in Node 10 aligned the Windows behavior
of os.uptime() with that of othe...
The process("node").mark("gc__stop") is actually
process("node").mark("gc__done"). Use the prope...
Foreground tasks that repost themselves can force the draining loop
to run indefinitely long wit...
PR-URL: https://github.com/nodejs/node/pull/20244
Reviewed-By: Daniel Bevenius <daniel.bevenius@...
PR-URL: https://github.com/nodejs/node/pull/20277
Refs: https://github.com/nodejs/node/pull/1912...
This commit adds a checkMethods function so that it can be reused to
avoid duplicating the same ...
PR-URL: https://github.com/nodejs/node/pull/20243
Reviewed-By: Colin Ihrig <[email protected]>
R...
PR-URL: https://github.com/nodejs/node/pull/20271
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmai...
Modernize:
* Replace `var` with `const` / `let`.
* Replace common functions with arrow functions...
The code currently assumes that `console` is already writable, but
that's only if it was previou...
PR-URL: https://github.com/nodejs/node/pull/20265
Reviewed-By: Gus Caplan <[email protected]>
Reviewed...
PR-URL: https://github.com/nodejs/node/pull/20257
Refs: https://github.com/nodejs/node/pull/1940...
Fixes: https://github.com/nodejs/node/issues/20194
PR-URL: https://github.com/nodejs/node/pull/...
c5b34590031472fac421b4b2095b94bd96a60430 authored over 6 years ago by Rich Trott <[email protected]>
This commit renames the verification result variable, that is currently
named i, to verify_resul...
This commit updates node_crypto.cc VerifySpkac function to use false
instead of 0 for its return...
Much like regular modules, N-API modules can also benefit from having
a special symbol which the...
4289402aa3a7b02554aa0ebcb70af40447083fc8 authored over 6 years ago by James M Snell <[email protected]>
* Assert
* Calling `assert.fail()` with more than one argument is deprecated. #70dcacd710
* ...
The current openssl checks assembler version only x86_64 or ia32
target arch for use of AES-NI, ...
PR-URL: https://github.com/nodejs/node/pull/20226
Fixes: https://github.com/nodejs/node/issues/1...
PR-URL: https://github.com/nodejs/node/pull/20226
Fixes: https://github.com/nodejs/node/issues/1...
Add parameters for the callback for the Http2Session:error event
inline with the pattern in the ...
PR-URL: https://github.com/nodejs/node/pull/20199
Reviewed-By: Luigi Pinca <[email protected]...
Specifically `readable` and `writable` that default to `false`.
PR-URL: https://github.com/node...
a342cd693cd8cb2c25dd5ff8a14e99f1043fa516 authored over 6 years ago by Santiago Gimeno <[email protected]>
PR-URL: https://github.com/nodejs/node/pull/20124
Fixes: https://github.com/nodejs/node/issues/2...
freeParser already unsets parser property of socket if socket is passed
in specifically. There's...
Cleanup constructor and freeParser to manage all existing parser
properties, not just some.
PR-...
cda94b2bb82315e498f8dbfcb33126c7c04ed92e authored over 6 years ago by Anatoli Papirovski <[email protected]>
A comment explaining kOnHeaders function of the parser was duplicated
in two places but the seco...
Remove constant deep property access by storing a reference,
use more const, make some condition...
PR-URL: https://github.com/nodejs/node/pull/20027
Reviewed-By: Gibson Fahnestock <gibfahn@gmail....
The output is now improved by showing most properties all at once.
Besides that this adds a warn...
We need to look for a special symbol even if the module self-registers
when the module self-regi...
- Reduce reference to the global `statValues` by returning
the changed stats array from the sy...
PR-URL: https://github.com/nodejs/node/pull/20092
Fixes: https://github.com/nodejs/node/issues/1...
PR-URL: https://github.com/nodejs/node/pull/20172
Reviewed-By: Anna Henningsen <[email protected]...
PR-URL: https://github.com/nodejs/node/pull/20197
Reviewed-By: Anna Henningsen <[email protected]...
Adds a test-case to cover loading modules
the user does not have permission to write to.
Covers...
2fd7284add63b3d7101be35c3de6da5e321f7d5b authored over 6 years ago by Bill Ticehurst <[email protected]>
PR-URL: https://github.com/nodejs/node/pull/20129
Fixes: https://github.com/nodejs/node/issues/2...
PR-URL: https://github.com/nodejs/node/pull/20074
Reviewed-By: Luigi Pinca <[email protected]>
PR-URL: https://github.com/nodejs/node/pull/20209
Reviewed-By: Trivikram Kamat <trivikr.dev@gmai...
PR-URL: https://github.com/nodejs/node/pull/20207
Fixes: https://github.com/nodejs/node/issues/1...
Terminating the execution of a script would create a pending
exception, therefore we should canc...
This commit removes the assignment of exports since it is not used
in these files and there is n...
PR-URL: https://github.com/nodejs/node/pull/20142
Reviewed-By: Luigi Pinca <[email protected]...
PR-URL: https://github.com/nodejs/node/pull/20125
Reviewed-By: Ruben Bridgewater <ruben@bridgewa...
This commit removes the binding const as it is only used in one place
which is in the following ...
The method has been deprecated in upstream V8, with messaging
indicating that it is the default ...
PR-URL: https://github.com/nodejs/node/pull/20125
Reviewed-By: Ruben Bridgewater <ruben@bridgewa...
PR-URL: https://github.com/nodejs/node/pull/20125
Reviewed-By: Ruben Bridgewater <ruben@bridgewa...
PR-URL: https://github.com/nodejs/node/pull/20125
Reviewed-By: Ruben Bridgewater <ruben@bridgewa...
PR-URL: https://github.com/nodejs/node/pull/20125
Reviewed-By: Ruben Bridgewater <ruben@bridgewa...
PR-URL: https://github.com/nodejs/node/pull/20125
Reviewed-By: Ruben Bridgewater <ruben@bridgewa...
PR-URL: https://github.com/nodejs/node/pull/20125
Reviewed-By: Ruben Bridgewater <ruben@bridgewa...
PR-URL: https://github.com/nodejs/node/pull/20125
Reviewed-By: Ruben Bridgewater <ruben@bridgewa...
Remove upgrade from the regular expression as it is no longer used
and switch to using `.test()`...
Using a switch improves clarity of the code but also performance for
all but a few edge cases wh...
Handrolling and checking char by char is no longer faster than just
using toLowerCase and strict...
Adds strictEqual method to assert on stream.session.alpnProtocol
to verify expected value 'h2'. ...
PR-URL: https://github.com/nodejs/node/pull/20200
Fixes: https://github.com/nodejs/node/issues/2...
Add parameters for the callback for the Http2Session:connect event
inline with the pattern in th...
The `node_modules` rule below is treated as a glob pattern,
so it includes the `tools/node_modul...
PR-URL: https://github.com/nodejs/node/pull/20187
Reviewed-By: Ben Noordhuis <[email protected]...
When the tests are run as root in Ubuntu, process.setgid() is called
with 'nobody' as an argumen...
When an AssertionError happens, the value of headers['set-cookie']
is not reported. That informa...
`console.trace()` function prints 'Trace: ', not 'Trace :'.
PR-URL: https://github.com/nodejs/n...
0d56982e565ab958e1824c93065d5b6ecc86f967 authored over 6 years ago by Mykola Bilochub <[email protected]>
test-child-process-send-returns-boolean was unreliable in CI and
locally.
* use 'SIGKILL' for m...
82c52b5841b953fb1a8a7f1a67039d75544d031a authored over 6 years ago by Rich Trott <[email protected]>Buffer.from / new Buffer accept Uint8Array
Fixes: https://github.com/nodejs/node/issues/14118
...
9214d64760a15636175a0051e9c76869be9667c6 authored over 6 years ago by SheetJS <[email protected]>Wrap `buffer.md` at 80 characters and enforce with linter.
PR-URL: https://github.com/nodejs/no...
9495d9477bb0a22ca84a92a60ab27a1bb89051c3 authored over 6 years ago by Rich Trott <[email protected]>This reverts commit 91e0f8db118b125e805a77ce31dc21af57ee7abf.
PR-URL: https://github.com/nodejs...
fccff2702e23c97e0823922de2652e8c16eb5b41 authored over 6 years ago by Rod Vagg <[email protected]>
Adds a section at the very end to document all flags and links
to it from every function that ta...
Due to npm using workers on Windows which inititate processes
for code within node_modules, the ...
On inspector side we are going to deprecate .json protocol
definition formation and replace it w...
Make sure the connection is not closed until the 3 requests and the
`timeout` event are received...
This PR adds _readableState.errorEmitted and add the tracking of it.
Fixes: https://github.com/...
22819aa998001bf6f35ebb8d14f4910353c7632b authored over 6 years ago by Matteo Collina <[email protected]>
Due to a return statement with new inside the function, extending
Console class does not work wh...
String literals from two `assert.strictEquals()` calls were
removed as these string literals did...
Specifically, fix the inconsistency where the documentation
suggests running "$ make test" inste...
Move v8::HandleScope call to Emit removing it from previous locations
where it was added to avoi...
* Use present tense consistently in parameters description.
* Wrap at 80 characters.
* "partial ...
This makes sure the returned value when calling `block` is actually
of type promise in case `ass...
This improves a slow code part in `util.inspect` by directly
retrieving the `Symbol.toStringTag`...
Compilers were updated for 10.X as per
discussion in: https://github.com/nodejs/build/issues/925...
PR-URL: https://github.com/nodejs/node/pull/20149
Reviewed-By: Ruben Bridgewater <ruben@bridgewa...
A recent set of changes removed _consuming tracking from server
incoming messages which ensures ...
This commit removes the duplicate require calls of the _tls_common and
_tls_wrap modules. The mo...
PR-URL: https://github.com/nodejs/node/pull/20121
Reviewed-By: Daniel Bevenius <daniel.bevenius@...
PR-URL: https://github.com/nodejs/node/pull/20121
Reviewed-By: Daniel Bevenius <daniel.bevenius@...
PR-URL: https://github.com/nodejs/node/pull/20121
Reviewed-By: Daniel Bevenius <daniel.bevenius@...
PR-URL: https://github.com/nodejs/node/pull/20121
Reviewed-By: Daniel Bevenius <daniel.bevenius@...
- Moves THROW_AND_RETURN_IF_NOT_BUFFER and
THROW_AND_RETURN_IF_NOT_STRING from node_crypto.cc ...
This patch migrates the "Index out of range" errors in C++
to errors with the code `ERR_INDEX_OU...
PR-URL: https://github.com/nodejs/node/pull/20121
Reviewed-By: Daniel Bevenius <daniel.bevenius@...
PR-URL: https://github.com/nodejs/node/pull/20073
Reviewed-By: Trivikram Kamat <trivikr.dev@gmai...
This commit updates canonicalizeIP and Buffer to use object
destructuring which is done for othe...
PR-URL: https://github.com/nodejs/node/pull/20132
Fixes: https://github.com/nodejs/TSC/issues/48...
PR-URL: https://github.com/nodejs/node/pull/20133
Fixes: https://github.com/nodejs/node/pull/196...
One entry is placed in the wrong module and other entries are not in
alphabetical order.
PR-URL...
c31f0d0ba2b4d4b221ff89d986a36c82ce54fa73 authored over 6 years ago by Rich Trott <[email protected]>