Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

v6.11.2 proposal #14356

Merged
merged 222 commits into from
Aug 1, 2017
Merged

v6.11.2 proposal #14356

merged 222 commits into from
Aug 1, 2017

Conversation

MylesBorins
Copy link
Contributor

@MylesBorins MylesBorins commented Jul 18, 2017

2017-08-01, Version 6.11.2 'Boron' (LTS), @MylesBorins

This LTS release comes with 221 commits. This includes 80 which are test related,
52 which are doc related, 32 which are build / tool related and 10 commits which are updates to dependencies.

Notable Changes

  • configure:
    • add mips64el to valid_arch (Aditya Anand) #13620
  • crypto:
  • deps:
    • upgrade OpenSSL to version 1.0.2.l (Shigeki Ohtsu) #12913
  • http:
    • parse errors are now reported when NODE_DEBUG=http (Sam Roberts) #13206
    • Agent construction can now be envoked without new (cjihrig) #12927
  • zlib:
    • node will now throw an Error when zlib rejects the value of windowBits, instead of crashing (Alexey Orlenko) #13098

Commits

  • [8d043876c1] - doc/tools: fix more type inconsistencies (Roman Reiss) #11697
  • [8860117600] - addons: remove semicolons from after module definition (Gabriel Schulhof) #12919
  • [bb3f54771b] - benchmark: update an obsolete path (Vse Mozhet Byt) #12904
  • [7cc68e2c62] - benchmark: add final clean-up to module-loader.js (Vse Mozhet Byt) #12102
  • [0cc7addcb2] - benchmark,windows: TCP.readStart() meaningful only after completion (Refael Ackermann) #12258
  • [8dec80211e] - build: run test-hash-seed at the end of test-v8 (Michaël Zasso) #14219
  • [bb1b06a4e5] - build: check for linter in bin rather than lib (Rich Trott) #13645
  • [f571868b1b] - build: fail linter if linting not available (Gibson Fahnestock) #13658
  • [b0c6bf829b] - build: use existing variable to reduce complexity (Bryce Baril) #2883
  • [ebbde61927] - build: xz tarball extreme compression (Peter Dave Hello) #10626
  • [a354134f6a] - build: ignore more VC++ artifacts (Refael Ackermann) #13208
  • [85829a65e8] - build: avoid /docs/api and /docs/doc/api upload (Rod Vagg) #12957
  • [7bda9620c9] - build: simplify if in setting of arg_paths (Refael Ackermann) #12653
  • [2724fe34ef] - build: add static option to vcbuild.bat (Tony Rice) #12764
  • [7458d4ef98] - build: disable -O3 for C++ coverage (Anna Henningsen) #12406
  • [8b8bf39822] - build: avoid passing kill empty input in Makefile (Gibson Fahnestock) #12158
  • [914f368efd] - build: clear stalled jobs on POSIX CI hosts (Rich Trott) #11246
  • [890e210a5f] - build: fix openssl link error on windows (Daniel Bevenius) #13078
  • [3bb117e310] - build: enable cctest to use generated objects (Daniel Bevenius) #11956
  • [e5ca046c0a] - build, doc, tools: add eslint-plugin-markdown (Vse Mozhet Byt) #14067
  • [b46cf35526] - child_process: fix deoptimizing use of arguments (Vse Mozhet Byt) #11535
  • [edbe442938] - cluster, dns, repl, tls, util: fix RegExp nits (Vse Mozhet Byt) #13536
  • [a5f3b6fa7c] - configure: add mips64el to valid_arch (Aditya Anand) #13620
  • [3b44e5e32c] - crypto: return CHECK_OK in VerifyCallback (Daniel Bevenius) #13241
  • [1bfd177f09] - crypto: update root certificates (Ben Noordhuis) #13279
  • [b6f3581ea4] - crypto: update root certificates (Ben Noordhuis) #12402
  • [1d509801e9] - crypto: throw proper errors if out enc is UTF-16 (Anna Henningsen) #12752
  • [8f8dd97072] - crypto: clear err stack after ECDH::BufferToPoint (Ryan Kelly) #13275
  • [3891759afc] - deps: update openssl asm and asm_obsolete files (Shigeki Ohtsu) #12913
  • [92583c4c81] - deps: cherry-pick 4ae5993 from upstream OpenSSL (Shigeki Ohtsu) #12913
  • [ee40a73d44] - deps: update openssl asm and asm_obsolete files (Daniel Bevenius) #13233
  • [a6a85c49c3] - deps: update openssl config files (Daniel Bevenius) #13233
  • [a579a776a3] - deps: add -no_rand_screen to openssl s_client (Shigeki Ohtsu) nodejs/io.js#1836
  • [b937c41405] - deps: fix asm build error of openssl in x86_win32 (Shigeki Ohtsu) iojs/io.js#1389
  • [69570d370a] - deps: fix openssl assembly error on ia32 win32 (Fedor Indutny) iojs/io.js#1389
  • [5703d22849] - deps: copy all openssl header files to include dir (Daniel Bevenius) #13233
  • [77a9198aca] - deps: upgrade openssl sources to 1.0.2l (Daniel Bevenius) #13233
  • [5b4c431365] - deps: add example of comparing OpenSSL changes (Daniel Bevenius) #13234
  • [18cbee236d] - dns: fix crash using dns.setServers after resolve4 (XadillaX) #13050
  • [8c0849d5db] - doc: conform to rules for eslint-plugin-markdown (Vse Mozhet Byt) #12563
  • [7deb259ccb] - doc: prepare js code for eslint-plugin-markdown (Vse Mozhet Byt) #12563
  • [59eb761797] - doc: document and test that methods return this (Sam Roberts) #13553
  • [fcb27fa7a1] - doc: remove leftover WHATWG url.format section (Roman Reiss) #14351
  • [e400ef9a76] - doc: don't suggest setEncoding for binary streams (Rick Bullotta) #11363
  • [092bba5cbf] - doc: update backporting guide (Refael Ackermann) #13749
  • [e2abda87f5] - doc: mention rebasing of v?.x-staging post release (Anna Henningsen) #13742
  • [24feb333c8] - doc: path.relative uses cwd (DuanPengfei) #13714
  • [71581e9308] - doc: small makeover for onboarding.md (Anna Henningsen) #13413
  • [8f430e774b] - doc: note that EoL platforms are not supported (Gibson Fahnestock) #12672
  • [9fa70069b3] - doc: use HTTPS URL for suggested upstream remote (Nikolai Vavilov) #13602
  • [fa209323af] - doc: update new CTC members (Refael Ackermann) #13534
  • [054f8cdc4d] - doc: corrects reference to tlsClientError (Tarun) #13533
  • [17da29ce84] - doc: emphasize Collaborators in GOVERNANCE.md (Rich Trott) #13423
  • [aea953abc2] - doc: minimal documentation for Emeritus status (Rich Trott) #13421
  • [42a42c0892] - doc: remove note highlighting in GOVERNANCE doc (Rich Trott) #13420
  • [cc492c361f] - doc: resume a stream after pipe() and unpipe() (Matteo Collina) #13329
  • [ae00f25a69] - doc: suggest xcode-select --install (Gibson Fahnestock) #13264
  • [8daab3be31] - doc: remove 'you' from writing-tests.md (Michael Dawson) #13319
  • [f2ede07f17] - doc: add tniessen to collaborators (Tobias Nießen) #13371
  • [a33c6759b6] - doc: create list of CTC emeriti (Rich Trott) #13232
  • [3745fbaa5d] - doc: remove Gitter badge from README (Rich Trott) #13231
  • [a7b51af049] - doc: make spelling of behavior consistent (Michael Dawson) #13245
  • [277de4257d] - doc: add jasongin & kunalspathak to collaborators (Jason Ginchereau) #13200
  • [fb07fbcc81] - doc: don't use useless constructors in stream.md (Vse Mozhet Byt) #13145
  • [cb03bd1f48] - doc: update code example for Windows in stream.md (Vse Mozhet Byt) #13138
  • [079b04e58d] - doc: improve formatting of STYLE_GUIDE.md (Alexey Orlenko) #13135
  • [5f87252969] - doc: fix incorrect keyboard shortcut (Alexey Orlenko) #13134
  • [d4edc82aa5] - doc: edit Error.captureStackTrace html comment (Artur Vieira) #12962
  • [1f9713362d] - doc: add additional useful ci job to list (Michael Dawson) #13086
  • [2d5e2e9cab] - doc: document method for reverting commits (Gibson Fahnestock) #13015
  • [b31e6dfef5] - doc: update COLLABORATOR_GUIDE.md (morrme) #12555
  • [b854d27330] - doc: Change options at STEP 5 in CONTRIBUTING.md (kysnm) #12830
  • [c01a2d545e] - doc: add docs for server.address() for pipe case (Flarna) #12907
  • [83f272d4ee] - doc: fix typo in streams.md (Glenn Schlereth) #12924
  • [28add410c2] - doc: improve path.posix.normalize docs (Steven Lehn) #12700
  • [023ec46d2c] - doc: remove test-npm from general build doc (Rich Trott) #12840
  • [74a6929938] - doc: upgrade Clang requirement to 3.4.2 (Michaël Zasso) #12388
  • [5b379e0aad] - doc: clarify the callback arguments of dns.resolve (Roman Reiss) #9532
  • [f6e58c35b2] - doc: add missing make command to UPGRADING.md (Daniel Bevenius) #13233
  • [a7869541e4] - doc: increase Buffer.concat() documentation (cjihrig) #11845
  • [3b1d9112e0] - doc: update readFileSync in fs.md (Aditya Anand) #12800
  • [bc66495061] - doc: document vm timeout option perf impact (Anna Henningsen) #12751
  • [a3ae360ea6] - doc: modernize and fix code examples in repl.md (Vse Mozhet Byt) #12634
  • [2435af9db6] - doc: update os.uptime() and process.uptime() info (Vse Mozhet Byt) #12294
  • [b2e58b6c7a] - doc: minor improvements in BUILDING.md (Sakthipriyan Vairamani (thefourtheye)) #11963
  • [7ba172f56f] - doc: argument types for https methods (Amelia Clarke) #11681
  • [eb9e281b6b] - doc: update output examples in debugger.md (Vse Mozhet Byt) #10944
  • [b62cec8b02] - doc: linkify type[] syntax, support lowercase for primitives (Roman Reiss) #11167
  • [dd1fb98bda] - doc: consistent case for primitive types (Roman Reiss) #11167
  • [c43866954e] - doc,build: update configure help messages (Gibson Fahnestock) #12978
  • [0d35bcdf84] - doc,stream: clarify 'data', pipe() and 'readable' (Matteo Collina) #13432
  • [351be2d5a8] - dtrace: resolve conversion warnings from SLURP_INT (Christopher J. Brody) #10143
  • [046bd79cf7] - events: remove unreachable code (cjihrig) #12501
  • [8bf64d135f] - events: do not keep arrays with a single listener (Luigi Pinca) #12043
  • [f66f09f5d1] - http: describe parse err in debug output (Sam Roberts) #13206
  • [cab1285ccf] - http: fix first body chunk fast case for UTF-16 (Anna Henningsen) #12747
  • [01302989a7] - https: support rejectUnauthorized for unix sockets (cjihrig) #13505
  • [d51cd61713] - https: support agent construction without new (cjihrig) #12927
  • [5eb11ba73e] - lib: correct typo in createSecureContext (Daniel Bevenius) #13653
  • [102671823c] - lib: "iff" changed to "if and only if" (Jacob Jones) #13496
  • [1609c7f0c5] - lib: remove useless default caught (Jackson Tian) #12884
  • [ef133b36c5] - lib,test: use regular expression literals (Rich Trott) #12807
  • [0cb5bd7268] - meta: fix nits in README.md collaborators list (Vse Mozhet Byt) #12866
  • [4c51d969ee] - openssl: fix keypress requirement in apps on win32 (Shigeki Ohtsu) iojs/io.js#1389
  • [47e702059d] - os,vm: fix segfaults and CHECK failure (Tobias Nießen) #12371
  • [c97b167f47] - profiler: declare missing printErr (Fedor Indutny) #13590
  • [bd323a71a8] - repl: fix /dev/null history file regression (Brian White) #12762
  • [b2acb81016] - repl: support hidden history file on Windows (Bartosz Sosnowski) #12207
  • [79592fe44a] - src: correct indentation for X509ToObject (Daniel Bevenius) #13543
  • [69143ffcf9] - src: make IsConstructCall checks consistent (Daniel Bevenius) #13473
  • [48f00b5170] - src: add comment for TicketKeyCallback (Anna Henningsen) #13193
  • [37e1929257] - src: check IsConstructCall in TLSWrap constructor (Daniel Bevenius) #13097
  • [2e23da1a12] - src: remove unused node_buffer.h include (Daniel Bevenius) #13095
  • [41661287f2] - src: split CryptoPemCallback into two functions (Daniel Bevenius) #12827
  • [f92e065d12] - src: assert that uv_async_init() succeeds (cjihrig) #13116
  • [f43c969061] - src: turn buffer type-CHECK into exception (Anna Henningsen) #12753
  • [19259f46d0] - src: rename CryptoPemCallback -> PasswordCallback (Daniel Bevenius) #12787
  • [7aa5a993b2] - src: make cross-context MakeCallback() calls work (Ben Noordhuis) #9221
  • [b1dc2d455f] - src: remove superfluous env_string string (Ben Noordhuis) #9213
  • [48a923af37] - stream: remove unnecessary parameter (Leo) #12767
  • [9cfec4ba0f] - test: fix RegExp nits (Vse Mozhet Byt) #13770
  • [a3e2560f7a] - test: mark test-npm-install flaky on arm (Refael Ackermann) #14035
  • [8a7f13bd00] - test: mark test-fs-readdir-ucs2 flaky (João Reis) #13989
  • [34fc7a03d2] - test: change deprecated method to recommended (Rich Trott) #13649
  • [ef3698cad8] - test: refactor test-cluster-worker-isconnected.js (cjihrig) #13685
  • [fa75be7901] - test: fix nits in test-fs-mkdir-rmdir.js (Vse Mozhet Byt) #13680
  • [9e9a9c342c] - test: increase bufsize in child process write test (Rich Trott) #13626
  • [53b345c506] - test: fix flaky test-tls-socket-close (Rich Trott) #13529
  • [a37165a2cc] - test: exercise once() with varying arguments (cjihrig) #13524
  • [779402ec5f] - test: validate full error messages (aniketshukla) #13453
  • [7190d06d1f] - test: add known_test request with Unicode in the URL (David D Lowe) #13297
  • [cbcc9c1bbf] - test: add coverage for socket write after close (cjihrig) #13171
  • [47d59e7f97] - test: fix sequential test-net-connect-local-error (Sebastian Plesciuc) #13064
  • [1d3596561b] - test: bind to 0 in dgram-send-callback-buffer-length (Artur Vieira) #12943
  • [7909c6d46f] - test: use dynamic port in test-dgram-send-callback-buffer (Artur Vieira) #12942
  • [92cc96fa6b] - test: allow for absent nobody user in setuid test (Rich Trott) #13112
  • [253c5aa794] - test: move net reconnect error test to sequential (Artur G Vieira) #13033
  • [e279eb5aa3] - test: ignore spurious 'EMFILE' (Refael Ackermann) #12698
  • [3e5e38e868] - test: use dynamic port in test-cluster-dgram-reuse (Artur Vieira) #12901
  • [5fe68402bd] - test: refactor test-vm-new-script-new-context (Akshay Iyer) #13035
  • [2aa68282fc] - test: track callback invocations (Rich Trott) #13010
  • [0c83573b61] - test: add a simple abort check in windows (Sreepurna Jasti) #12914
  • [07137ab4db] - test: fix too optimistic guess in setproctitle (Vse Mozhet Byt) #12792
  • [7419338b33] - test: make the rest of tests path-independent (Vse Mozhet Byt) #12972
  • [ac400a7b09] - test: check curve algorithm is supported (Karl Cheng)
  • [5b74e635e5] - test: reduce string concatenations (Vse Mozhet Byt) #12735
  • [c902265b90] - test: fix parallel/test-setproctitle.js on alpine (David Cai) #12413
  • [50bb452510] - test: fixed flaky test-net-connect-local-error (Sebastian Plesciuc) #12964
  • [0cf3e10ce2] - test: remove unneeded string splitting (Vse Mozhet Byt) #12992
  • [6e7b77fdbb] - test: use mustCall in tls-connect-given-socket (vperezma) #12592
  • [c10525c562] - test: add not-called check to heap-profiler test (Rich Trott) #12985
  • [2451665157] - test: move test-dgram-bind-shared-ports to sequential (Rafael Fragoso) #12452
  • [d35648ffc2] - test: use dynamic port in test-https-connect-address-family (Artur G Vieira) #12915
  • [1cd41e7a56] - test: dynamic port in cluster disconnect (Sebastian Plesciuc) #12545
  • [d71de281fa] - test: detect all types of aborts in windows (Gireesh Punathil) #12856
  • [d743783875] - test: use assert regexp in tls no cert test (Artur Vieira) #12891
  • [29d35d0ef1] - test: use dynamic port instead of common.PORT (Aditya Anand) #12473
  • [186c0758b3] - test: added net.connect lookup type check (Luca Maraschi) #11873
  • [c35f4909f4] - test: remove unused testpy code (Rich Trott) #12844
  • [52b7d5ecb1] - test: refactor test-querystring (Łukasz Szewczak) #12661
  • [8414659d02] - test: refactoring test with common.mustCall (weewey) #12702
  • [608c30913e] - test: refactored test-repl-persistent-history (cool88) #12703
  • [aaf8044a81] - test: remove common.PORT in test tls ticket cluster (Oscar Martinez) #12715
  • [802a945d81] - test: add mustCall in timers-unrefed-in-callback (Zahidul Islam) #12594
  • [739c579134] - test: fix flakyness with yes.exe (Refael Ackermann) #12821
  • [14e835831f] - test: dynamic port in dgram tests (Sebastian Plesciuc) #12623
  • [361bc845dc] - test: verify listener leak is only emitted once (cjihrig) #12502
  • [f236dcbdd9] - test: move WPT to its own testing module (Rich Trott) #12736
  • [4eb28c80e8] - test: introduce common.crashOnUnhandledRejection (Anna Henningsen) #12489
  • [2411318f60] - test: add second argument to assert.throws (Michaël Zasso) #12270
  • [eca9e72a87] - test: add regex in test_cyclic_link_protection (Clarence Dimitri CHARLES) #11622
  • [6020e720b5] - test: improve test-fs-open-flags (Vinícius do Carmo) #10908
  • [e6d6a4111c] - test: extended test to makeCallback cb type check (Luca Maraschi) #12140
  • [d74019d98d] - test: improve test-crypto-rsa-dsa (Adrian Estrada) #10681
  • [bab8a36f94] - test: improve the code in test-crypto-dh (Adrian Estrada) #10734
  • [752bc24943] - test: validate errors in test-buffer-indexof (Adrian Estrada) #10752
  • [9e7f02187a] - test: improve test-buffer-includes.js (toboid) #11203
  • [c309bb0695] - test: validate error message from buffer.equals (Sebastian Roeder) #11215
  • [62c56806fc] - test: add msg validation to test-buffer-compare (Josh Hollandsworth) #10807
  • [fc9e7a98ed] - test: make tests cwd-independent (Vse Mozhet Byt) #12812
  • [fff0e39933] - test: add regex check in test-vm-is-context (jeyanthinath) #12785
  • [74dc86d239] - test: add callback to fs.close() in test-fs-stat (Vse Mozhet Byt) #12804
  • [a47a9b7cf4] - test: add callback to fs.close() in test-fs-chmod (Vse Mozhet Byt) #12795
  • [eefa840118] - test: increase readline coverage (Anna Henningsen) #12761
  • [54decfa2ce] - test: replace indexOf with includes (gwer) #12604
  • [03adb94ee6] - test: dynamic port in parallel regress tests (Sebastian Plesciuc) #12639
  • [8a59f6b038] - test: dynamic port in cluster worker wait close (Sebastian Plesciuc) #12466
  • [0383048b76] - test: fix coverity UNINIT_CTOR cctest warning (Ben Noordhuis) #12387
  • [f2467edc62] - test: remove common.PORT from multiple tests (Tarun Batra) #12451
  • [a23aca4f12] - test: replace [].join() with ''.repeat() (Jackson Tian) #12305
  • [e512906aab] - test: run the addon tests last (Sebastian Van Sande) #12062
  • [abc2c82bf3] - test: remove disabled test-dgram-send-error (Rich Trott) #12330
  • [d9866ce9c7] - test: remove disabled tls_server.js (Rich Trott) #12275
  • [19d95519c7] - test: add basic cctest for base64.h (Alexey Orlenko) #12238
  • [01073bc26a] - test: add internal/socket_list tests (DavidCai) #12109
  • [a5fe098b85] - test: move common.PORT debug tests to sequential (Gibson Fahnestock) #13592
  • [0b8adedb88] - test: move test-debug-brk to sequential (Gibson Fahnestock) #13580
  • [97b6911ade] - test: enable setuid/setgid test (Rich Trott) #12403
  • [4dff12849f] - test,doc: document crashOnUnhandledRejection() (Anna Henningsen) #12699
  • [7e6a956a29] - test,lib,doc: use function declarations (Rich Trott) #12711
  • [910fa50e0e] - tools: fix error in custom ESLint rule (Rich Trott) #13758
  • [bb74da309c] - tools: apply stricter indentation rules to tools (Rich Trott) #13758
  • [04934b04c3] - tools: fix indentation in required-modules.js (Rich Trott) #13758
  • [550577749f] - tools: remove no-useless-regex-char-class-escape (Rich Trott) #10561
  • [4ffe804c81] - tools: update ESLint to v4.0.0 (Rich Trott) #13645
  • [fb214bbcff] - tools: be explicit about including key-id (Myles Borins) #13309
  • [f831015928] - tools: update certdata.txt (Ben Noordhuis) #13279
  • [bc2e73a05f] - tools: update certdata.txt (Ben Noordhuis) #12402
  • [99da83b54d] - tools: relax lint rule for regexps (Rich Trott) #12807
  • [3d564a4ed1] - tools: require function declarations (Rich Trott) #12711
  • [6afa5fe348] - tools: add table parsing capability to the doctool (Roman Reiss) #9532
  • [9c67032b9a] - tools: enforce two arguments in assert.throws (Michaël Zasso) #12270
  • [95d13d59e4] - tools: remove unused code from test.py (Rich Trott) #12806
  • [70e9058a8e] - tools: ignore node_trace.*.log (Daijiro Wachi) #12754
  • [61427471af] - tools: replace custom assert.fail lint rule (Rich Trott) #12287
  • [b2a08fb130] - tools: replace custom new-with-error rule (Rich Trott) #12249
  • [beb8485998] - tools: fix lint issue in doctool (Roman Reiss) #11658
  • [d9a8f80c0d] - v8: fix build errors with g++ 7 (Zuzana Svetlikova) #12392
  • [8b3aacc96a] - vm: fix race condition with timeout param (Marcel Laverdet) #13074
  • [6e60c838c9] - vm: fix displayErrors in runIn.. functions (Marcel Laverdet) #13074
  • [55cbe24c60] - zlib: fix node crashing on invalid options (Alexey Orlenko) #13098

