summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
* build: remove incorrect argument in vcbuild.batJeremiah Senkpiel2015-03-181-1/+1
| | | | | | | This change had been incorrectly committed in f19e9b6 PR-URL: https://github.com/iojs/io.js/pull/1198 Reviewed-By: Rod Vagg <rod@vagg.org>
* lib: don't error in repl when cwd doesn't existBen Noordhuis2015-03-192-2/+36
| | | | | | | | | | | The current working directory may not exist when the REPL starts up. Don't treat that as an error because it's still possible to do many useful things. This is like the previous commit but for the REPL. Fixes: https://github.com/iojs/io.js/issues/1184 PR-URL: https://github.com/iojs/io.js/pull/1194 Reviewed-By: Johan Bergström <bugs@bergstroem.nu> Reviewed-By: Rod Vagg <rod@vagg.org>
* src: don't error at startup when cwd doesn't existBen Noordhuis2015-03-193-25/+35
| | | | | | | | | | | | | | | The current working directory may not exist when iojs starts up. Don't treat that as an error because it's still possible to do many useful things, like evaluating a command line script or starting a REPL. This commit also fixes an age-old Windows bug where process.argv[0] was not properly expanded, that's why the parallel/test-process-argv-0 test gets an update as well. Fixes: https://github.com/iojs/io.js/issues/1184 PR-URL: https://github.com/iojs/io.js/pull/1194 Reviewed-By: Johan Bergström <bugs@bergstroem.nu> Reviewed-By: Rod Vagg <rod@vagg.org>
* test: Introduce knowledge of FreeBSD jailsJohan Bergström2015-03-197-8/+43
| | | | | | | | | | | | | | | FreeBSD jails act differently than your average vm or similar application container. All routing passes through one ip address, which makes things like localhost or 0.0.0.0 resolve differently. Introduce a helper that allows us to verify if we're in a jail and another one for returning an ip address for localhost. Also, skip one test instead of trading additional complexity in common.js for one specific user scenario. PR-URL: https://github.com/iojs/io.js/pull/1167 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
* src: fix crypto bio integer wraparound on 32 bitsBen Noordhuis2015-03-181-2/+4
| | | | | | | | | | | | | | | | Fix a bug where a size_t was negated and passed to a function that takes an int64_t. It works by accident when sizeof(size_t) == sizeof(int64_t) but it causes the value to underflow when size_t is a 32 bits type. v8::Isolate::AdjustAmountOfExternalAllocatedMemory() is the function I'm talking about. The goal of that call is to tell V8 that some memory has been freed but due to that underflow, we were actually reporting that we had just allocated gigabytes of memory. It set off a garbage collector frenzy and essentially brought the VM to a standstill. Fixes: https://github.com/iojs/io.js/issues/1188 PR-URL: https://github.com/iojs/io.js/pull/1192 Reviewed-By: Fedor Indutny <fedor@indutny.com>
* doc: add yosuke-furukawa as collaboratorYosuke Furukawa2015-03-181-0/+1
| | | | | | | PR-URL: https://github.com/iojs/io.js/pull/1183 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Rod Vagg <rod@vagg.org> Reviewed-By: Shigeki Ohtsu <ohtsu@iij.ad.jp>
* doc: update test section in CONTRIBUTING.mdBen Noordhuis2015-03-171-3/+3
| | | | | | | | test/simple no longer exists, tell contributors to add their tests to test/parallel. PR-URL: https://github.com/iojs/io.js/pull/1181 Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
* doc: add petkaantonov as collaboratorPetka Antonov2015-03-171-0/+1
| | | | | | PR-URL: https://github.com/iojs/io.js/pull/1179 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: cjihrig <cjihrig@gmail.com>
* doc: add silverwind as collaboratorRoman Reiss2015-03-171-0/+1
| | | | | PR-URL: https://github.com/iojs/io.js/pull/1176 Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
* doc: add jbergstroem as collaboratorJohan Bergström2015-03-181-0/+1
| | | | | PR-URL: https://github.com/iojs/io.js/pull/1175 Reviewed-By: Chris Dickinson <christopher.s.dickinson@gmail.com>
* configure: use gcc and g++ as CC and CXX defaultsBen Noordhuis2015-03-171-2/+3
| | | | | | | | | It matches what GYP's Makefile generator does and it should improve compiler detection because cc and c++ are not always gcc and g++. Fixes: https://github.com/iojs/io.js/issues/1173 PR-URL: https://github.com/iojs/io.js/pull/1174 Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
* doc: fix typo in buffer module documentationAlex Yursha2015-03-171-1/+1
| | | | | PR-URL: https://github.com/iojs/io.js/pull/1169 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
* benchmark: add output format option [csv]Brendan Ashworth2015-03-161-6/+29
| | | | | | | | | | | | | | | | | This commit adds an `OUTPUT_FORMAT` environment variable option for all benchmark tests that allow either 'csv' or 'default' output. Default output has been left unchanged, and csv output prints out the csv headers along with the csv formatted per-test output, each test also seperated by a newline. It can be used like the following: $ OUTPUT_FORMAT=csv iojs benchmark/common.js http Not specifying the OUTPUT_FORMAT env var will default to 'default'. Specifying a bad value will throw an error. PR-URL: https://github.com/iojs/io.js/pull/777 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
* benchmark: add plot_csv R graphing scriptBrendan Ashworth2015-03-161-0/+38
| | | | | | | | | | | | | | | | | | | | | This commit adds a graphing script (in R) for graphing the CSV output of a benchmark. It can be run like this: ``` $ OUTPUT_FORMAT=csv iojs benchmark/http/client-request-body.js > data.csv $ ./benchmark/plot_csv.R data.csv graph.png bytes type ``` This will graph the output to `graph.png`, using the output's `bytes` value as X and the result value for each as Y. Output will be grouped by `type`. Running as the example yields a beautiful graph like this: http://pbrd.co/1vBhUfy. PR-URL: https://github.com/iojs/io.js/pull/777 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
* v8: fix --max_old_space_size=4096 integer overflowBen Noordhuis2015-03-171-4/+5
| | | | | | | | See https://code.google.com/p/v8/issues/detail?id=3857 for the bug report and https://codereview.chromium.org/897543002 for the CL. PR-URL: https://github.com/iojs/io.js/pull/1166 Reviewed-By: Fedor Indutny <fedor@indutny.com>
* http: add flushHeaders and deprecate flushYosuke Furukawa2015-03-173-3/+27
| | | | | | | | PR-URL: https://github.com/iojs/io.js/pull/1156 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Christian Tellnes <christian@tellnes.no> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
* make: remove node_dtrace from cpplint excludesJulien Gilli2015-03-162-6/+5
| | | | | PR-URL: https://github.com/joyent/node/pull/8741 Reviewed-By: Fedor Indutny <fedor@indutny.com>
* net: use cached peername to resolve remote fieldsJames Hartig2015-03-162-3/+16
| | | | | | | | | Allows socket.remote* properties to still be accessed even after the socket is closed. Fixes: https://github.com/joyent/node/issues/9287 PR-URL: https://github.com/joyent/node/pull/9366 Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
* doc: fix '\\' typos on WindowsSteven Vercruysse2015-03-161-3/+3
| | | | | | | | This commit changes the Windows examples in path.markdown to correctly display '\\'. PR-URL: https://github.com/joyent/node/pull/9412 Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
* build: allow custom PackageMaker pathJulien Gilli2015-03-161-2/+2
| | | | | | | | Make PACKAGEMAKER customizable because PackageMaker is not necessarily installed in /Developer on OSX anymore. PR-URL: https://github.com/joyent/node/pull/9377 Reviewed-By: Timothy J Fontaine <tjfontaine@gmail.com>
* lib: remove broken NODE_MODULE_CONTEXTS featureBen Noordhuis2015-03-164-50/+9
| | | | | | | | | | | This feature has no tests and has been broken for ages, see for example https://github.com/iojs/io.js/pull/1160. Don't bother fixing it, it's pretty much broken by design and there can't be too many users because it's almost undocumented. A quick Google search suggests that it causes more grief than joy to the few that do use it. Remove it. PR-URL: https://github.com/iojs/io.js/pull/1162 Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
* src: use Number::New() for heapTotal/heapUsedBen Noordhuis2015-03-161-4/+4
| | | | | | | | | | | | With --max_old_space_size=12345 it's possible to create a JS heap that is larger than what fits in an unsigned int so use Number::New() rather than Integer::NewFromUnsigned(). Performance-wise, it doesn't matter much. If V8 can fit the double in a SMI, it will. PR-URL: https://github.com/iojs/io.js/pull/1148 Reviewed-By: Trevor Norris <trev.norris@gmail.com>
* src: don't create js string twice on errorBen Noordhuis2015-03-161-14/+15
| | | | | | | | Rewrite ErrnoException() so that it doesn't turn the file path into a string twice. PR-URL: https://github.com/iojs/io.js/pull/1148 Reviewed-By: Trevor Norris <trev.norris@gmail.com>
* path: add type checking for path inputscjihrig2015-03-163-30/+81
| | | | | | | | | | | | This commit adds type checking of path inputs to exported methods in the path module. The exception is _makeLong(), which seems to explicitly support any data type. Fixes: https://github.com/iojs/io.js/issues/1139 PR-URL: https://github.com/iojs/io.js/pull/1153 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com> Reviewed-By: Brendan Ashworth <brendan.ashworth@me.com>
* doc: reflect new require('events') behaviourAlex Yursha2015-03-161-1/+5
| | | | | | | | We don't need to do `require('events').EventEmitter` any longer. PR-URL: https://github.com/iojs/io.js/pull/975 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Brendan Ashworth <brendan.ashworth@me.com>
* querystring: optimize parse and stringifyBrian White2015-03-151-35/+90
| | | | | | | | | | | | | | | | | | | | parse optimizations: * Move try-catch to separate function to keep entire function from being deoptimized. * Use key array lookup instead of using hasOwnProperty. * Avoid decoding known empty strings. * Avoid possibly unnecessary switch to slower decoder for values if key decoding throws. stringify optimizations: * Use manual loop for default encoder instead of encodeURIComponent. * Use string concatenation instead of joining an array of strings. * Avoid caching result of typeof. PR-URL: https://github.com/iojs/io.js/pull/847 Reviewed-By: Trevor Norris <trev.norris@gmail.com>
* deps: make node-gyp work with io.jscjihrig2015-03-153-25/+25
| | | | | | | | | | | | Every npm version bump requires a few patches to be floated on node-gyp for io.js compatibility. These patches are found in 03d199276e21c1fa08d8df14eeb654c90cc5aa20, 5de334c23096492014a097ff487f07ad8eaee6d2, and da730c76e98fb9fd18dac445dafbbec74d79f802. This commit squashes them into a single commit. PR-URL: https://github.com/iojs/io.js/pull/990 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
* deps: upgrade npm to 2.7.1Forrest L Norvell2015-03-15209-681/+6297
| | | | | | PR-URL: https://github.com/iojs/io.js/pull/1142 Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com> Reviewed-By: Rod Vagg <rod@vagg.org>
* tls: re-enable `.writev()` on TLSWrapFedor Indutny2015-03-141-1/+1
| | | | | | | | | | | | Fix the `parallel/test-tls-over-http-tunnel.js` on Windows by re-enabling the accidentally disabled `.writev()` method on TLSWrap. It appears that there is some subtle issue with shutdown timing and it manifests itself when the chunks are written in separate packets. This leads to concurrent `shutdown`/`destroy`, which breaks the test. PR-URL: https://github.com/iojs/io.js/pull/1155 Reviewed-By: Bert Belder <bertbelder@gmail.com>
* tls: fix leak on `DoWrite()` errorsFedor Indutny2015-03-141-2/+4
| | | | | | | | It is very unlikely to happen, but still the write request should be disposed in case of immediate failure. PR-URL: https://github.com/iojs/io.js/pull/1154 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
* src: revert -r/--require flagsChris Dickinson2015-03-137-194/+65
| | | | | | | | | This reverts commit 7bde3f1a8f53d82992a8fed73e5f93159bb400b3. The added test (test/parallel/test-preload.js) fails on Windows. PR-URL: https://github.com/iojs/io.js/pull/1150 Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
* doc: fix vm module examplesFangDun Cai2015-03-131-2/+4
| | | | | PR-URL: https://github.com/iojs/io.js/pull/1147 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
* src: add -r/--require flags for preloading modulesAli Ijaz Sheikh2015-03-137-65/+194
| | | | | | | | | | | | | | | -r/--require can be used to preload modules on node startup. The option takes a single module name. The option can be repeated as necessary to preload multiple modules. This patch allows 'vendors' (such a cloud host) to inject functionality that gets executed at application startup without requiring an explicit require from the user's application. This can be useful to load vendor specific application monitoring APIs transparently. PR-URL: https://github.com/iojs/io.js/pull/881 Reviewed-By: Trevor Norris <trev.norris@gmail.com> Reviewed-By: Chris Dickinson <christopher.s.dickinson@gmail.com>
* test: fix test-http-content-lengthJeremiah Senkpiel2015-03-131-1/+1
| | | | | | | | | Previously the test did not allow the last request to complete. Fixes: https://github.com/iojs/io.js/pull/1137 PR-URL: https://github.com/iojs/io.js/pull/1145 Reviewed-By: Brian White <mscdex@mscdex.net> Reviewed-By: Bert Belder <bertbelder@gmail.com>
* crypto: add deprecated ValiCert CA for cross certShigeki Ohtsu2015-03-132-0/+26
| | | | | | | | | | | | | | The host of melissadata.net has a cross root certification between Starfield Class 2 and ValiCert Class 2. OpenSSL-1.0.1 only looks up a cert chain to the deprecated ValiCert Class 2 CA and causes untrusted error. We add it for a short-term remedy and it is to be removed after upgrading OpenSSSL-1.0.2 and applying private patches to support alternative cert chains. See #402 and #589. Fixes: https://github.com/iojs/io.js/issues/923 PR-URL: https://github.com/iojs/io.js/pull/1135 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
* test: fix ext commands to be double quotedShigeki Ohtsu2015-03-1210-17/+16
| | | | | | | | | Paths used on the Windows command line need to be enclosed in double quotes, or they'll be parsed incorrectly when there are spaces in the path. PR-URL: https://github.com/iojs/io.js/pull/1122 Reviewed-by: Bert Belder <bertbelder@gmail.com>
* test: add test for reading a large file through a pipeSantiago Gimeno2015-03-122-0/+79
| | | | | PR-URL: https://github.com/iojs/io.js/pull/1074 Reviewed-By: Bert Belder <bertbelder@gmail.com>
* fs: use stat.st_size only to read regular filesSantiago Gimeno2015-03-121-2/+4
| | | | | | | | Using st_size to read non-regular files can lead to not reading all the data. PR-URL: https://github.com/iojs/io.js/pull/1074 Reviewed-By: Bert Belder <bertbelder@gmail.com>
* test: fix readfile-zero-byte-liar testSantiago Gimeno2015-03-121-1/+1
| | | | | PR-URL: https://github.com/iojs/io.js/pull/1074 Reviewed-By: Bert Belder <bertbelder@gmail.com>
* src: do not leak handles on debug and exitFedor Indutny2015-03-121-1/+2
| | | | | | | | | | Ensure HandleScope is created before creating any new handles (which `Context::Scope` and `Environment::GetCurrent` does). PR-URL: https://github.com/iojs/io.js/pull/1133 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Rod Vagg <rod@vagg.org> Reviewed-By: Brian White <mscdex@mscdex.net>
* v8: fix build on solaris platformsJohan Bergström2015-03-121-0/+2
| | | | | | | | | | | `v8/3c7e4403` introduced a different cast which broke building on Illumos. Revert to previous behavior for V8_OS_SOLARIS. Found on SmartOS while building with gcc 4.9.0. V8-Issue: https://code.google.com/p/v8/issues/detail?id=3935 V8-Patch: https://codereview.chromium.org/990063002 PR-URL: https://github.com/iojs/io.js/pull/1079 Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
* build: fix incorrect set in vcbuild.batBert Belder2015-03-111-1/+1
| | | | Thanks Shigeki Ohtsu for catching this.
* buffer: align chunks on 8-byte boundaryFedor Indutny2015-03-111-0/+6
| | | | | | | | | | | | | When slicing global pool - ensure that the underlying buffer's data ptr is 8-byte alignment to do not ruin expectations of 3rd party C++ addons. NOTE: 0.10 node.js always returned aligned pointers and io.js should do this too for compatibility. PR-URL: https://github.com/iojs/io.js/pull/1126 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Trevor Norris <trev.norris@gmail.com> Reviewed-by: Bert Belder <bertbelder@gmail.com>
* doc: make tools/update-authors.sh cross-platformBen Noordhuis2015-03-113-18/+27
| | | | | | | | | | | | And by cross-platform I mean Linux and OS X. The awk script is not compatible with BSD awk, that's why this commit changes it to perl. Update the .mailmap to remove some duplicates and regenerate the AUTHORS file. Fixes: https://github.com/iojs/io.js/issues/1120 PR-URL: https://github.com/iojs/io.js/pull/1121 Reviewed-By: Rod Vagg <rod@vagg.org>
* https: don't overwrite servername optionskenqbx2015-03-102-5/+47
| | | | | | PR-URL: https://github.com/iojs/io.js/pull/1110 Reviewed-By: Fedor Indutny <fedor@indutny.com> Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
* doc: add Malte-Thorben Bruns to .mailmapBen Noordhuis2015-03-101-0/+2
| | | | | PR-URL: https://github.com/iojs/io.js/pull/1118 Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
* lib: allow server.listen({ port: "1234" })Ben Noordhuis2015-03-102-13/+51
| | | | | | | | | | net.connect() accepts `{ port: "1234" }` (i.e. a string) as of commit 9d2b89d06 ("net: allow port 0 in connect()") but net.Server#listen() did not, creating a minor inconsistency. This commit rectifies that. Fixes: https://github.com/iojs/io.js/issues/1111 PR-URL: https://github.com/iojs/io.js/pull/1116 Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
* doc: move checkServerIdentity option to tls.connect()skenqbx2015-03-101-4/+4
| | | | | PR-URL: https://github.com/iojs/io.js/pull/1107 Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
* doc: fix missing periods in url.markdownRyuichi Okumura2015-03-101-4/+4
| | | | | PR-URL: https://github.com/iojs/io.js/pull/1115 Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
* tls_wrap: proxy handle methods in prototypeFedor Indutny2015-03-092-10/+13
| | | | | | | | | | Set proxied methods wrappers in `TLSWrap` prototype instead of doing it on every socket allocation. Should speed up things a bit and will certainly make heapsnapshot less verbose. PR-URL: https://github.com/iojs/io.js/pull/1108 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Trevor Norris <trev.norris@gmail.com>
View Lorry Controller Status