- Jul 18, 2020
-
-
Joyee Cheung authored
This runs `lib/internal/bootstrap/loaders.js` before creating the builtin snapshot and deserialize the loaders from the snapshot in deserialization mode. PR-URL: https://github.com/nodejs/node/pull/32984 Reviewed-By:
Anna Henningsen <anna@addaleax.net> Reviewed-By:
Daniel Bevenius <daniel.bevenius@gmail.com>
-
Joyee Cheung authored
Keep track of snapshotted modules in JS land, and move bootstrap switches into StartExecution() so that they are not included into part of the environment-independent bootstrap process. PR-URL: https://github.com/nodejs/node/pull/32984 Reviewed-By:
-
Joyee Cheung authored
This includes the initial Environment (without running bootstrap scripts) into the builtin snapshot PR-URL: https://github.com/nodejs/node/pull/32984 Reviewed-By:
-
Joyee Cheung authored
Pass the flags down to node_mksnapshot so that we can use them when generating the snapshot (e.g. to debug or enable V8 flags) PR-URL: https://github.com/nodejs/node/pull/32984 Reviewed-By:
-
Joyee Cheung authored
Add an ExternalReferenceRegistry class for registering static external references. To register the external JS to C++ references created in a binding (e.g. when a FunctionTemplate is created): - Add the binding name (same as the id used for `internalBinding()` and `NODE_MODULE_CONTEXT_AWARE_INTERNAL`) to `EXTERNAL_REFERENCE_BINDING_LIST` in `src/node_external_reference.h`. - In the file where the binding is implemented, create a registration function to register the static C++ references (e.g. the C++ functions in `v8::FunctionCallback` associated with the function templates), like this: ```c++ void RegisterExternalReferences( ExternalReferenceRegistry* registry) { registry->Register(cpp_func_1); } ``` - At the end of the file where `NODE_MODULE_CONTEXT_AWARE_INTERNAL` is also usually called, register the registration function with ``` NODE_MODULE_EXTERNAL_REFERENCE(binding_name, RegisterExternalReferences); ``` PR-URL: https://github.com/nodejs/node/pull/32984 Reviewed-By: -
Joyee Cheung authored
So that it's possible to create an Environment not yet attached to any V8 context. We'll use this to deserialize the V8 context before attaching an Environment to it. PR-URL: https://github.com/nodejs/node/pull/32984 Reviewed-By:
-
- Jul 17, 2020
-
-
Rich Trott authored
Switch to tmpDir rather than fixtures for file writing. PR-URL: https://github.com/nodejs/node/pull/34376 Reviewed-By:
Richard Lau <riclau@uk.ibm.com> Reviewed-By:
James M Snell <jasnell@gmail.com>
-
Rich Trott authored
Extra output makes test-benchmark-streams fail. Change console.log() to console.assert(). Fixes: https://github.com/nodejs/node/issues/34409 PR-URL: https://github.com/nodejs/node/pull/34411 Reviewed-By:
Robert Nagy <ronagy@icloud.com>
-
rickyes authored
PR-URL: https://github.com/nodejs/node/pull/34354 Reviewed-By:
Matteo Collina <matteo.collina@gmail.com> Reviewed-By:
Zeyu Yang <himself65@outlook.com> Reviewed-By:
-
Robert Nagy authored
Reimplement as an async generator instead of a custom iterator class. PR-URL: https://github.com/nodejs/node/pull/34035 Reviewed-By:
Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By:
-
Rich Trott authored
common.localhostIPv6 is almost entirely unused and is unnecessary. Remove it. PR-URL: https://github.com/nodejs/node/pull/34373 Reviewed-By:
Michaël Zasso <targos@protonmail.com> Reviewed-By:
Luigi Pinca <luigipinca@gmail.com> Reviewed-By:
Trivikram Kamat <trivikr.dev@gmail.com> Reviewed-By:
-
Stewart Addison authored
Zone IDs on Linux are network interface names. The regex we use to determine valid IPs does not allow for non-alphanumeric characters in the zone ID suffix. Some machines (including the RHEL Linux/s390x machines from Marist) have zone IDs with a '.' character in them which the regex in net.isIP rejects. This changes the regex. Ref: https://github.com/nodejs/node/pull/14500 Signed-off-by:
Stewart Addison <sxa@uk.ibm.com> PR-URL: https://github.com/nodejs/node/pull/34364 Reviewed-By:
Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By:
Rich Trott <rtrott@gmail.com> Reviewed-By:
Michael Dawson <michael_dawson@ca.ibm.com>
-
Rich Trott authored
Use `common.hasIPv6` instead of an OS check for more reliable operation. PR-URL: https://github.com/nodejs/node/pull/34359 Reviewed-By:
Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By:
Tobias Nießen <tniessen@tnie.de>
-
- Jul 16, 2020
-
-
Robert Nagy authored
finished should invoke callback on closed OutgoingMessage the same way as for regular streams. Fixes: https://github.com/nodejs/node/issues/34301 PR-URL: https://github.com/nodejs/node/pull/34313 Fixes: https://github.com/nodejs/node/issues/34274 Reviewed-By:
-
Rich Trott authored
PR-URL: https://github.com/nodejs/node/pull/34352 Reviewed-By:
-
James M Snell authored
PR-URL: https://github.com/nodejs/node/pull/34283 Reviewed-By:
-
James M Snell authored
When entering the closing or draining periods, servers should wait three times the current probe timeout before releasing session state. PR-URL: https://github.com/nodejs/node/pull/34283 Reviewed-By:
-
James M Snell authored
PR-URL: https://github.com/nodejs/node/pull/34283 Reviewed-By:
-
James M Snell authored
PR-URL: https://github.com/nodejs/node/pull/34283 Reviewed-By:
-
James M Snell authored
PR-URL: https://github.com/nodejs/node/pull/34283 Reviewed-By:
-
James M Snell authored
PR-URL: https://github.com/nodejs/node/pull/34283 Reviewed-By:
-
James M Snell authored
QuicClientSession and QuicServerSessions are now always immediately ready for use when they are created, so no more need to track ready state. PR-URL: https://github.com/nodejs/node/pull/34283 Reviewed-By:
-
James M Snell authored
PR-URL: https://github.com/nodejs/node/pull/34283 Reviewed-By:
-
James M Snell authored
PR-URL: https://github.com/nodejs/node/pull/34283 Reviewed-By:
-
James M Snell authored
This is the start of a conversion over to a fully Promise-centric API for the QUIC implementation. PR-URL: https://github.com/nodejs/node/pull/34283 Reviewed-By:
-
James M Snell authored
PR-URL: https://github.com/nodejs/node/pull/34283 Reviewed-By:
-
James M Snell authored
Ensure that the QuicSocket is properly destroyed if the QuicEndpoint is destroyed directly rather than through QuicSocket destroy PR-URL: https://github.com/nodejs/node/pull/34283 Reviewed-By:
-
James M Snell authored
PR-URL: https://github.com/nodejs/node/pull/34283 Reviewed-By:
-
James M Snell authored
Restricting this to pre-bind keeps things simple PR-URL: https://github.com/nodejs/node/pull/34283 Reviewed-By:
-
James M Snell authored
Doesn't need to be a function PR-URL: https://github.com/nodejs/node/pull/34283 Reviewed-By:
-
James M Snell authored
PR-URL: https://github.com/nodejs/node/pull/34283 Reviewed-By:
-
James M Snell authored
PR-URL: https://github.com/nodejs/node/pull/34283 Reviewed-By:
-
James M Snell authored
Since the `ipv6Only` option was mutually exclusive with using `'udp6'`, making it it's own type simplifies things a bit. PR-URL: https://github.com/nodejs/node/pull/34283 Reviewed-By:
-
rickyes authored
PR-URL: https://github.com/nodejs/node/pull/34339 Refs: https://nodejs.org/dist/latest-v14.x/docs/api/stream.html#stream_stream_finished_stream_options_callback Reviewed-By:
-
gengjiawen authored
PR-URL: https://github.com/nodejs/node/pull/34335 Reviewed-By:
-
Ye-hyoung Kang authored
PR-URL: https://github.com/nodejs/node/pull/34316 Reviewed-By:
-
Anna Henningsen authored
Add support for “sensitive”/“never-indexed” HTTP2 headers. Fixes: https://github.com/nodejs/node/issues/34091 PR-URL: https://github.com/nodejs/node/pull/34145 Reviewed-By:
Denys Otrishko <shishugi@gmail.com>
-
ExE Boss authored
Refs: https://github.com/nodejs/node/issues/31808 PR-URL: https://github.com/nodejs/node/pull/34032 Reviewed-By:
Gus Caplan <me@gus.host>
-
Francisco Ryan Tolmasky I authored
Prior to this commit, API docs would be highlighted after the page loaded using `highlightjs`. This commit still uses `highlightjs`, but runs the generation during the compilation of the documentation, making it so no script needs to load on the client. This results in a faster load, as well as allowing the pages to fully functional even when JavaScript is turned off. PR-URL: https://github.com/nodejs/node/pull/34148 Reviewed-By:
-
Robert Nagy authored
Due to compat reasons Transform streams don't always wait for flush to complete before finishing the stream. Try to wait when possible, i.e. when the user does not override _final. Fixes: https://github.com/nodejs/node/issues/34274 PR-URL: https://github.com/nodejs/node/pull/34314 Reviewed-By:
-