danbev and others added 30 commits July 11, 2017 17:42
This commit tries to make it simpler to add unit tests (cctest) for
code that needs to test node core funtionality but that might not be
appropriate as an addon or a JavaScript test. An example of this could
be adding functionality targeted for situations when Node itself is
embedded.

Currently it was not as easy, or efficient, as one would have hoped to
add such tests. The object output directories vary for different
operating systems which we need to link to so that we don't have an
additional compilation step.

PR-URL: #11956
Backport-PR-URL: #12948
Ref: #9163
Reviewed-By: James M Snell <jasnell@gmail.com>
This commit attempts to fix an issue when building on windows using the
following command line options:
.\vcbuild.bat dll debug x64 vc2015

This will result in the following options passed to configure:
configure --debug --shared --dest-cpu=x64 --tag=

This commit excludes the dependency to openssl if node is configured
with --shared.

Also, FP_API to the categories to export in mkssldef when generating
the module definition (openssl.def) allowing the build to compile and
link successfully.

Fixes: #12952
PR-URL: #13078
Backport-PR-URL: #12948
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Refael Ackermann <refack@gmail.com>
PR-URL: #11167
Backport-PR-URL: #13054
Reviewed-By: Timothy Gu <timothygu99@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
PR-URL: #11167
Backport-PR-URL: #13054
Reviewed-By: Timothy Gu <timothygu99@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
PR-URL: #11658
Backport-PR-URL: #13054
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
- fix a number of uppercase types
- lowercase 'integer'
- consistent formatting in crypto

