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
Fixes: https://github.com/nodejs/node/issues/13314
Refs: https://github.com/nodejs/node/issues/6...
This test currently fails. It illustrates that Unicode in the URL does
not arrive intact to the ...
PR-URL: https://github.com/nodejs/node/pull/13301
Reviewed-By: Michael Dawson <michael_dawson@ca...
PR-URL: https://github.com/nodejs/node/pull/13299
Reviewed-By: Michael Dawson <michael_dawson@ca...
PR-URL: https://github.com/nodejs/node/pull/13261
Reviewed-By: Roman Reiss <[email protected]>
Re...
It appears that either c-ares no longer calls callbacks synchronously
or we have since explicitl...
0.0.0.0 is more common than other special ipv4 addresses, so
it is possible that we may not get ...
PR-URL: https://github.com/nodejs/node/pull/13026
Ref: https://github.com/w3c/web-platform-tests...
PR-URL: https://github.com/nodejs/node/pull/13206
Reviewed-By: Refael Ackermann <[email protected]...
PR-URL: https://github.com/nodejs/node/pull/13315
Reviewed-By: Jeremiah Senkpiel <fishrock123@ro...
Once the Readline interface is closed, the 'data' event listener should
be removed.
PR-URL: htt...
7d53abad8b0913362679bfc88f337c8ec0216a44 authored over 7 years ago by Gibson Fahnestock <[email protected]>
gpg 2.1 no longer includes the key-id by default which breaks
the release script. This makes sur...
Pure refactor, makes no functional changes but the renaming helped me
see more clearly what the ...
The test is flaky under load. These changes greatly improve reliability.
* Use a recurring inte...
ce5745bf92f586c58366e9f738441d69118f2c18 authored over 7 years ago by Rich Trott <[email protected]>
PR-URL: https://github.com/nodejs/node/pull/13313
Reviewed-By: Colin Ihrig <[email protected]>
R...
Ref: https://github.com/nodejs/node/issues/13343
45d78393a9956ec813fc4a522e0f157093175456 authored over 7 years ago by Refael Ackermann <[email protected]>
PR-URL: https://github.com/nodejs/node/pull/13316
Reviewed-By: Anna Henningsen <[email protected]...
PR-URL: https://github.com/nodejs/node/pull/13379
Reviewed-By: Anna Henningsen <[email protected]...
Noticed this while reading through writing-tests.md today.
As per style guide avoid the use of y...
It takes time to build each of the addons used to test n-api.
Consolidate a few of the smaller o...
Fixes #13356
PR-URL: https://github.com/nodejs/node/pull/13360
Reviewed-By: Rich Trott <rtrott@...
Author: Thorsten Lorenz <[email protected]>
Author: Andreas Madsen <[email protected]>
PR-URL: htt...
Previously, napi_wrap() would only work with objects created from a
constructor returned by napi...
PR-URL: https://github.com/nodejs/node/pull/13371
Reviewed-By: Refael Ackermann <[email protected]...
* Remove useless constructor.
* Use template literals.
* Update code example.
Now all arrays...
* Use common.mustNotCall() and common.mustCall() as appropriate
* Use block scoping
* Move asser...
* Update outputs.
* Refine spaces.
* Restore missing part.
PR-URL: https://github.com/nodejs/no...
f1b2e68d651e4fb68ad3b27e1138542673d3b502 authored over 7 years ago by Vse Mozhet Byt <[email protected]>This reverts commit 9836cf571708a82396218957cacb3ed1ed468d05.
Ref: https://github.com/npm/npm/i...
ae6c7044c8d48cc8e7c267efca7429d5aa4df993 authored over 7 years ago by Anna Henningsen <[email protected]>Added bytesRead property to Zlib engines
Fixes: https://github.com/nodejs/node/issues/8874
PR-U...
Added option to expose engine to convenience methods
Refs: https://github.com/nodejs/node/issue...
d0b1b52edba8a4815086a1fa5e978ceaa91f6d3a authored over 7 years ago by Alexander O'Mara <[email protected]>
Use `common.mustCall()` to make sure noop function is called as
expected.
PR-URL: https://githu...
36867786d095de89a37ced4378d07c8efe30cdfa authored over 7 years ago by Rich Trott <[email protected]>
PR-URL: https://github.com/nodejs/node/pull/13274
Reviewed-By: Luigi Pinca <[email protected]...
Functions that call `ECDH::BufferToPoint` were not clearing the
error stack on failure, so an in...
Re-enable test-http-abort-stream-end and put it into parallel
category. Use system random port w...
PR-URL: https://github.com/nodejs/node/pull/13323
Reviewed-By: Luigi Pinca <[email protected]>
On macOS, a watcher created with fs.watch() does not necessarily
start receiving events immediat...
Use `common.mustNotCall()` in test-stream2-objects.js to confirm that
noop function is never inv...
One of the N-API weak-reference test cases already had to be made
asynchronous to handle differe...
* **Async Hooks**
* The `async_hooks` module has landed in core
[[`4a7233c178`](https://gi...
Add testing for all types of typed arrays.
Add testing for napi_is_arraybuffer.
PR-URL: https:/...
effeff1843db3e7e47cf0242cd25273e083b2180 authored over 7 years ago by Michael Dawson <[email protected]>
BUILDING.md
+ L122: Missing code-language flag
+ L170: Strong should use `*` as a marker
do...
1d60fc3aa5b7a6032665595c29bdb382bd526e43 authored over 7 years ago by Daijiro Wachi <[email protected]>
This floats https://github.com/npm/npm/pull/16791 onto npm v5.0.0
to fix an edge case that was f...
Allow binding to a randomly assigned port number with `--inspect=0`
or `--inspect-brk=0`.
PR-UR...
399cb25f6266a17f0e5ed838b37b9c8299a87224 authored over 7 years ago by Ben Noordhuis <[email protected]>
When V8 is built from its master branch, it adds a " (candidate)"
suffix to the version string. ...
Currently this test will fail with the following error message when
configured --without-ssl:
Er...
PR-URL: https://github.com/nodejs/node/pull/13276
Reviewed-By: James M Snell <[email protected]>...
PR-URL: https://github.com/nodejs/node/pull/13220
Reviewed-By: Refael Ackermann <[email protected]...
Update the list of root certificates in src/node_root_certs.h with
tools/mk-ca-bundle.pl.
Certi...
58af75e5fbaafdbd2e14d9c7aa18c6ba08ba29d8 authored over 7 years ago by Ben Noordhuis <[email protected]>
This is the certdata.txt[0] that ships in NSS 3.30.2, released on
2017-04-20.
[0] https://hg.mo...
d3028270ec5ea4245629b0e5dca34b555169f1d2 authored over 7 years ago by Ben Noordhuis <[email protected]>
PR-URL: https://github.com/nodejs/node/pull/13232
Ref: https://github.com/nodejs/CTC/issues/92
R...
Remove the Gitter badge.
Gitter is not supported by us. We use IRC channels on Freenode. Having...
19fdb54ffa656056a2694779e098db86b3927101 authored over 7 years ago by Rich Trott <[email protected]>
PR-URL: https://github.com/nodejs/node/pull/12949
Fixes: https://github.com/nodejs/node/issues/1...
PR-URL: https://github.com/nodejs/node/pull/12710
Reviewed-By: Joyee Cheung <[email protected]...
PR-URL: https://github.com/nodejs/node/pull/12710
Reviewed-By: Joyee Cheung <[email protected]...
When upgrading OpenSSL, Step 6 in upgrading guide explains the steps
that need to be taken if as...
VerifyCallback returns 1 in two locations but CHECK_CERT_REVOKED in a
third return statment. Thi...
Check that callbacks are not executed when errors are expected to be
thrown.
PR-URL: https://gi...
f00475d9d69da7e81411b726aff4a15b3b350954 authored over 7 years ago by Rich Trott <[email protected]>
Synchronize the argument list for `dns.resolve()` with what's in the
documentation.
Improve the...
758a17f1d5f5985f6d5e3823e0c4759e78252c52 authored over 7 years ago by Rich Trott <[email protected]>
doc/api/fs.md
+ L314: Missing code-language flag
doc/api/stream.md
+ L2120: Do not use defi...
Assign a `PromiseWrap` instance to Promises that do not have one
yet when the PromiseHook is bei...
Promises do not have any internal fields by default. V8 recently added
the capability of configu...
In the api docs there were some instances of behaviour
and many more with behavior. I was asked...
PR-URL: https://github.com/nodejs/node/pull/11296
Reviewed-By: James M Snell <[email protected]>...
PR-URL: https://github.com/nodejs/node/pull/10626
Reviewed-By: Ben Noordhuis <[email protected]...
PR-URL: https://github.com/nodejs/node/pull/13215
Refs: https://github.com/nodejs/node/issues/12...
PR-URL: https://github.com/nodejs/node/pull/13208
Reviewed-By: Gibson Fahnestock <gibfahn@gmail....
Use the same dtrace command on ARM as on i386. Patch obtained from
FreeBSD PR 218081 [1].
1. ht...
f84666f923e1d89c2cf3f5a31075985846016c13 authored over 7 years ago by Bradley T. Hughes <[email protected]>
PR-URL: https://github.com/nodejs/node/pull/13142
Reviewed-By: Matthew Loring <mattloring@google...
PR-URL: https://github.com/nodejs/node/pull/13142
Reviewed-By: Matthew Loring <mattloring@google...
PR-URL: https://github.com/nodejs/node/pull/11608
Reviewed-By: Matteo Collina <matteo.collina@gm...
Migrate console.js to use internal/errors.js.
PR-URL: https://github.com/nodejs/node/pull/11340...
0ecdf2934039b6e847aa3d1441e3ec235a70d125 authored over 7 years ago by mskec <[email protected]>
Currently the InspectorStarted function is guarded by the else clause of
the NODE_USE_V8_PLATFOR...
In test-child-process-spawnsync-validation-errors, check that functions
used inappropriately as ...
Add napi_get_version function so that addons can
query the level of N-API supported.
PR-URL: ht...
d9ee297afb54a3991bffb67b56370cec58c6e2c8 authored over 7 years ago by Michael Dawson <[email protected]>Currently the following compiler warning is displayed:
../src/inspector_agent.cc:218:5: warni...
2db655675caec32b7fedc5d75533c9eb6399322b authored over 7 years ago by Daniel Bevenius <[email protected]>
- The V8 inspector is no longer experimental.
- Note that building without SSL disables other fe...
Refactored the `unescapeBuffer` function in order to simplify it,
and also to improve the perfor...
This commit adds test coverage for the scenario where a socket's
handle has been closed prior to...
Implement server.keepAliveTimeout in addition to server.timeout to
prevent temporary socket/memo...
This fixes a resource leak detected by a Coverity scan.
PR-URL: https://github.com/nodejs/node/...
21c0c0275a553bc38fbac99129702f5b7c8ed1d3 authored over 7 years ago by Eugene Ostroukhov <[email protected]>
Fixes: https://github.com/nodejs/node/issues/13176
PR-URL: https://github.com/nodejs/node/pull/1...
`AsyncEvent` is not a good name given its semantics.
PR-URL: https://github.com/nodejs/node/pul...
90dee89d8ee4e6aaa78e163997bb14ad3719a79a authored over 7 years ago by Anna Henningsen <[email protected]>
In Writable, 'finish' was not emitted when using writev() and
cork() in the event of an Error du...
PR-URL: https://github.com/nodejs/node/pull/13013
Reviewed-By: James M Snell <[email protected]>
PR-URL: https://github.com/nodejs/node/pull/13182
Reviewed-By: Refael Ackermann <[email protected]...
Original commit message:
Allow embedder to set promise internal field count
Asynchronous co...
e6395cc07ae54f81699e369d292da29ecf113a16 authored over 7 years ago by Matt Loring <[email protected]>
PR-URL: https://github.com/nodejs/node/pull/13187
Reviewed-By: Benjamin Gruenbaum <benjamingr@gm...
Use `common.mustNotCall()` to confirm that noop callbacks are not run
when functions throw error...
root_cert_store is defined as extern in node_crypto.h but only used in
node_crypto.cc. It is the...
If node is configured --without-inspector/--without-ssl this test will
fail with the following e...
Add test to cover napi_has_named_property
PR-URL: https://github.com/nodejs/node/pull/13178
Rev...
Make the style of "Note:" paragraphs consistent and document the
guidelines in `doc/STYLE_GUIDE....
PR-URL: https://github.com/nodejs/node/pull/13028
Reviewed-By: Refael Ackermann <[email protected]>
also added some missing bits that didn't make it into #12818
PR-URL: https://github.com/nodejs/...
a109032260148242fa1f1b911a1a8a142ce95d86 authored over 7 years ago by Daniel Pihlstrom <[email protected]>
Use common.mustNotCall() to confirm that listener handles never run (as
no events are emitted).
...
* add block-scoping
* use common.mustCall() on callbacks that should not execute
PR-URL: https:...
16357f2d728c3ef8d42e3da97b77a57be4979f0a authored over 7 years ago by Rich Trott <[email protected]>
PR-URL: https://github.com/nodejs/node/pull/13174
Reviewed-By: Andreas Madsen <[email protected]...
This commit adds test coverage for the case where a dgram socket
successfully binds, but the han...
PR-URL: https://github.com/nodejs/node/pull/13126
Refs: https://github.com/whatwg/url/issues/297...
This commit attempts to fix the following TODO:
// TODO(bnoordhuis) Should perhaps also check wh...