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.12.1 proposal #17180

Merged
merged 264 commits into from
Dec 5, 2017
Merged

v6.12.1 proposal #17180

merged 264 commits into from
Dec 5, 2017

Conversation

MylesBorins
Copy link
Contributor

@MylesBorins MylesBorins commented Nov 21, 2017

2017-12-05, Version 6.12.1 'Boron' (LTS), @MylesBorins

This LTS release comes with 263 commits. This includes 173 which are test related,
41 which are doc related, 18 which are build / tool related and 1 commit which is an update to a dependency.

Notable Changes

  • build:
    • fix npm install with --shared (Ben Noordhuis) #16438
  • build:
    • building with python 3 is now supported (Emily Marigold Klassen) #16058
  • src:
    • v8 options can be specified with either '_' or '-' in NODE_OPTIONS (Sam Roberts) #14093

Commits

  • [575a920a16] - assert: fix actual and expected order (Steve Jenkins) #15866
  • [a0c1d10e91] - build: remove cctest extension (Yihong Wang) #16680
  • [c287f1235c] - build: include src\tracing when linting on win (Daniel Bevenius) #16720
  • [706812bc2f] - build: skip bin override on windows (Hitesh Kanwathirtha) #16460
  • [f4627603aa] - build: fix npm install with --shared (Ben Noordhuis) #16438
  • [6d63612e93] - build: correct minor typo in lttng help message (Daniel Bevenius) #16101
  • [de82db7f85] - build: ignore empty folders in test-addons (Gregor) #16031
  • [ac1beb0fb0] - build: use bin override if no python in PATH (Bradley T. Hughes) #16241
  • [d4b3b633d8] - build: allow build with system python 3 (Emily Marigold Klassen) #16058
  • [fc2ab06014] - build, windows: use /bigobj for debug builds (Nikolai Vavilov) #16289
  • [ccca11d026] - build,win: set /MP separately in Debug and Release (Nikolai Vavilov) #16415
  • [a14f564686] - build,win: use /MP for debug builds (Nikolai Vavilov) #16333
  • [8813867577] - child_process: set shell to false in fork() (Alex Gresnel) #15352
  • [f2cafff9b0] - crypto: fix error of createCipher in wrap mode (Shigeki Ohtsu) #15037
  • [7115079c4f] - crypto: warn if counter mode used in createCipher (Shigeki Ohtsu) #13821
  • [50c3dabc0f] - deps: backport 4af8029 from upstream V8 (Michaël Zasso) #17290
  • [101eb981fe] - doc: mention constant-time in crypto doc (Mithun Sasidharan) #16604
  • [1bc5c3836c] - doc: recommend node-core-utils for metadata (Rich Trott) #16978
  • [4583f1be0c] - doc: reorganize COLLABORATOR_GUIDE.md (Rich Trott) #15710
  • [fce790285f] - doc: improve documentation for the vm module (Franziska Hinkelmann) #16867
  • [727a0fe641] - doc: update subprocess.killed (cjihrig) #16748
  • [44c0385b04] - doc: more accurate zlib windowBits information (Anna Henningsen) #16511
  • [732af9b8a4] - doc: add Gibson Fahnestock to Release team (Gibson Fahnestock) #16620
  • [935b15285f] - doc: slightly relax 50 character rule (James M Snell) #16523
  • [39c63da6d2] - doc: add note to releases.md (Jon Moss) #16507
  • [60ae428f30] - doc: add dot in documentations (erwinwahyura) #16542
  • [7ae23b744b] - doc: fix missing newline character (Daijiro Wachi) #16447
  • [af869f03c1] - doc: add recommendations for first timers (Refael Ackermann) #16350
  • [b7d609c2f8] - doc: replace undocumented encoding aliases (Vse Mozhet Byt) #16368
  • [2cbf75da7e] - doc: replace methods used in the example code (Damian) #16416
  • [0b5a0ada2a] - doc: fix comment in assert.md (umatoma) #16335
  • [4fbc490704] - doc: add space after period (Diego Rodríguez Baquero) #16334
  • [c3cc0fd258] - doc: minor correction to note on process section (Daniel Bevenius) #16311
  • [47bf494979] - doc: add apapirovski to collaborators (Anatoli Papirovski) #16302
  • [9c96d7f4fd] - doc: clarify os.cpus() returns logical CPU cores (Luke Childs) #16282
  • [ba62b0e48a] - doc: support multidimensional arrays in type link (Vse Mozhet Byt) #16207
  • [aefaed40f0] - doc: move Shigeki to TSC Emeritus (Rich Trott) #16195
  • [1fdcf75f9c] - doc: Update a typo in module.js' comments (Orta) #16205
  • [799c6fdc1c] - doc: add missing comma (Jon Moss) #16204
  • [8c070f9ed5] - doc: added note to fs.watchFile on previousStat (NiveditN) #16099
  • [2515cad90e] - doc: ensure collaborators validate commits (Bradley Farias) #16162
  • [7647d41da1] - doc: move 8 collaborators to emeriti (Rich Trott) #16173
  • [de8155ebf2] - doc: include V8 commit URL in V8 backport guide (Gibson Fahnestock) #16054
  • [6f1ba792d7] - doc: add pronoun for fhinkel (F. Hinkelmann) #16069
  • [8da3b51472] - doc: document windows shell support (Tim Ermilov) #16104
  • [281023b20d] - doc: exempt test/doc only changes from 48-hr rule (Anna Henningsen) #16135
  • [04d5835722] - doc: rename good first contrib label (Jeremiah Senkpiel) #16150
  • [1064258f9d] - doc: remove bold typography from STYLE_GUIDE.md (Rich Trott) #16085
  • [23e9bba9c8] - doc: ctc -> tsc in onboarding extras (Bryan English) #15621
  • [ff66d63642] - doc: fix emitKeypressEvents stream type (Oblosys) #15399
  • [1bd6962842] - doc: make stream.Readable consistent (Sakthipriyan Vairamani (thefourtheye)) #16786
  • [6b9bd51021] - doc: correct effects to affects (gowpen) #16794
  • [6af9311939] - doc: correct EventEmitter reference (gowpen) #16791
  • [1a633e3cd8] - doc: add docs for Zlib#close() (Luigi Pinca) #16592
  • [290df5ac41] - doc: add details about rss on process.memoryUsage (Anthony Nandaa) #16566
  • [3e6da45ce0] - doc: howto decode buffers extending from Writable (dicearr) #16403
  • [c64ed977fc] - doc, win: remove note about resize (Bartosz Sosnowski) #16320
  • [644989cf6a] - fs: use Number::New since all fields are uint64_t (Huáng Jùnliàng) #16705
  • [925e58fecb] - fs: fix stat dev unsigned cast overflow (Huáng Jùnliàng) #16705
  • [92b13e455f] - https: Use secureProtocol in Agent#getName (Andreas Lind) #9452
  • [b0ac76d145] - meta: add note about email sync to CONTRIBUTING.md (Vse Mozhet Byt) #16340
  • [bf7f63d51b] - net: change assert to conform to other files (James Hodgskiss) #15861
  • [ae3ad5502b] - src: remove unused includes from node_wrap.h (Daniel Bevenius) #16179
  • [a368e5fa63] - src: make StreamBase prototype accessors robust (Joyee Cheung) #16860
  • [c79dd9e3ce] - src: CHECK() for argument overflow in Spawn() (cjihrig) #16761
  • [7c69ca58e0] - src: move handle properties to prototype (Ben Noordhuis) #16482
  • [c87a620ed8] - src: remove superfluous HandleScope (Ben Noordhuis) #16482
  • [2f5edc6fd5] - src: remove unused include in tty_wrap.h (Daniel Bevenius) #16379
  • [42cb64ee91] - src: fix etw provider include on Windows (Joyee Cheung) #16639
  • [b00ced5b52] - src: do not include x.h if x-inl.h is included (Joyee Cheung) #16548
  • [a4688b0c43] - src: make header file self-contained (Joyee Cheung) #16518
  • [cf80089477] - src: node_dtrace line continuations clean up (Daniel Bevenius) #15777
  • [4639cfff0a] - src: rename perfctr_macros.py->noperfctr_macros.py (Daniel Bevenius) #16100
  • [70f574e6ff] - src: add help for NODE_PENDING_DEPRECATION env (Thomas Corbière) #15609
  • [75b1e30e6b] - src: add --pending-deprecation to NODE_OPTIONS (Thomas Corbière) #15494
  • [f659e49862] - src: whitelist v8 options with '_' or '-' (Sam Roberts) #14093
  • [79171e0c2f] - src: turn key length exception into CHECK (Ben Noordhuis) #15183
  • [e18df46092] - src: notify V8 for low memory when alloc fails (Anna Henningsen) #8482
  • [6a0eb9f6cf] - src: provide allocation + nullptr check shortcuts (Anna Henningsen) #8482
  • [4aec8cfcd2] - src: pass desired return type to allocators (Anna Henningsen) #8482
  • [19f3ac9749] - src: add Malloc() size param + overflow detection (Anna Henningsen) #8482
  • [6269ba334d] - test: allow tests to pass without internet (Daniel Bevenius) #16255
  • [f0eeddb4b8] - test: reuse existing PassThrough implementation (Tobias Nießen) #16936
  • [4752fc4336] - test: refactor comments in test-child-process-spawnsync-maxbuf (ChrBergert) #16829
  • [f226ca6b12] - test: used fixturesDir from fixtures modules (Klemen Kogovsek) #16813
  • [5e2231e407] - test: add a test description (Grant Gasparyan) #16833
  • [a8cff7ad4a] - test: use common/fixtures module in hash-seed test (Javier Blanco) #16823
  • [090cc9713e] - test: improve template value for test message (Stephan Smith) #16826
  • [1d3793eb77] - test: change concatenated string to template (Deepthi Sebastian) #16929
  • [79dfc3f475] - test: change concatenated string to template (Anawesha Khuntia) #16912
  • [2232231d4f] - test: change string concatenation to template (Suryanarayana Murthy N) #16919
  • [674cbf8402] - test: replace string concatenation with template (Kabir Islam) #16916
  • [969defaae9] - test: enable mustCall() during child exit (Vipin Menon) #16915
  • [9d4abaa243] - test: replace string concatenation with template (Tanvi Kini) #16913
  • [2a1ebae567] - test: cover vm.runInNewContext() (cjihrig) #16906
  • [2043ce39d5] - test: improve assertion messages (Neil Vass) #16885
  • [668644008e] - test: improve assert messages in stream test (Katie Stockton Roberts) #16884
  • [714eb0bc7c] - test: improve assertion in test-require-dot (Adam Wegrzynek) #16805
  • [8e5b4f543c] - test: add values to error message (Adam Jeffery) #16831
  • [b3b7858a97] - test: replace common.fixtiresDir with fixtures.readKey() (woj) #16817
  • [3acf156b68] - test: remove message argument in cluster setup test (mbornath) #16838
  • [cedf8a1cb2] - test: move test-http-keepalive-maxsockets to sequential (Rich Trott) #16777
  • [ffbb4e68e8] - test: use default assertion message (jonask) #16819
  • [dd558a56af] - test: include file mode in assert message (Sascha Tandel) #16815
  • [3d8b3f7b4a] - test: refactor tls test to use fixtres.readSync (Brian O'Connell) #16816
  • [54d4557199] - test: use fixtures module in test-repl (Maring, Damian Lion) #16809
  • [9f9e824fc5] - test: update test to use fixtures.readKey (Dara Hayes) #16811
  • [a99755f3fd] - test: fix typos in read-buffer tests (Jimi van der Woning) #16834
  • [e7a456a5ee] - test: replace common.fixturesDir with fixtures module (Dumitru Glavan) #16803
  • [04af0fdab7] - test: replace common.fixturesDir with fixtures.readSync() (Adri Van Houdt) #16802
  • [755f5e3fd1] - test: update test to use fixtures (Adam Wegrzynek) #16799
  • [143d8a1b3d] - test: fix typo (Oscar Funes) #15938
  • [84741fdc81] - test: update test-timers-block-eventloop.js (zhangzifa) #16314
  • [8e62fcb2cf] - test: replace fixturesDir in test-tls-connect (Casie Lynch) #15849
  • [d6dc579f3c] - test: use fixtures module (Iryna Yaremtso) #15901
  • [10c24a157c] - test: add details in assertions in test-vm-context (Vladimir Ilic) #16116
  • [cb1d16d26b] - test: increase fs.exists coverage (Nigel Kibodeaux) #15963
  • [d3981ae552] - test: use fixtures module in test-fs-realpath.js (Raphael Rheault) #15904
  • [532c9606b3] - test: use fixtures module (Scott J Beck) #15843
  • [58fe9b4ec3] - test: imporove assert messages (Hadis-Fard) #16021
  • [91f9779794] - test: show values instead of assertion message (Cheyenne Arrowsmith) #15979
  • [0ace5a158d] - test: include values in assertion messages (nhoel) #15996
  • [8663b05711] - test: use process.features.debug in common module (Rich Trott) #16537
  • [1fffa165a1] - test: use common.buildType in repl-domain-abort (Rich Trott) #16538
  • [7d93da54bb] - test: skip test-process-config if no config.gypi (Gibson Fahnestock) #16436
  • [5c20164354] - test: use fixtures module in tls-handshake-error (Mark Walker) #15939
  • [4f04d15aa3] - test: add failing vm tests to known_issues (Michaël Zasso) #16410
  • [2b1042bb29] - test: allow for different nsswitch.conf settings (Daniel Bevenius) #16378
  • [5095b991c0] - test: handle blank shells in test-os.js (Gibson Fahnestock) #16287
  • [62dd6a2c40] - test: increase enoughTestMem to 1.75 Gb (Rich Trott) #16374
  • [9c229b4bd3] - test: use fixtures.readKey in https-timeout-server (Nicolas 'Pixel' Noble) #15871
  • [773652903d] - test: use fixtures.readKey instead of fixturesDir (Paul Marion Camantigue) #15976
  • [34dfce7710] - test: replace fixturesDir with fixtures module (tpurcell) #16262
  • [0a88e1bd60] - test: replace fixturesDir with fixtures module (André Føyn Berge) #15947
  • [9e74e542a2] - test: skip test due to file size limit (jBarz) #16273
  • [e070e592dd] - test: remove error msg in test-vm-symbols.js (Daniel Abrão) #15873
  • [257ece287c] - test: remove error messages in test-buffer-alloc (Braden Whitten) #15867
  • [32fa91519a] - test: update assert error messages (Omar Gonzalez) #16035
  • [da85e6c552] - test: expand error message (Stefania Sharp) #15991
  • [cbbe125f71] - test: use fixtures module (Kanika Shah) #15959
  • [6f15b011c0] - test: remove literal messages (Oscar Funes) #15938
  • [aa269ad59b] - test: fix stderr reference (Oscar Funes) #15938
  • [3f35fc063e] - test: use fixtures module in test-https-truncate (Gene Wu) #15875
  • [c58eaaf1a8] - test: use fixtures module (Alvaro Cruz) #15874
  • [48e1320c44] - test: use fixtures module (Lance Barlaan) #15872
  • [339bdca558] - test: use default message for assert.strictEqual (hwaisiu) #15970
  • [ab580c3ae2] - test: improve assert message in internet test (Nikki St Onge) #15998
  • [6285e7221e] - test: replace common.fixturesDir (Shawn McGinty) #15834
  • [fa8315cb68] - test: refactor test-process-kill-null (Luigi Pinca) #16236
  • [c26abc8e94] - test: add missing spaces in concatenations (Vse Mozhet Byt) #16244
  • [a94a75f69a] - test: update output to include exit code & signal (Jenna Zeigen) #15945
  • [8eb84d6780] - test: change common.fixturesDir to fixtures.path (tejbirsingh) #15860
  • [806f03e54c] - test: split up and refactor test-domain (Anna Henningsen) #13614
  • [e5fbc03563] - test: replace fixturesDir with common.fixtures (Kasim Doctor) #15810
  • [2ab826c497] - test: replaced fs.readSync with fixtures.readSync (Lam Chan) #15882
  • [1fe3e866cf] - test: improve coverage for process.umask (Evan Lucas) #16188
  • [0689ea66ed] - test: remove message from notStrictEqual (twk-b) #16048
  • [fafbbb6347] - test: use fixtures module (Ben Hallion) #15808
  • [f2108fa51d] - test: use ES6 classes instead of util.inherits (Tobias Nießen) #16938
  • [eb11a70424] - test: refactor test-cluster-setup-master (Jean-Baptiste Brossard) #16065
  • [e00a4c820f] - test: replace fixtureDir with fixtures methods (Vladimir Ilic) #16114
  • [f46e1187b3] - test: remove error messages in crypto-binary test (Kim Gentes) #15981
  • [086d8519a1] - test: use fixtures module over fixturesDir (JamesNimlos) #15847
  • [38179fd1ed] - test: use common.fixtures module (Shaun Sweet) #15992
  • [229a1fa299] - test: replace fixturesDir with fixtures.path (Bear Trickey) #15994
  • [c10594f70f] - test: update fixturesDir import (Tyler Seabrook) #15887
  • [53449f303f] - test: replace fixturesDir with fixtures methods (Komivi Agbakpem) #15967
  • [a28d666f0e] - test: replace fixturesDir with the fixtures module (WeiPlanet) #16027
  • [d59175090d] - test: change crypto decipheriv assertion messages (Daniel Kostro) #16007
  • [541866ea86] - test: replaces fixturesDir with fixtures (Mike Fleming) #15835
  • [57ae105c72] - test: remove test messages for assert.strictEqual (Ali Groening) #15995
  • [87b9b7c8c4] - test: move to common.fixtures (Justin Beckwith) #15987
  • [72f69f3c2c] - test: added fixtures module (Michael Pal) #15980
  • [65c5ff8e92] - test: use fixtures in test-tls-multi-key.js (Cheyenne Arrowsmith) #15844
  • [9eac5aab8c] - test: switch to use common.fixtures.fixturesDir (Roger Jiang) #15814
  • [449538851c] - test: use common.fixtures module (Chi-chi Wang) #16012
  • [04f3f6dd6a] - test: escape script filename on Windows (Bartosz Sosnowski) #16124
  • [501acdf38c] - test: improve assert message in test-dh-regr (Mabry Cervin) #15912
  • [4c98e07702] - test: fixtures in test-net-pipe-connect-errors (Eric Freiberg) #15922
  • [244bfb398d] - test: fixtures in test-process-redirect-warnings-env (Kat Rosario) #15930
  • [18479d3cff] - test: fix ordering of strictEqual actual/expected (Chad Zezula) #16008
  • [66fd6a1409] - test: use fixtures.readSync (szhang351)
  • [6d33564b1a] - test: replaced fixturesDir with common.fixtures (Dolapo Toki) #15836
  • [a6f04bec9e] - test: use fixtures.fixturesDir (Gene Wu) #15822
  • [2103453977] - test: replaces fixturesDir with fixtures methods (Christian Murphy) #15817
  • [e705ad2076] - test: fixtures in test-process-redirect-warnings (Nicolas Chaulet) #15917
  • [9ddbcc877b] - test: update test-crypto-from-binary (Raj Parekh) #16011
  • [6b8830c1df] - test: use fixtures in test-https-set-timeout-server (Bob Clewell) #15886
  • [57590cd097] - test: make use of common/fixtures.fixturesDir (Jem Bezooyen) #15815
  • [c9d07faa04] - test: use common/fixtures in test-https-close (Alberto Lopez de Lara) #15870
  • [68a2d394dd] - test: use fixtures in test-process-warnings (Suresh Srinivas) #15869
  • [28756b318a] - test: use fixtures in tls-friendly-error-message (tobyfarley) #15905
  • [a05fe5f716] - test: use common/fixtures in tls-connect-no-host (Donovan Buck) #15986
  • [cf31eb7532] - test: use common/fixtures in test-https-agent (jpaulptr) #15941
  • [c9c37d076c] - test: use common fixtures module (Kat Rosario) #15856
  • [76ab029bea] - test: fs.readFileSync -> fixtures.readKey (Ethan Brown) #16030
  • [dabdb2d186] - test: reduce run time for misc benchmark tests (Rich Trott) #16120
  • [3f56ac4450] - test: improve assertion message in dgram test (Shakeel Mohamed) #16121
  • [44a60c3807] - test: use of fixtures in test-pipe-head (Nicolas Chaulet) #15868
  • [c4db4e44b8] - test: use fixtures in test-https-localaddress.js (Charles T Wall III) #15811
  • [c252d874d7] - test: use common/fixtures in fs-symlink test (AlexeyM) #15830
  • [07c14f3054] - test: replace common.fixtures with fixtures module (Jonathan Eskew) #15877
  • [0f23836e7b] - test: improve assert message (Tri Nguyen) #15909
  • [bbdbf8b9b0] - test: replace fixturesDir with fixtures method (suraiyah) #15894
  • [c35420d21d] - test: normalize fixtures use (Ruxandra Fediuc) #15855
  • [3c176fd6f6] - test: replace common.fixturesDir w/common.fixtures (Jason Walton) #15853
  • [77f9ef32bd] - test: switch to use common.fixtures module for fixturesDir (r1cebank) #15821
  • [71e68799ef] - test: fixturesDir replaced to fixtures module (Pawel Golda) #15809
  • [d70f9f6a35] - test: replace common.fixturesDir with fixtures (Stefania Sharp) #16015
  • [4cf84ea76e] - test: replaces common.fixturesDir usage (Ruy Adorno) #15818
  • [788d7db4e9] - test: use common.fixtures.path() (Tobias Kieslich) #16112
  • [b7865ea70d] - test: replace common.fixturesDir with fixtures (Shakeel Mohamed) #15857
  • [9b39ca6cbb] - test: use fixtures module in test (Nigel Kibodeaux) #16117
  • [5e65069289] - test: use template literals in test-string-decoder (Edward Andrew Robinson) #15884
  • [d2b74fe1e3] - test: switch to fixtures module (Christopher Sidebottom) #15880
  • [1144be09b7] - test: rewrite assert message (Martin Michaelis) #15879
  • [095df35a5e] - test: change fixturesDir to fixtures.path (Guilherme Akio Sakae) #15863
  • [4fd5bf5ff7] - test: replace fixturesDir with common.fixtures (Oliver Luebeck) #15907
  • [e3e234ea1c] - test: update http test client function signatures (Jakub Mrowiec - Alkagar) #15807
  • [08ca73f52a] - test: replace common.fixturesDir w/ fixtures.path (Druotic) #15819
  • [39ae3f1802] - test: replace fixtureDir with fixtures.path (matthewreed26) #15943
  • [1365a6f597] - test: use common.fixtures module for file path (Adil L) #16017
  • [bd8d4401ee] - test: use fixtures module (Maurice Hayward) #16034
  • [bba5263d00] - test: replace fixturesDir with fixtures module (tabulatedreams) #16036
  • [a8e7fa4e75] - test: replace fixturesDir with fixtures module (Ivan Etchart) #15893
  • [1fc3851642] - test: change fixturesDir to fixtures.path (Savio Lucena) #15902
  • [683e48cb55] - test: changed fixtures require (creisle) #15899
  • [f82f691d5e] - test: replaced fixturesDir with fixtures module (Alex McKenzie) #15908
  • [e68ef291e7] - test: use common.fixtures in tls test (Ben Michel) #15965
  • [71daa68c3d] - test: use fixtures module instead of common (Joe Grace) #15925
  • [e81fc8aca7] - test: replaced fixturesDir with fixtures module (Alex McKenzie) #15927
  • [33ea6deeab] - test: replace fixturesDir with fixtures module (Greg Matthews) #15932
  • [be2b70bb56] - test: replace fixturesDir with fixtures (Mujtaba Al-Tameemi) #15949
  • [25a5bf02c7] - test: remove common.fixturesDir (Luis Del Águila) #15950
  • [51d87e338e] - test: replace fixturesDir with fixtures module (BinarySo1o) #15961
  • [05286b6c80] - test: replaced fixturesDir with common.fixtures (jopann) #15971
  • [683c5fa58f] - test: use common.fixtures module in test-preload (Laura Cabrera) #15975
  • [000965d427] - test: replaced common.fixturesDir with readKey (Sean Cox) #15933
  • [0f8b315a9e] - test: replace fixturesDir in tls-env-bad-extra-ca (Annie Weng) #15813
  • [48a55d1364] - test: use common.fixtures in checkServerIdentity (Emily Marigold Klassen) #15951
  • [909e587a93] - test: replaced common.fixturesDir with readKey (rhalldearn) #15952
  • [544cbd7884] - test: replace fixturesDir with fixtures.readKey (Thomas Schorn) #15948
  • [4005ed619f] - test: replace common.fixturesDir with fixtures. (Sam Skjonsberg) #15802
  • [8c5b51d9c3] - test: replace fixturesDir with common.fixtures (rachelnicole) #16051
  • [107acb1c56] - test: update fixturesDir to fixtures.readKey (bitandbang) #16016
  • [643a2c6b19] - test: replace fixturesDir with common.fixtures (Pooya Paridel) #15837
  • [14aee78554] - test: update 'fixturesDir' refs in a test file (James M. Greene) #15824
  • [e1c45efdbb] - test: use fixtures.readKey in https-agent test (Greg Byram) #15913
  • [2c6aa17fa9] - test: add test for fork() + shell (cjihrig) #15352
  • [148a030345] - test: remove node-tap lookalike (cjihrig) #13707
  • [fa5c706bec] - test: refactor exitedAfterDisconnect test (Rich Trott) #16729
  • [9416dab7ac] - test: use fixtures module in test-https-pfx (Ken Takagi) #15895
  • [7e9779aade] - test: refactor test-readline-keys (Rich Trott) #11281
  • [8264328087] - test,net: remove scatological terminology (Rich Trott) #16599
  • [bb81390db2] - timers: fix eventloop block (zhangzifa) #15072
  • [f3749d7b2c] - tools: remove unneeded parentheses in doc/html.js (Vse Mozhet Byt) #16845
  • [1c192f50f6] - tools: replace string concatenation with template literals (Kevin Yu) #16804
  • [ce007be05b] - tools: replace string concatenation with template literals (Giovanni Lela) #16806
  • [d165d3fd1c] - tools: replace string concetation with templates (Patrick Heneise) #16801
  • [a8d7f5f52e] - tools: fix cpplint.py when path contains non-ascii (sharkfisher) #16047
  • [b48471ac10] - tools: rename unused variale in more pythonic way (Nikhil Komawar) #16171
  • [5b5b5c0f15] - tools: use template literal in error message (Tim Chon) #15846
  • [ae5930bbe4] - tty,doc: add type-check to isatty (Bryan English) #15567

addaleax and others added 3 commits November 14, 2017 13:57
Provide shortcut `node::CheckedMalloc()` and friends that
replace `node::Malloc()` + `CHECK_NE(·, nullptr);` combinations
in a few places.

Backport-PR-URL: #16587
PR-URL: #8482
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: Ilkka Myller <[email protected]>
Call `v8::Isolate::GetCurrent()->LowMemoryNotification()` when
an allocation fails to give V8 a chance to clean up and return
memory before retrying (and possibly giving up).

Backport-PR-URL: #16587
PR-URL: #8482
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: Ilkka Myller <[email protected]>
This commit removes the small node-tap lookalike from several
of the streams2 tests. It's only used by six tests, and is
inconsistent with all other tests.

Backport-PR-URL: #17024
PR-URL: #13707
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@nodejs-github-bot nodejs-github-bot added 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. v6.x labels Nov 21, 2017
MylesBorins added a commit that referenced this pull request Nov 21, 2017
Notable Changes:

Coming Soon

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

Looks like we have a legit failure for coffee-script 😢

MylesBorins added a commit that referenced this pull request Nov 21, 2017
Notable Changes:

Coming Soon

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

MylesBorins commented Nov 21, 2017

I've backed out the commit breaking coffeescript, it was a backport explicitly to v6.x

CI: https://ci.nodejs.org/job/node-test-pull-request/11604/
citgm: https://ci.nodejs.org/view/Node.js-citgm/job/citgm-smoker/1102/
rc.2: https://nodejs.org/download/rc/v6.12.1-rc.2/

@bnoordhuis
Copy link
Member

I've backed out the commit breaking coffeescript, it was a backport explicitly to v6.x

What PR/commit was it?

@MylesBorins
Copy link
Contributor Author

#15773

shigeki and others added 14 commits November 28, 2017 13:09
`crypto.createCipher()` sets the fixed IV derived from password and it
leads to a security risk of nonce reuse when counter mode is used.
A warning is emitted when CTR, GCM or CCM is used in
`crypto.createCipher()` to notify users to avoid nonce reuse.

Backport-PR-URL: #16583
Fixes: #13801
PR-URL: #13821
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Fedor Indutny <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Tobias Nießen <[email protected]>
EVP_CIPHER_CTX_FLAG_WRAP_ALLOW flag needs to be set in using wrap mode
ciphers. In `crypto.createCipher()`, AES key wrap mode does not use a
default IV defined in RFC3394 but a generated IV with
`EVP_BytesToKey()` to be consistent API behaviors with other ciphers.

The built-in AES wrap mode in OpenSSL is not supported in FIPS mode as
http://openssl.6102.n7.nabble.com/AES-Key-Wrap-in-FIPS-Mode-td50238.html
so its tests in FIPS mode are skipped.

Backport-PR-URL: #16584
Fixes: #15009
PR-URL: #15037
Reviewed-By: Fedor Indutny <[email protected]>
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: James M Snell <[email protected]>
This exception can logically never happen because of the key stretching
that takes place first.  Failure must therefore be a bug in Node.js and
not in the executing script.

Backport-PR-URL: #16585
PR-URL: #15183
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: James M Snell <[email protected]>
V8 options allow either '_' or '-' to be used in options as a seperator,
such as "--abort-on_uncaught-exception". Allow these case variations
when used with NODE_OPTIONS.

PR-URL: #14093
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Refael Ackermann <[email protected]>
Reviewed-By: James M Snell <[email protected]>
PR-URL: #15399
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: James M Snell <[email protected]>
This commit ensures that spawn()'s shell option is unconditionally
set to false when fork() is called.

Refs: #15299
Fixes: #13983
PR-URL: #15352
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
This commit verifies that the child_process fork() method does
not honor the shell option.

Refs: #15299
PR-URL: #15352
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
PR-URL: #15494
Fixes: #15484
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
PR-URL: #15621
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Yuta Hiroto <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Khaidi Chu <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
PR-URL: #15609
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Daniel Bevenius <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Yuta Hiroto <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
PR-URL: #15913
Reviewed-By: Gibson Fahnestock <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
This was a task from Code & Learn at NINA17.

PR-URL: #15824
Reviewed-By: Ryan Graham <[email protected]>
Reviewed-By: Refael Ackermann <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Tobias Nießen <[email protected]>
PR-URL: #15837
Reviewed-By: Gibson Fahnestock <[email protected]>
Reviewed-By: Refael Ackermann <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
PR-URL: #16016
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Refael Ackermann <[email protected]>
imkabir and others added 20 commits November 28, 2017 13:11
PR-URL: #16916
Reviewed-By: Vse Mozhet Byt <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Daniel Bevenius <[email protected]>
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Franziska Hinkelmann <[email protected]>
PR-URL: #16919
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: Franziska Hinkelmann <[email protected]>
PR-URL: #16912
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: Vse Mozhet Byt <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Franziska Hinkelmann <[email protected]>
PR-URL: #16929
Reviewed-By: Vse Mozhet Byt <[email protected]>
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Anatoli Papirovski <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: Franziska Hinkelmann <[email protected]>
Include value that cause failure in error message in
test-cluster-master-kill.js.

PR-URL: #16826
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Replace `common.fixturesDir` with `fixtures.path()` usage in
test/pummel/test-hash-seed.js.

PR-URL: #16823
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Based on feedback during a recent collaborator onboarding, move the
metadata description closer to where the instructions for editing a
commit message are.

Indicate which metadata are required and which are optional.

Make the punctuation more consistent. Previously, sentences prior to
instructions ended in a period, a colon, or no punctuation at all. Colon
seems best, so changed the Technical How-To section to use colons.

PR-URL: #15710
Reviewed-By: Vse Mozhet Byt <[email protected]>
Reviewed-By: Benedikt Meurer <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Refael Ackermann <[email protected]>
PR-URL: #16833
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Vse Mozhet Byt <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
In test-fs-realpath-on-substed-drive, require common/fixtures
module and swapped the location of fixturesDir
from common to fixtures module.

PR-URL: #16813
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
PR-URL: #16978
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Khaidi Chu <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Anatoli Papirovski <[email protected]>
Reviewed-By: James M Snell <[email protected]>
* remove comment that isn't relevant/important
* add comment that explains what the test does

PR-URL: #16829
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: James M Snell <[email protected]>
PR-URL: #16936
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Refael Ackermann <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
The `dev_t` is unsigned on Linux, use Integer::NewFromUnsigned to fix cast overflow

PR-URL: #16705
Fixes: #16496
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Khaidi Chu <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
PR-URL: #16705
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Khaidi Chu <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Currently when running the test without an internet connection there are
two JavaScript test failures

This commit moves the two JavaScript tests to test/internet.

Backport-PR-URL: #17171
PR-URL: #16255
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: James M Snell <[email protected]>
This commit excludes src\tracing\trace_event.h and
src\tracing\trace_event_common.h from the linter but allows the
rest of the files in src\tracing to be examined by the linter
which is similar to what the Makefile does.

Refs: #16720
Backport-PR-URL: #17172
PR-URL: #16720
Reviewed-By: James M Snell <[email protected]>
I cannot find any usages of these includes and think they can be
removed.

Backport-PR-URL: #17092
PR-URL: #16179
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Original commit message:

    [turbofan] Fix missing lazy deopt in object literals.

    This adds a missing lazy bailout point when defining data properties
    with computed property names in object literals. The runtime call to
    Runtime::kDefineDataPropertyInLiteral can trigger deopts. The necessary
    bailout ID already exists and is now properly used.

    [email protected]
    TEST=mjsunit/regress/regress-crbug-621816
    BUG=chromium:621816

    Review-Url: https://codereview.chromium.org/2099133003
    Cr-Commit-Position: refs/heads/master@{#37294}

Refs: v8/v8@4af8029
PR-URL: #17290
Fixes: #14326
Reviewed-By: Franziska Hinkelmann <[email protected]>
Reviewed-By: Ben Noordhuis <[email protected]>
cctest has `so.59` extension when building node shared library in linux.
The appending is defined in node.gypi and the cctest target in node.gyp
includes node.gypi. Moving the appending from node.gypi to node target
in node.gyp fixes the issue.

Signed-off-by: Yihong Wang <[email protected]>

Backport-PR-URL: #17255
PR-URL: #16680
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Gibson Fahnestock <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: Daniel Bevenius <[email protected]>
Reviewed-By: Refael Ackermann <[email protected]>
Notable Changes:

* build:
  - fix npm install with --shared (Ben Noordhuis)
    #16438
* build:
  - building with python 3 is now supported (Emily Marigold Klassen)
    #16058
* src:
  - v8 options can be specified with either '\_' or '-' in NODE_OPTIONS
    (Sam Roberts) #14093

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

@MylesBorins MylesBorins merged commit 4a99c2b into v6.x Dec 5, 2017
MylesBorins added a commit that referenced this pull request Dec 5, 2017
MylesBorins added a commit that referenced this pull request Dec 5, 2017
Notable Changes:

* build:
  - fix npm install with --shared (Ben Noordhuis)
    #16438
* build:
  - building with python 3 is now supported (Emily Marigold Klassen)
    #16058
* src:
  - v8 options can be specified with either '\_' or '-' in NODE_OPTIONS
    (Sam Roberts) #14093

PR-URL: #17180
@gibfahn gibfahn deleted the v6.12.1-proposal branch December 5, 2017 17:53
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.