PR-URL: #11697
Backport-PR-URL: #13054
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Timothy Gu <timothygu99@gmail.com>
Refactor test for situations where it was expected to fail.
Move from disabled directory to parallel.

PR-URL: #12403
Backport-PR-URL: #13060
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
This is a partial backport of semver-patch bits of
9e4660b.

This commit fixes the Node process crashing when constructors of classes
of the zlib module are given invalid options.

* Throw an Error when the zlib library rejects the value of windowBits,
  instead of crashing with an assertion.

* Treat windowBits and memLevel options consistently with other ones and
  don't crash when non-numeric values are given.

PR-URL: #13098
Backport-PR-URL: #13201
Fixes: #13082
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Functions that call `ECDH::BufferToPoint` were not clearing the
error stack on failure, so an invalid key could leave leftover
error state and cause subsequent (unrelated) signing operations
to fail.

PR-URL: #13275
Backport-PR-URL: #13397
Reviewed-By: Fedor Indutny <fedor.indutny@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
This is a local patch because upstream fixed it differently by moving
large chunks of code out of objects.h.  We cannot easily back-port
those changes due to their size and invasiveness.

Fixes: #10388
PR-URL: #12392
Backport-PR-URL: #13574
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
The test uses common.PORT, and has already been deleted on master.

