- Oct 29, 2017
-
-
Nigel Kibodeaux authored
PR-URL: https://github.com/nodejs/node/pull/15963 Reviewed-By:
Anatoli Papirovski <apapirovski@mac.com> Reviewed-By:
Colin Ihrig <cjihrig@gmail.com> Reviewed-By:
Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By:
Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By:
Michael Dawson <michael_dawson@ca.ibm.com>
-
Raphael Rheault authored
PR-URL: https://github.com/nodejs/node/pull/15904 Reviewed-By:
Daijiro Wachi <daijiro.wachi@gmail.com> Reviewed-By:
Tobias Nießen <tniessen@tnie.de>
-
Sarah Meyer authored
Create a minimal test for the util benchmark files. PR-URL: https://github.com/nodejs/node/pull/16050 Reviewed-By:
Joyee Cheung <joyeec9h3@gmail.com> Reviewed-By:
-
Scott J Beck authored
PR-URL: https://github.com/nodejs/node/pull/15843 Reviewed-By:
-
Hadis-Fard authored
Improve assert messages in test-cluster-worker-disconnect.js. PR-URL: https://github.com/nodejs/node/pull/16021 Reviewed-By:
Rich Trott <rtrott@gmail.com>
-
Cheyenne Arrowsmith authored
PR-URL: https://github.com/nodejs/node/pull/15979 Reviewed-By:
Luigi Pinca <luigipinca@gmail.com> Reviewed-By:
-
Tom Boutell authored
PR-URL: https://github.com/nodejs/node/pull/16005 Fixes: https://github.com/nodejs/node/issues/15864 Reviewed-By:
Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By:
James M Snell <jasnell@gmail.com> Reviewed-By:
Anna Henningsen <anna@addaleax.net> Reviewed-By:
Timothy Gu <timothygu99@gmail.com> Reviewed-By:
Michaël Zasso <targos@protonmail.com> Reviewed-By:
-
Jon Moss authored
PR-URL: https://github.com/nodejs/node/pull/16507 Refs: https://github.com/nodejs/nodejs.org/pull/1425 Reviewed-By:
Yuta Hiroto <hello@about-hiroppy.com> Reviewed-By:
-
Jon Moss authored
Should not be nested under the bullet point. Headings need to be at the root level. PR-URL: https://github.com/nodejs/node/pull/16507 Refs: https://github.com/nodejs/nodejs.org/pull/1425 Reviewed-By:
-
nhoel authored
Specifically for the test-crypto-hash.js test file. PR-URL: https://github.com/nodejs/node/pull/15996 Reviewed-By:
-
Rami Moshe authored
PR-URL: https://github.com/nodejs/node/pull/15565 Reviewed-By:
Refael Ackermann <refack@gmail.com> Reviewed-By:
-
- Oct 28, 2017
-
-
Bradley Farias authored
PR-URL: https://github.com/nodejs/node/pull/16261 Reviewed-By:
Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By:
-
Guy Bedford authored
The reason is that absolute URLs do not go through extension and index checks. By switching to an absolute path, the resolver still applies extensions properly to the top-level main. PR-URL: https://github.com/nodejs/node/pull/16526 Fixes: https://github.com/nodejs/node/issues/16476 Reviewed-By:
Myles Borins <myles.borins@gmail.com> Reviewed-By:
-
guybedford authored
It can be useful to load dependencies as part of the loader hook definition file. This fixes a bug where `import x from 'x'` would always return `x` as `undefined` if the import was made in a loader hooks definition module. A parallel change to the CJS loading injection process meant that the CJS module wasn't being injected into the correct loader instance, which is corrected here with a test. PR-URL: https://github.com/nodejs/node/pull/16381 Reviewed-By:
-
Michaël Zasso authored
Original commit message: [heap] Print the number of chunks in unmapper queue in --trace-gc-nvp Bug: chromium:771966 Change-Id: I146b279c4713b7dd716c6d55ca5e6c6e23a3ad7e Reviewed-on: https://chromium-review.googlesource.com/704740 Reviewed-by:Michael Lippautz <mlippautz@chromium.org> Commit-Queue: Ulan Degenbaev <ulan@chromium.org> Cr-Commit-Position: refs/heads/master@{#48338} Refs: https://github.com/v8/v8/commit/e0d64dc67c6abf36a37efdc6e8e6903bb114ebd3 Refs: https://github.com/nodejs/help/issues/917#issuecomment-339292642 Refs: https://bugs.chromium.org/p/chromium/issues/detail?id=771966 PR-URL: https://github.com/nodejs/node/pull/16490 Reviewed-By:
Franziska Hinkelmann <franziska.hinkelmann@gmail.com> Reviewed-By:
Ali Ijaz Sheikh <ofrobots@google.com> Reviewed-By:
-
Michaël Zasso authored
Original commit message: [heap] Fix threshold for delayed chunks after 2c7561. Bug: chromium:771966 Change-Id: Iac5ee55c0d31de477f21f091f4be015a1ca8d00c Reviewed-on: https://chromium-review.googlesource.com/702382 Reviewed-by: -
Michaël Zasso authored
Original commit message: [heap] Ensure progress in unmapping memory chunks. If sweeping is not making progress and there are many young generation GCs happening, then this can lead to accumulation of memory chunks in the unmapper queue. Bug: chromium:771966 Change-Id: Ief73ada0d17198a80b668850c6d2e7ea413113e7 Reviewed-on: https://chromium-review.googlesource.com/702479 Reviewed-by: -
sharkfisher authored
PR-URL: https://github.com/nodejs/node/pull/16047 Reviewed-By:
-
Anatoli Papirovski authored
Adjust stream buffer size to allow full 4 default-sized frames including their frame headers to allow more efficient sending of data to the socket. PR-URL: https://github.com/nodejs/node/pull/16445 Reviewed-By:
-
Rich Trott authored
Replace process.config.target_defaults.default_configuration check with process.features.debug. PR-URL: https://github.com/nodejs/node/pull/16537 Ref: https://github.com/nodejs/node/pull/4431#issuecomment-173663527 Reviewed-By:
-
Rich Trott authored
use `common.buildType` instead of `process.config.target_defaults.default_configuration` in repl-domain-abort addon test. PR-URL: https://github.com/nodejs/node/pull/16538 Reviewed-By:
Daniel Bevenius <daniel.bevenius@gmail.com> Reviewed-By:
-
- Oct 27, 2017
-
-
Gibson Fahnestock authored
If you run the tests in a different machine to the one you built on, this test will fail. Avoid this by skipping if the file doesn't exist. We shouldn't need to check that the file exists in this test, as the build won't pass without a config.gypi anyway. Also adds console.logs, so you can see what the actual difference between the objects was, as `assert.deepStrictEqual()` only shows you the first three lines. PR-URL: https://github.com/nodejs/node/pull/16436 Reviewed-By:
Richard Lau <riclau@uk.ibm.com> Reviewed-By:
-
James M Snell authored
With the exception of ThrowCryptoError, use internal/errors to report fips unavailable or forced PR-URL: https://github.com/nodejs/node/pull/16428 Reviewed-By:
Matteo Collina <matteo.collina@gmail.com>
-
Sebastiaan Deckers authored
The PRIORITY field is only supported by HTTP/2 in a HEADERS frame initiated by the client, not in a push request sent by the server. Documents the recommended approach to set a silent priority as used internally by nghttp2. PR-URL: https://github.com/nodejs/node/pull/16451 Reviewed-By:
-
Mark Walker authored
PR-URL: https://github.com/nodejs/node/pull/15939 Reviewed-By:
-
Michaël Zasso authored
Currently, `Reflect.ownKeys(this)`, `Object.getOwnPropertyNames(this)` and `Object.getOwnPropertySymbols(this)` when run in a sandbox, fails to retrieve Symbol and non-enumerable values. PR-URL: https://github.com/nodejs/node/pull/16410 Reviewed-By:
-
Franziska Hinkelmann authored
This patch fixes the problem with variables that are declared only on the sandbox but not on the global proxy. PR-URL: https://github.com/nodejs/node/pull/16487 Fixes: https://github.com/nodejs/node/issues/12300 Reviewed-By:
-
erwinwahyura authored
PR-URL: https://github.com/nodejs/node/pull/16542 Reviewed-By:
Vse Mozhet Byt <vsemozhetbyt@gmail.com>
-
Peter Marton authored
PR-URL: https://github.com/nodejs/node/pull/16142 Reviewed-By:
-
Bryan English authored
Rather than using `http`, use `_http_client`, etc. directly. Also moving all the exports to the bottom, in line with most of the rest of the codebase. PR-URL: https://github.com/nodejs/node/pull/16395 Reviewed-By:
-
Refael Ackermann authored
also undocument the `vcbuild.bat` command since it's broken and seems to only be relevant to release builds PR-URL: https://github.com/nodejs/node/pull/16372 Refs: https://github.com/nodejs/node/pull/16010 Refs: https://github.com/nodejs/node/issues/16278 Reviewed-By:
Gibson Fahnestock <gibfahn@gmail.com> Reviewed-By:
-
Refael Ackermann authored
`.tmp` prefix allows easier exclusion PR-URL: https://github.com/nodejs/node/pull/16372 Refs: https://github.com/nodejs/node/pull/16010 Refs: https://github.com/nodejs/node/issues/16278 Reviewed-By:
-
Refael Ackermann authored
Makes eslint exclude directories without enumerating their content PR-URL: https://github.com/nodejs/node/pull/16372 Refs: https://github.com/nodejs/node/pull/16010 Refs: https://github.com/nodejs/node/issues/16278 Reviewed-By:
-
Hitesh Kanwathirtha authored
PR-URL: https://github.com/nodejs/node/pull/16460 Reviewed-By:
-
Joyee Cheung authored
Make node_crypto_clienthello-inl.h self-contained. PR-URL: https://github.com/nodejs/node/pull/16518 Reviewed-By:
-
James M Snell authored
The previous throws should never happen, and if they do, they signal a larger issue in core. Make these checks rather than throws. PR-URL: https://github.com/nodejs/node/pull/16453 Reviewed-By:
-
Bartosz Sosnowski authored
Libuv 1.15.0 improved console resize detection on Windows. This note is no longer needed. PR-URL: https://github.com/nodejs/node/pull/16320 Reviewed-By:
Jeremiah Senkpiel <fishrock123@rocketmail.com> Reviewed-By:
-
- Oct 26, 2017
-
-
Jinwoo Lee authored
This is for issue 16452. When 'set-cookie' header is set with an array that has only one string value, it's split into its individual characters. Fix by resetting `isArray` to false when the value is converted from an array to a string. Fixes: https://github.com/nodejs/node/issues/16452 PR-URL: https://github.com/nodejs/node/pull/16458 Reviewed-By:
-
James M Snell authored
PR-URL: https://github.com/nodejs/node/pull/16454 Reviewed-By:
-
James M Snell authored
PR-URL: https://github.com/nodejs/node/pull/16450 Reviewed-By:
Teddy Katz <teddy.katz@gmail.com>
-
