- Mar 28, 2018
-
-
Shigeki Ohtsu authored
This replaces all sources of openssl-1.0.2o.tar.gz into deps/openssl/openssl PR-URL: https://github.com/nodejs/node/pull/19638 Reviewed-By:
Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By:
Colin Ihrig <cjihrig@gmail.com> Reviewed-By:
Tobias Nießen <tniessen@tnie.de> Reviewed-By:
Rod Vagg <rod@vagg.org>
-
Ben Noordhuis authored
Original commit message follows: Before this commit `Content-Length: 4 2` was accepted as a valid header and recorded as `parser->content_length = 42`. Now it is a parse error that fails with error `HPE_INVALID_CONTENT_LENGTH`. Downstream users that inspect `parser->content_length` and naively parse the string value using `strtoul()` might get confused by the discrepancy between the two values. Resolve that by simply not letting it happen. Fixes: https://github.com/nodejs-private/security/issues/178 PR-URL: https://github.com/nodejs-private/http-parser-private/pull/1 Reviewed-By:Сковорода Никита Андреевич <chalkerx@gmail.com> Reviewed-By:
Evan Lucas <evanlucas@me.com> Reviewed-By:
Fedor Indutny <fedor.indutny@gmail.com> Reviewed-By:
James M Snell <jasnell@gmail.com> Reviewed-By:
-
Ben Noordhuis authored
PR-URL: https://github.com/nodejs-private/http-parser-private/pull/1 Reviewed-By:
-
- Mar 27, 2018
-
-
Anna Henningsen authored
PR-URL: https://github.com/nodejs/node/pull/19558 Reviewed-By:
Tiancheng "Timothy" Gu <timothygu99@gmail.com> Reviewed-By:
Trivikram Kamat <trivikr.dev@gmail.com> Reviewed-By:
-
Thomas Hunter II authored
PR-URL: https://github.com/nodejs/node/pull/19620 Reviewed-By:
Rich Trott <rtrott@gmail.com> Reviewed-By:
Vse Mozhet Byt <vsemozhetbyt@gmail.com> Reviewed-By:
Anna Henningsen <anna@addaleax.net>
-
Rich Trott authored
This changes a sentence fragment into a full sentence and provides a few other minor improvements. PR-URL: https://github.com/nodejs/node/pull/19622 Reviewed-By:
Shingo Inoue <leko.noor@gmail.com> Reviewed-By:
Ruben Bridgewater <ruben@bridgewater.de>
-
Rich Trott authored
Rewording, punctuation, consistent sentence structure and italics, wrap section at 80 characters. PR-URL: https://github.com/nodejs/node/pull/19567 Reviewed-By:
Luigi Pinca <luigipinca@gmail.com> Reviewed-By:
Gibson Fahnestock <gibfahn@gmail.com>
-
Rich Trott authored
PR-URL: https://github.com/nodejs/node/pull/19567 Reviewed-By:
-
Rich Trott authored
Change v6 to 6.0.0. We abandoned v-notation for versions to avoid confusion between v8 (version 8.0.0) and V8 (the JavaScript engine). PR-URL: https://github.com/nodejs/node/pull/19567 Reviewed-By:
-
Vse Mozhet Byt authored
PR-URL: https://github.com/nodejs/node/pull/19583 Reviewed-By:
-
Timothy Gu authored
Previously, "HTTP/2" was strictly used to describe the protocol, and HTTP2 the module. This distinction is deemed unnecessary, and consistency between the two terms is enforced. PR-URL: https://github.com/nodejs/node/pull/19603 Reviewed-By:
Chen Gang <gangc.cxy@foxmail.com> Reviewed-By:
-
Anna Henningsen authored
It’s not obvious what the paragraph is supposed to say. In particular, whether and what kind of buffering mechanism a process uses for its stdio streams does not affect that, in general, no guarantees can be made about when it consumes data that was sent to it. PR-URL: https://github.com/nodejs/node/pull/19552 Reviewed-By:
-
Beth Griggs authored
PR-URL: https://github.com/nodejs/node/pull/19610 Reviewed-By:
Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By:
Yuta Hiroto <hello@hiroppy.me> Reviewed-By:
Richard Lau <riclau@uk.ibm.com> Reviewed-By:
Joyee Cheung <joyeec9h3@gmail.com>
-
davis.okoth@kemsa.co.ke authored
Removing third argument in calls to assert.strictEqual() so that the values of the first two arguments are shown instead as this is more useful for debugging. Refs: https://nodejs.org/api/assert.html#assert_assert_strictequal_actual_expected_message PR-URL: https://github.com/nodejs/node/pull/19536 Reviewed-By:
Franziska Hinkelmann <franziska.hinkelmann@gmail.com> Reviewed-By:
-
Ayush Gupta authored
Documented `make docopen` as a way to read documentation in the browser. PR-URL: https://github.com/nodejs/node/pull/19321 Reviewed-By:
-
Dan Kaplun authored
Fixes test/parallel/test-console-no-swallow-stack-exceeded.js PR-URL: https://github.com/nodejs/node/pull/19423 Reviewed-By:
-
Vse Mozhet Byt authored
PR-URL: https://github.com/nodejs/node/pull/19539 Reviewed-By:
Daijiro Wachi <daijiro.wachi@gmail.com>
-
Vse Mozhet Byt authored
PR-URL: https://github.com/nodejs/node/pull/19599 Reviewed-By:
-
Rich Trott authored
Remove instances of `Example:` that introduce code that is self-evidently example code. Move descriptive text about examples into comments in the code. Wrap adjacent text to 80 characters. PR-URL: https://github.com/nodejs/node/pull/19582 Reviewed-By:
-
Tobias Nießen authored
PR-URL: https://github.com/nodejs/node/pull/19590 Reviewed-By:
Michaël Zasso <targos@protonmail.com> Reviewed-By:
-
DingDean authored
Default error message contains the actual and expected values, which is more informative than the custom error message to be deleted. PR-URL: https://github.com/nodejs/node/pull/19526 Reviewed-By:
-
Nathaniel Weeks authored
PR-URL: https://github.com/nodejs/node/pull/19276 Reviewed-By:
-
Tobias Nießen authored
PR-URL: https://github.com/nodejs/node/pull/19585 Refs: https://github.com/nodejs/node/pull/18801 Reviewed-By:
-
Christopher Hiller authored
PR-URL: https://github.com/nodejs/node/pull/19487 Reviewed-By:
-
juggernaut451 authored
PR-URL: https://github.com/nodejs/node/pull/18802 Fixes: https://github.com/nodejs/node/issues/18774 Reviewed-By:
Matheus Marchini <matheus@sthima.com> Reviewed-By:
-
jvelezpo authored
Adds the added in label on the documentation for each one of the properties of Class: fs.Stats PR-URL: https://github.com/nodejs/node/pull/19266 Fixes: https://github.com/nodejs/node/issues/19178 Reviewed-By:
Adrian Estrada <edsadr@gmail.com> Reviewed-By:
-
willhayslett authored
Converted the 'message' argument values from the last two free socket assert.strictEqual() calls to code comments as they fail to provide the necessary details and values specific to why the test is failing. The default message returned from the strictEqual() call should provide sufficient details for debugging errors. PR-URL: https://github.com/nodejs/node/pull/19525 Reviewed-By:
Daniel Bevenius <daniel.bevenius@gmail.com> Reviewed-By:
-
Tobias Nießen authored
PR-URL: https://github.com/nodejs/node/pull/19512 Reviewed-By:
-
Tobias Nießen authored
PR-URL: https://github.com/nodejs/node/pull/19512 Reviewed-By:
-
Tobias Nießen authored
PR-URL: https://github.com/nodejs/node/pull/19578 Refs: https://github.com/nodejs/node/pull/17412 Reviewed-By:
-
Ujjwal Sharma authored
PR-URL: https://github.com/nodejs/node/pull/19495 Refs: https://github.com/nodejs/node/issues/19105 Reviewed-By:
-
juggernaut451 authored
PR-URL: https://github.com/nodejs/node/pull/19092 Fixes: https://github.com/nodejs/node/issues/14544 Reviewed-By:
-
Vse Mozhet Byt authored
`require.main` was documented in a non-standard way. With this PR, the previous section is left as is to not break all the possible link references inside and outside Node.js docs. A standard section is added to the `require` properties with a reference to the old description. PR-URL: https://github.com/nodejs/node/pull/19573 Reviewed-By:
-
Tobias Nießen authored
PR-URL: https://github.com/nodejs/node/pull/19569 Refs: https://github.com/nodejs/node/pull/18399 Reviewed-By:
-
Vse Mozhet Byt authored
PR-URL: https://github.com/nodejs/node/pull/19571 Reviewed-By:
-
Rich Trott authored
A few bug fixes result in more stringent linting rules. PR-URL: https://github.com/nodejs/node/pull/19528 Reviewed-By:
-
Rich Trott authored
ESLint 4.19.1 fixes some bugs in rules. These changes prepare us for the upgrade. PR-URL: https://github.com/nodejs/node/pull/19528 Reviewed-By:
-
Steven R. Loomis authored
- using a length of 6 for `"hello"` includes a terminating null yielding `"hello\u0000"` - length of 5 would work, but comparing to the N-API docs gives `NAPI_AUTO_LENGTH` instead PR-URL: https://github.com/nodejs/node/pull/19205 Reviewed-By:
Michael Dawson <michael_dawson@ca.ibm.com>
-
Rich Trott authored
The buffer module was introduced in v0.1.90. Updated `introduced_in` value in `buffer.md` to reflect this. PR-URL: https://github.com/nodejs/node/pull/19545 Reviewed-By:
-
Rich Trott authored
PR-URL: https://github.com/nodejs/node/pull/19547 Reviewed-By:
-