PR-URL: #13580
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
They tend to hang if they happen to run in parallel with another test
that uses common.PORT.

PR-URL: #13592
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
PR-URL: #10944
Backport-PR-URL: #13751
Reviewed-By: Josh Gavant <joshgavant@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
PR-URL: #12102
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
PR-URL: #12109
Reviewed-By: James M Snell <jasnell@gmail.com>
Ref: #9399
PR-URL: #11681
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
1. necessarily reliably => necessarily reliable
2. projects root directory => project's root directory
3. remove `console` highlighting, as `test` alone is highlighted
4. fix broken link for Android NDK
5. highlight the directory location `/usr/local/ssl/fips-2.0`
6. update expected output to an example for `process.versions.openssl` as the
   version displayed is not mentioned in the document

PR-URL: #11963

Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
This commit adds C++ tests for `base64_encode()` and `base64_decode()`
functions defined in `base64.h`.  The functionality is already being
tested indirectly in JavaScript tests for Buffer, but it won't hurt to
test the low-level functions too, especially given that they aren't only
used in the internal Buffer implementation, Chrome inspector protocol
support relies upon them too.

PR-URL: #12238
Refs: #12146 (comment)
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com>
Use no-restricted-syntax to implement the requirement that `Error`
objects must be thrown with the `new` keyword.

