- Feb 04, 2015
-
-
Rod Vagg authored
Also added binary download documentation to the README.md and GPG release key fingerprint for @rvagg. PR-URL: https://github.com/iojs/io.js/pull/681 Reviewed-By:
Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By:
Fedor Indutny <fedor@indutny.com>
-
Fedor Indutny authored
Fix: iojs/io.js#461 PR-URL: https://github.com/iojs/io.js/pull/706 Reviewed-By:
Ben Noordhuis <info@bnoordhuis.nl>
-
Fedor Indutny authored
Fix: iojs/io.js#461 PR-URL: https://github.com/iojs/io.js/pull/706 Reviewed-By:
Ben Noordhuis <info@bnoordhuis.nl>
-
Fedor Indutny authored
Fix: iojs/io.js#461 PR-URL: https://github.com/iojs/io.js/pull/706 Reviewed-By:
Ben Noordhuis <info@bnoordhuis.nl>
-
- Feb 03, 2015
-
-
Glen Keane authored
Fixes Windows compile failures introduced in d75fecf6 PR-URL: https://github.com/iojs/io.js/pull/703 Reviewed-By:
Rod Vagg <rod@vagg.org>
-
Jonathan Ong authored
PR-URL: https://github.com/iojs/io.js/pull/634 Reviewed-BY:
Nicu Micleușanu <micnic90@gmail.com> Reviewed-By:
Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By:
Stephen Belanger <admin@stephenbelanger.com>
-
Brendan Ashworth authored
This commit cleans up `benchmark/common.js` with a few generic changes such as the following: - declare all `require()`'d libraries at the top instead of in the middle - add some empty whitespace where it helps readability - changes ambiguous variable names - standardizes most if / else blocks - missing semicolons PR-URL: https://github.com/iojs/io.js/pull/662 Reviewed-By:
Nicu Micleușanu <micnic90@gmail.com> Reviewed-By:
Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By:
Stephen Belanger <admin@stephenbelanger.com>
-
Calvin Metcalf authored
Private keys may be used along with publicEncrypt since the private key includes the public one. This adds the ability to use encrypted private keys which previously threw an error. This commit also makes sure the user exposed functions have names. PR-URL: https://github.com/iojs/io.js/pull/626 Reviewed-By:
Ben Noordhuis <info@bnoordhuis.nl>
-
Sam Roberts authored
When the last signal listener is removed, the signal wrap should be closed, restoring the default signal handling behaviour. This is done in a (patched) process.removeListener(). However, events.removeAllListeners has an optimization to avoid calling removeListener() if there are no listeners for the 'removeListener' event, introduced in 56668f54. That caused the following code to fail to terminate: process.stdin.resume(); function listener() {}; process.on('SIGINT', listener); process.removeAllListeners('SIGINT'); process.kill(process.pid, 'SIGINT') while the following will terminate: process.stdin.resume(); function listener() {}; process.on('SIGINT', listener); process.removeListener('SIGINT', listener); process.kill(process.pid, 'SIGINT') Replace the method patching with use of the 'newListener' and 'removeListener' events, which will fire no matter which methods are used to add or remove listeners. PR-URL: https://github.com/iojs/io.js/pull/687 Reviewed-By:
Ben Noordhuis <info@bnoordhuis.nl>
-
Sam Roberts authored
The order of the `newListener` and `removeListener` events with respect to the actual adding and removing from the underlying listeners array should be deterministic. There is no compelling reason for leaving it indeterminate. Changing the ordering is likely to result in breaking code that was unwittingly relying on the current behaviour, and the indeterminancy makes it impossible to use these events to determine when the first or last listener is added for an event. PR-URL: https://github.com/iojs/io.js/pull/687 Reviewed-By:
Ben Noordhuis <info@bnoordhuis.nl>
-
Glen Keane authored
Removed DTRACE_NET_SOCKET_READ and DTRACE_NET_SOCKET_WRITE as they were never called from within the source code. PR-URL: https://github.com/iojs/io.js/pull/694 Reviewed-By:
Ben Noordhuis <info@bnoordhuis.nl>
-
Yosuke Furukawa authored
PR-URL: https://github.com/iojs/io.js/pull/609 Reviewed-By:
Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By:
Evan Lucas <evanlucas@me.com>
-
Yosuke Furukawa authored
PR-URL: https://github.com/iojs/io.js/pull/609 Reviewed-By:
Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By:
Evan Lucas <evanlucas@me.com>
-
- Feb 01, 2015
-
-
cjihrig authored
Many of the util.is*() methods used to check data types simply compare against a single value or the result of typeof. This commit replaces calls to these methods with equivalent checks. This commit does not touch calls to the more complex methods (isRegExp(), isDate(), etc.). Fixes: https://github.com/iojs/io.js/issues/607 PR-URL: https://github.com/iojs/io.js/pull/647 Reviewed-By:
Ben Noordhuis <info@bnoordhuis.nl>
-
Bert Belder authored
* Fetch from the correct url. * Link compiled addons with iojs.lib instead of node.lib. * Disable checksum checks for iojs.lib until our website supports them. PR: https://github.com/iojs/io.js/pull/422 Reviewed-by:
Ben Noordhuis <info@bnoordhuis.nl> Reviewed-by:
Rod Vagg <rod@vagg.org>
-
Ben Noordhuis authored
Apply a small patch that makes node-gyp fetch the tarballs from https://iojs.org/ instead of http://nodejs.org/ A patch better suited for inclusion upstream will be put together shortly. PR-URL: https://github.com/iojs/io.js/pull/343 Reviewed-By:
Rod Vagg <rod@vagg.org>
-
Forrest L Norvell authored
PR-URL: https://github.com/iojs/io.js/pull/663 Reviewed-By:
Ben Noordhuis <info@bnoordhuis.nl>
-
- Jan 31, 2015
-
-
cjihrig authored
Currently, JSON.stringify() is used to create error messages on failed assertions. This causes an error when stringifying objects with circular references. This commit switches out JSON.stringify() for util.inspect(), which can handle circular references. PR-URL: https://github.com/iojs/io.js/pull/668 Reviewed-By:
Julien Gilli <julien.gilli@joyent.com> Reviewed-By:
Bert Belder <bertbelder@gmail.com> Reviewed-By:
Chris Dickinson <christopher.s.dickinson@gmail.com>
-
Bert Belder authored
* Include a description for the error message * For rename, link, and symlink, include both the source and destination path in the error message. * Expose the destination path as the `dest` property on the error object. * Fix a bug where `ThrowUVException()` would incorrectly delegate to `Environment::TrowErrnoException()`. API impact: * Adds an extra overload for node::UVException() which takes 6 arguments. PR: https://github.com/iojs/io.js/pull/675 Fixes: https://github.com/iojs/io.js/issues/207 Closes: https://github.com/iojs/io.js/pull/293 Reviewed-by:
Ben Noordhuis <info@bnoordhuis.nl>
-
Ben Noordhuis authored
A number -> uint32 type coercion bug made buffer sizes larger than kMaxLength (0x3fffffff) wrap around. Instead of rejecting the requested size with an exception, the constructor created a buffer with the wrong size. PR-URL: https://github.com/iojs/io.js/pull/657 Reviewed-By:
Trevor Norris <trev.norris@gmail.com>
-
Sam Roberts authored
Server sockets should be shared by default, and client sockets should be exclusive by default. For net/TCP, this is how it is, for dgram/UDP, its a little less clear what a client socket is, but a socket that is auto-bound during a dgram.send() is not usefully shared among cluster workers, any more than an outgoing TCP connection would be usefully shared. Since implicit binds become exclusive, implicit/client dgram sockets can now be used with cluster on Windows. Before, neither explicit nor implicitly bound sockets could be used, causing dgram to be completely unsupported with cluster on Windows. After this change, they become half supported. PR: https://github.com/iojs/io.js/pull/325 PR: https://github.com/joyent/node/pull/8643 Reviewed-by:
Ben Noordhuis <info@bnoordhuis.nl> Reviewed-by:
Bert Belder <bertbelder@gmail.com>
-
Brendan Ashworth authored
This commit removes the benchmark spacing modification in `client-request-body.js` and `end-vs-write-end.js` which adds two spaces to the end of some variables to make sure the lines line up. The reason behind this is that its totally pointless (the lines don't actually line up with it) and it disallows you to parse the output with a tool like awk, or at least makes it a lot harder. PR-URL: https://github.com/iojs/io.js/pull/650 Reviewed-By:
Colin Ihrig <cjihrig@gmail.com> Reviewed-By:
Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By:
Jeremiah Senkpiel <fishrock123@rocketmail.com>
-
Fedor Indutny authored
Improves `crypto.createHash().update().digest()` performance by 10%. PR-URL: https://github.com/iojs/io.js/pull/670 Reviewed-By:
Ben Noordhuis <info@bnoordhuis.nl>
-
- Jan 30, 2015
-
-
Fedor Indutny authored
Boosts speed up to 10% on primitive `createHash().update().digest()` benchmark. PR-URL: https://github.com/iojs/io.js/pull/664 Reviewed-By:
Ben Noordhuis <info@bnoordhuis.nl>
-
Fedor Indutny authored
PR-URL: https://github.com/iojs/io.js/pull/664 Reviewed-By:
Ben Noordhuis <info@bnoordhuis.nl>
-
Fedor Indutny authored
Handle most popular cases in a trie-style, branching on a first character. Remove useless HandleScope which was only eating time without producing any value. PR-URL: https://github.com/iojs/io.js/pull/664 Reviewed-By:
Ben Noordhuis <info@bnoordhuis.nl>
-
Calvin Metcalf authored
Adds a note that the default padding for publicDecrypt/privateEncrypt is RSA_PKCS1_PADDING instead of RSA_PKCS1_OAEP_PADDING as it is for privateDecrypt/publicEncrypt. PR-URL: https://github.com/iojs/io.js/pull/659 Reviewed-By:
Brendan Ashworth <brendan.ashworth@me.com>
-
Qasim Zaidi authored
PR-URL: https://github.com/iojs/io.js/pull/480 Reviewed-By:
Stephen Belanger <admin@stephenbelanger.com>
-
Ben Noordhuis authored
PR-URL: https://github.com/iojs/io.js/pull/656 Reviewed-By:
Fedor Indutny <fedor.indutny@gmail.com>
-
- Jan 29, 2015
-
-
Micleusanu Nicu authored
PR-URL: https://github.com/iojs/io.js/pull/651 Reviewed-By:
Brendan Ashworth <brendan.ashworth@me.com>
-
Fishrock123 authored
Adds info on the `wrk` prerequisite for http benchmarks and how to run benchmarks with options. PR-URL: https://github.com/iojs/io.js/pull/629 Reviewed-By:
Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By:
Evan Lucas <evanlucas@me.com>
-
Saúl Ibarra Corretgé authored
PR-URL: https://github.com/iojs/io.js/pull/644 Reviewed-By:
Ben Noordhuis <info@bnoordhuis.nl>
-
Roman Reiss authored
This restores the pretty-printed options output for improved readability and adds a final warning message to inform on possibly scrolled-off warnings. PR-URL: https://github.com/iojs/io.js/pull/638 Reviewed-By:
Ben Noordhuis <info@bnoordhuis.nl>
-
Ben Noordhuis authored
Signal dispositions are inherited by child processes. Restore ours to sane defaults in case our parent process changed it, to prevent quirky behavior when the parent does something silly like ignoring SIGSEGV. PR-URL: https://github.com/iojs/io.js/pull/615 Reviewed-By:
Sam Roberts <sam@strongloop.com>
-
Ben Noordhuis authored
Instead of installing an early debug signal handler, simply block the SIGUSR1 signal at start-up and unblock it when the debugger is ready. Both approaches are functionally equivalent but blocking the signal accomplishes it in fewer lines of code. PR-URL: https://github.com/iojs/io.js/pull/615 Reviewed-By:
Sam Roberts <sam@strongloop.com>
-
Ben Noordhuis authored
Execute the per-platform initialization logic as early as possible, for two reasons: 1. It opens the way for an upcoming commit to simplify early SIGUSR1 handling. 2. It should make life easier for embedders because io.js no longer mucks around with the file descriptor limit or signal disposition of the process. PR-URL: https://github.com/iojs/io.js/pull/615 Reviewed-By:
Sam Roberts <sam@strongloop.com>
-
- Jan 28, 2015
-
-
Ben Noordhuis authored
Add a regression test for https://github.com/iojs/io.js/issues/627. Before the http_parser rollback to 2.3.0, the request callback was called but an 'upgrade' event was not emitted, even though there is an Upgrade header present in the request. PR-URL: https://github.com/iojs/io.js/pull/628 Reviewed-By:
Fedor Indutny <fedor.indutny@gmail.com>
-
Ben Noordhuis authored
Commit 598efcbe ("deps: update http_parser to 2.4.1") introduced a regression in HTTP Upgrade header handling. Fixes: https://github.com/iojs/io.js/issues/627 PR-URL: https://github.com/iojs/io.js/pull/628 Reviewed-By:
Fedor Indutny <fedor.indutny@gmail.com>
-
Calvin Metcalf authored
Per #445 this removes a reference to this._readableState in hash._flush. It was used to get the encoding on the readable side to pass to the writable side but omitting it just causes the stream to handle the encoding issues. PR-URL: https://github.com/iojs/io.js/pull/610 Reviewed-By:
Chris Dickinson <christopher.s.dickinson@gmail.com> Reviewed-By:
Fedor Indutny <fedor.indutny@gmail.com> Reviewed-By:
Vladimir Kurchatkin <vladimir.kurchatkin@gmail.com>
-
Vladimir Kurchatkin authored
This makes possible to use `for..of` loop with buffers. Also related `keys`, `values` and `entries` methods are added for feature parity with `Uint8Array`. PR-URL: https://github.com/iojs/io.js/pull/525 Reviewed-By:
Ben Noordhuis <info@bnoordhuis.nl>
-