PR-URL: #12249
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Teddy Katz <teddy.katz@gmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Remove tls_server.js that has been disabled for about 6 years. It
appears to have worked in concert with some other file which has since
been removed. It seems to create a server and set up a bunch of
listeners, but it does not appear to have code that connects to the
server and triggers any of those listeners.

PR-URL: #12275
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Replace custom lint rule for `assert.fail()` function signature errors
with a restricted-syntax rule.

PR-URL: #12287
Reviewed-By: Teddy Katz <teddy.katz@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Sometimes, after a cluster or debug test fails, a fixture hangs around
and holds onto a needed port, causing subsequent CI runs to fail. This
adds a command I've been running manually when this occurs. The command
will clear the stalled jobs before a CI run.

PR-URL: #11246
Backport-PR-URL: #13754
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
Reviewed-By: Josh Gavant <josh.gavant@outlook.com>
PR-URL: #12294
Fixes: #12291
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
It's only used once at startup in a single place so create the string
in place instead of caching it for the lifetime of the isolate.

PR-URL: #9213
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Check that invoking a callback on a receiver from a different context
works.

It ran afoul of an `env->context() == isolate->GetCurrentContext()`
assertion so retrieve the environment from the callback context and
the context to enter from the environment's context() method.

We could also have retrieved the environment from the receiver's context
and that would have made little practical difference.  It just seemed
more correct to get it from the callback context because that is the
actual execution context.

PR-URL: #9221
Reviewed-By: Anna Henningsen <anna@addaleax.net>
This test was disabled in 2013 because it spams random IPs with UDP
messages. We've been doing fine for four years without so let's delete
it.

PR-URL: #12330
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Using `xargs -r` on some platforms and `xargs` on others doesn't work,
we can't guarantee whether xargs is GNU or not. Avoid the issue by only
running kill if there are processes to clean.

PR-URL: #12158
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Running the addon tests before the parallel, sequential, etc. tests
can be a problem if there is a bug in node that prevents the
addon tests from running properly. When the addon tests fail for any
reason, then none of the other tests (e.g. parallel, etc.)
are executed.

Running the addon tests last fixes this.

Refs: #12031
PR-URL: #12062
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
The `cflags` for `--coverage` included `-O0` so far, but that was
overridden by a later `-O3`. Resolve that by adding
`'cflags!': [ '-O3' ]` and increase coverage accuracy.

Ref: https://coverage.nodejs.org/
PR-URL: #12406
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
fixes #11972

PR-URL: #12258
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
targos and others added 2 commits July 21, 2017 14:35
The v8 and test-hash-seed targets cannot be run in parallel because they
need different copies of the deps/v8 directory.

Ref: #14004 (comment)
PR-URL: #14219
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
* Take RegExp creation out of cycles.
* Use test(), not match() in boolean context.
* Remove redundant RegExp parts.

Backport-PR-URL: #14348
PR-URL: #13536
Reviewed-By: Anna Henningsen <anna@addaleax.net>
MylesBorins added a commit that referenced this pull request Jul 21, 2017
Notable Changes:

Coming Soon

PR-URL: #14356
@MylesBorins
Copy link
Contributor Author

@MylesBorins
Copy link
Contributor Author

vsemozhetbyt and others added 12 commits July 30, 2017 23:11
This is an initial step to eliminate most of parsing errors.

Backport-PR-URL: #14067
PR-URL: #12563
Refs: #12557 (comment)
Reviewed-By: Teddy Katz <teddy.katz@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Timothy Gu <timothygu99@gmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
Backport-PR-URL: #14067
PR-URL: #12563
Refs: #12557 (comment)
Reviewed-By: Teddy Katz <teddy.katz@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Timothy Gu <timothygu99@gmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
* Install eslint-plugin-markdown@1.0.0-beta.7
* Add doc/.eslintrc.yaml
* Add `plugins: [markdown]` to the main .eslintrc.yaml
* .js files in doc folder added to .eslintignore
* Update Makefile, vcbuild.bat, and tools/jslint.js

Refs: #12563
Refs: #12640
Refs: #14047

PR-URL: #14067
Reviewed-By: James Snell <jasnell@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
The `no-useless-regex-char-class-escape` custom lint rule was introduced
as a less aggressive alternative to some enhancements that were
introduced into ESLint. Those enhancements were blocking us from
updating ESLint. However, they have since been relaxed and the custom
rule is no longer needed. Remove it.

Backport-PR-URL: #14360
PR-URL: #10561
Reviewed-By: Teddy Katz <teddy.katz@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
In preparation for applying the more strict indentation linting
available in ESLint 4.0.0, correct minor indentation issues in
tools/eslint-rules/required-modules.js.

This is the only file with indentation that does not conform to the
stricter checks.

Backport-PR-URL: #14360
PR-URL: #13758
Reviewed-By: Teddy Katz <teddy.katz@gmail.com>
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
ESLint 4.0.0 provides stricter (and more granular) indentation checking
than previous versions. Apply the stricter indentation rules to the
tools directory.

Backport-PR-URL: #14360
PR-URL: #13758
Reviewed-By: Teddy Katz <teddy.katz@gmail.com>
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Fix previously-unnoticed typo in `required-modules.js`.

Backport-PR-URL: #14360
Refs: #13758 (comment)
PR-URL: #13758
Reviewed-By: Teddy Katz <teddy.katz@gmail.com>
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
* Remove needless RegExp flag

  In fixed case, `/g` flag is needless in the boolean context.

* Remove needless RegExp capturing

  Use non-capturing grouping or remove capturing completely when:

  * capturing is useless per se, e.g. in test() check;
  * captured groups are not used afterward at all;
  * some of the later captured groups are not used afterward.

* Use test, not match/exec in boolean context

  match() and exec() return a complicated object,
  unneeded in a boolean context.

* Do not needlessly repeat RegExp creation

  This commit takes RegExp creation out of cycles and other repetitions.

  As long as the RegExp does not use /g flag and match indices,
  we are safe here.

  In tests, this fix hardly gives a significant performance gain,
  but it increases clarity and maintainability,
  reassuring some RegExps to be identical.

  RegExp in functions are not taken out of their functions:
  while these functions are called many times
  and their RegExps are recreated with each call,
  the performance gain in test cases
  does not seem to be worth decreasing function self-dependency.

Backport-PR-URL: #14370
PR-URL: #13770
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
This option has been broken for almost a year when used with any of the
vm.runIn.. family of functions, except for syntax errors.

Backport-PR-URL: #14373
PR-URL: #13074
Reviewed-By: Anna Henningsen <anna@addaleax.net>
This fixes a race condition in the watchdog timer used for vm timeouts.
The condition would terminate the main stack's execution instead of the
code running under the sandbox.

Backport-PR-URL: #14373
PR-URL: #13074
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Backport-PR-URL: #14416
PR-URL: #13206
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Brian White <mscdex@mscdex.net>
This commit allows self signed certificates to work with
unix sockets by forwarding the rejectUnauthorized option.

Backport-PR-URL: #14415
Fixes: #13470
PR-URL: #13505
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com>
MylesBorins added a commit that referenced this pull request Jul 31, 2017
Notable Changes:

Coming Soon

PR-URL: #14356
@MylesBorins
Copy link
Contributor Author

MylesBorins commented Jul 31, 2017

Dropped a commit breaking windows CI

CI: https://ci.nodejs.org/job/node-test-pull-request/9414/
CITGM: https://ci.nodejs.org/view/Node.js-citgm/job/citgm-smoker/930/

edit: CI is green. Nothing in CITGM looks particularly out of the ordinary

This LTS release comes with 221 commits. This includes 80 which are
test related, 52 which are doc related, 32 which are build / tool
related and 10 commits which are updates to dependencies.

Notable Changes:

* configure:
  - add mips64el to valid_arch (Aditya Anand)
    - #13620
* crypto:
  - Updated root certificates based on [NSS 3.30] (Ben Noordhuis)
    - #13279
    - #12402
    - https://developer.mozilla.org/en-US/docs/Mozilla/Projects/NSS/NSS_3.30_release_notes
* deps:
  - upgrade OpenSSL to version 1.0.2.l (Shigeki Ohtsu)
    - #12913
* http:
  - parse errors are now reported when NODE_DEBUG=http (Sam Roberts)
    - #13206
  - Agent construction can now be envoked without `new` (cjihrig)
    - #12927
* zlib:
  - node will now throw an Error when zlib rejects the value of windowBits,
    instead of crashing (Alexey Orlenko)
    - #13098

PR-URL: #14356
@MylesBorins MylesBorins merged commit f15e124 into v6.x Aug 1, 2017
MylesBorins added a commit that referenced this pull request Aug 1, 2017
MylesBorins added a commit that referenced this pull request Aug 1, 2017
This LTS release comes with 221 commits. This includes 80 which are
test related, 52 which are doc related, 32 which are build / tool
related and 10 commits which are updates to dependencies.

Notable Changes:

* configure:
  - add mips64el to valid_arch (Aditya Anand)
    - #13620
* crypto:
  - Updated root certificates based on [NSS 3.30] (Ben Noordhuis)
    - #13279
    - #12402
    - https://developer.mozilla.org/en-US/docs/Mozilla/Projects/NSS/NSS_3.30_release_notes
* deps:
  - upgrade OpenSSL to version 1.0.2.l (Shigeki Ohtsu)
    - #12913
* http:
  - parse errors are now reported when NODE_DEBUG=http (Sam Roberts)
    - #13206
  - Agent construction can now be envoked without `new` (cjihrig)
    - #12927
* zlib:
  - node will now throw an Error when zlib rejects the value of windowBits,
    instead of crashing (Alexey Orlenko)
    - #13098

PR-URL: #14356
@MylesBorins MylesBorins deleted the v6.11.2-proposal branch August 1, 2017 07:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Issues and PRs related to build files or the CI. doc Issues and PRs related to the documentations. meta Issues and PRs related to the general management of the project. openssl Issues and PRs related to the OpenSSL dependency. tools Issues and PRs related to the tools directory.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet