platform/upstream/nodejs.git
9 years agodeps: upgrade npm to 2.7.0
Forrest L Norvell [Fri, 6 Mar 2015 08:57:32 +0000 (02:57 -0600)]
deps: upgrade npm to 2.7.0

PR-URL: https://github.com/iojs/io.js/pull/1080
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
9 years agotls_wrap: do not hold persistent ref to parent
Fedor Indutny [Fri, 6 Mar 2015 01:27:58 +0000 (20:27 -0500)]
tls_wrap: do not hold persistent ref to parent

Hold non-persistent reference in JS, rather than in C++ to avoid cycles.

PR-URL: https://github.com/iojs/io.js/pull/1078
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
9 years agojs_stream: fix leak of instances
Fedor Indutny [Fri, 6 Mar 2015 01:26:12 +0000 (20:26 -0500)]
js_stream: fix leak of instances

Don't forget to call `MakeWeak` to ensure that instance objects are
garbage collectable.

PR-URL: https://github.com/iojs/io.js/pull/1078
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
9 years agostream_wrap: add HandleScope's in uv callbacks
Fedor Indutny [Thu, 5 Mar 2015 23:26:01 +0000 (18:26 -0500)]
stream_wrap: add HandleScope's in uv callbacks

Ensure that no handles will leak into global HandleScope by adding
HandleScope's in all JS-calling libuv callbacks in `stream_wrap.cc`.

Fix: https://github.com/iojs/io.js/issues/1075
PR-URL: https://github.com/iojs/io.js/pull/1078
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
9 years agobenchmark: fix `wrk` check
Brian White [Thu, 5 Mar 2015 22:12:17 +0000 (17:12 -0500)]
benchmark: fix `wrk` check

PR-URL: https://github.com/iojs/io.js/pull/1076
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
9 years agobuild: remove mdb from io.js
Johan Bergström [Mon, 2 Mar 2015 03:29:31 +0000 (14:29 +1100)]
build: remove mdb from io.js

After upgrading to a newer v8 mdb is pretty much broken - even if using
the latest updates from nodejs. If nodejs decides to update their v8 we
can give it another go, but for now it's better to remove it than have
it in our tree unsupported.

PR-URL: https://github.com/iojs/io.js/pull/1023
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Rod Vagg <rod@vagg.org>
9 years agohttp: send Content-Length when possible
Christian Tellnes [Tue, 3 Mar 2015 20:01:26 +0000 (21:01 +0100)]
http: send Content-Length when possible

This changes the behavior for http to send send a Content-Length header
instead of using chunked encoding when we know the size of the body when
sending the headers.

Fixes: https://github.com/iojs/io.js/issues/1044
PR-URL: https://github.com/iojs/io.js/pull/1062
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Rod Vagg <rod@vagg.org>
Reviewed-By: Brian White <mscdex@mscdex.net>
9 years agosrc: fix ucs-2 buffer encoding regression
Ben Noordhuis [Tue, 3 Mar 2015 14:44:54 +0000 (15:44 +0100)]
src: fix ucs-2 buffer encoding regression

StringBytes::Write() did a plain memcpy() when is_extern is true but
that's wrong when the source is a two-byte string and the destination
a one-byte or UTF-8 string.

The impact is limited to strings > 1,031,913 bytes because those are
normally the only strings that are externalized, although the use of
the 'externalize strings' extension (--expose_externalize_string) can
also trigger it.

This commit also cleans up the bytes versus characters confusion in
StringBytes::Write() because that was closely intertwined with the
UCS-2 encoding regression.  One wasn't fixable without the other.

Fixes: https://github.com/iojs/io.js/issues/1024
Fixes: https://github.com/joyent/node/issues/8683
PR-URL: https://github.com/iojs/io.js/pull/1042
Reviewed-By: Trevor Norris <trev.norris@gmail.com>
9 years agosrc: fix external string length calculation
Ben Noordhuis [Tue, 3 Mar 2015 13:46:16 +0000 (14:46 +0100)]
src: fix external string length calculation

Make StringBytes::GetExternalParts() return the byte length for two-byte
strings, not the character length.  Its callers operate on bytes, not
characters.

This also fixes StringBytes::Size() reporting only half of the actual
number of bytes for external two-byte strings.

PR-URL: https://github.com/iojs/io.js/pull/1042
Reviewed-By: Trevor Norris <trev.norris@gmail.com>
9 years agotest: simplify parallel/test-stringbytes-external
Ben Noordhuis [Tue, 3 Mar 2015 15:07:26 +0000 (16:07 +0100)]
test: simplify parallel/test-stringbytes-external

Make the algorithm that creates the big input strings a little easier
to comprehend.  No functional changes, the string lengths are unchanged.

PR-URL: https://github.com/iojs/io.js/pull/1042
Reviewed-By: Trevor Norris <trev.norris@gmail.com>
9 years agosrc: rename confusingly named local variable
Ben Noordhuis [Tue, 3 Mar 2015 14:20:54 +0000 (15:20 +0100)]
src: rename confusingly named local variable

Rename `val_` to `string`.  The underscore suffix is normally reserved
for data members, not locals, and it's not a great name in the first
place.

PR-URL: https://github.com/iojs/io.js/pull/1042
Reviewed-By: Trevor Norris <trev.norris@gmail.com>
9 years agosrc: simplify node::Utf8Value()
Ben Noordhuis [Tue, 3 Mar 2015 14:16:07 +0000 (15:16 +0100)]
src: simplify node::Utf8Value()

* Remove kStorageSize constant.

* Remove superfluous local variable and reinterpret_cast.

* Reorder data members so the length field and data pointer
  (if not the data itself) fit in a single cache line.

PR-URL: https://github.com/iojs/io.js/pull/1042
Reviewed-By: Trevor Norris <trev.norris@gmail.com>
9 years agosrc: remove NODE_INVALID_UTF8 environment variable
Ben Noordhuis [Tue, 3 Mar 2015 14:09:58 +0000 (15:09 +0100)]
src: remove NODE_INVALID_UTF8 environment variable

Introduced in joyent/node v0.10 as a backwards compatibility measure.
It's an ugly hack and allowing invalid UTF-8 is not a good idea in the
first place, remove it.

PR-URL: https://github.com/iojs/io.js/pull/1042
Reviewed-By: Trevor Norris <trev.norris@gmail.com>
9 years agosrc: fix gc heuristic for external twobyte strings
Ben Noordhuis [Tue, 3 Mar 2015 00:42:18 +0000 (01:42 +0100)]
src: fix gc heuristic for external twobyte strings

Large external two-byte strings reported their character length instead
of their byte length, throwing off the garbage collector heuristic by
a factor of two.

PR-URL: https://github.com/iojs/io.js/pull/1042
Reviewed-By: Trevor Norris <trev.norris@gmail.com>
9 years agosrc: remove unused code
Ben Noordhuis [Tue, 3 Mar 2015 00:37:59 +0000 (01:37 +0100)]
src: remove unused code

PR-URL: https://github.com/iojs/io.js/pull/1042
Reviewed-By: Trevor Norris <trev.norris@gmail.com>
9 years agosrc: extract node env init out of process init
Petka Antonov [Tue, 3 Mar 2015 20:25:47 +0000 (22:25 +0200)]
src: extract node env init out of process init

PR-URL: https://github.com/iojs/io.js/pull/980
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
9 years agotest: don't spawn child processes in domain test
Ben Noordhuis [Thu, 26 Feb 2015 20:54:09 +0000 (21:54 +0100)]
test: don't spawn child processes in domain test

Make parallel/test-domain-abort-on-uncaught a little easier to debug,
make it execute the tests in the same process instead of each test in
a separate child process.

PR-URL: https://github.com/iojs/io.js/pull/974
Reviewed-By: Chris Dickinson <christopher.s.dickinson@gmail.com>
9 years agosrc: fix -Wempty-body compiler warnings
Ben Noordhuis [Thu, 26 Feb 2015 20:34:03 +0000 (21:34 +0100)]
src: fix -Wempty-body compiler warnings

Turn counter macros into no-op instructions when counters are disabled.
Evaluating to nothing makes gcc complain when the macro is used in a
conditional.  Fixes the following warning:

    ../src/tls_wrap.cc:320:5: warning:
    suggest braces around empty body in an 'if' statement [-Wempty-body]
         NODE_COUNT_NET_BYTES_SENT(write_size_);
         ^

PR-URL: https://github.com/iojs/io.js/pull/974
Reviewed-By: Chris Dickinson <christopher.s.dickinson@gmail.com>
9 years agobuild: add basic mips/mipsel support
Ben Noordhuis [Tue, 3 Mar 2015 20:55:59 +0000 (21:55 +0100)]
build: add basic mips/mipsel support

Extend the configure script so that it knows how to generate build files
for mips and mipsel.  Actually building io.js is pending MIPS buildbots.

PR-URL: https://github.com/iojs/io.js/pull/1045
Reviewed-By: Rod Vagg <rod@vagg.org>
9 years agolib: avoid .toLowerCase() call in Buffer#write()
Ben Noordhuis [Wed, 4 Mar 2015 00:09:33 +0000 (01:09 +0100)]
lib: avoid .toLowerCase() call in Buffer#write()

Avoid a costly String#toLowerCase() call in Buffer#write() in the
common case, i.e., that the string is already lowercase.  Reduces
the running time of the following benchmark by about 40%:

    for (var b = Buffer(1), i = 0; i < 25e6; ++i) b.write('x', 'ucs2');

PR-URL: https://github.com/iojs/io.js/pull/1048
Reviewed-By: Trevor Norris <trev.norris@gmail.com>
9 years agolib: hand-optimize Buffer constructor
Ben Noordhuis [Tue, 3 Mar 2015 23:50:30 +0000 (00:50 +0100)]
lib: hand-optimize Buffer constructor

The Buffer constructor is used pervasively throughout io.js, yet it was
one of the most unwieldy functions in core.  This commit breaks up the
constructor into several small functions in a way that makes V8 happy.

About 8-10% CPU time was attributed to the constructor function before
in buffer-heavy benchmarks.  That pretty much drops to zero now because
V8 can now easily inline it at the call site.  It shortens the running
time of the following simple benchmark by about 15%:

    for (var i = 0; i < 25e6; ++i) new Buffer(1);

And about 8% from this benchmark:

    for (var i = 0; i < 1e7; ++i) new Buffer('x', 'ucs2');

PR-URL: https://github.com/iojs/io.js/pull/1048
Reviewed-By: Trevor Norris <trev.norris@gmail.com>
9 years agotty: do not add `shutdown` method to handle
Fedor Indutny [Thu, 5 Mar 2015 16:04:00 +0000 (11:04 -0500)]
tty: do not add `shutdown` method to handle

UV_TTY does not support `uv_shutdown()` so adding this method in
StreamBase will cause an `abort()` in C land.

Fix: https://github.com/iojs/io.js/issues/1068
PR-URL: https://github.com/iojs/io.js/pull/1073
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
9 years agonet: allow port 0 in connect()
cjihrig [Mon, 23 Feb 2015 16:23:53 +0000 (11:23 -0500)]
net: allow port 0 in connect()

The added validation allows non-negative numbers and numeric
strings. All other values result in a thrown exception.

Fixes: https://github.com/joyent/node/issues/9194
PR-URL: https://github.com/joyent/node/pull/9268
Reviewed-By: Julien Gilli <julien.gilli@joyent.com>
Reviewed-By: Trevor Norris <trev.norris@gmail.com>
Reviewed-By: James M Snell <jasnell@users.noreply.github.com>
9 years agobuffer: reword Buffer.concat error message
Chris Dickinson [Fri, 14 Nov 2014 00:55:16 +0000 (16:55 -0800)]
buffer: reword Buffer.concat error message

this brings the error messaging in line with
other node TypeError messages.

Fixes: https://github.com/joyent/node/issues/7766
PR-URL: https://github.com/joyent/node/pull/8723
Reviewed-By: James M Snell <jasnell@users.noreply.github.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
9 years agowin,test: fix test-stdin-from-file
Bert Belder [Thu, 5 Mar 2015 02:11:06 +0000 (03:11 +0100)]
win,test: fix test-stdin-from-file

The test-stdin-from-from-file test runs a subprocess that verifies stdin
can be piped from a file.

The subprocess additionally attempts to verify that the file descriptor
for stdin never gets closed. It used to do this by creating a TCP server
and asserting that the associated file descriptor is greater than two.
However this strategy doesn't work on windows, because servers don't
have an associated file descriptor. With this patch an ordinary file is
opened instead of creating a server.

PR: https://github.com/iojs/io.js/pull/1067
Reviewed-By: Fedor Indutny <fedor@indutny.com>
Reviewed-By: Rod Vagg <rod@vagg.org>
Reviewed-By: Nikolai Vavilov <vvnicholas@gmail.com>
9 years agofs: remove unnecessary usage of .hasOwnProperty()
Jonathan Ong [Wed, 28 Jan 2015 04:18:08 +0000 (20:18 -0800)]
fs: remove unnecessary usage of .hasOwnProperty()

PR-URL: https://github.com/iojs/io.js/pull/635
Reviewed-By: Vladimir Kurchatkin <vladimir.kurchatkin@gmail.com>
9 years agotest: adds a test for undefined value in setHeader
Ken Perkins [Thu, 26 Feb 2015 18:36:40 +0000 (10:36 -0800)]
test: adds a test for undefined value in setHeader

As a result of 979d0ca8 there is a new check for undefined values on
OutgoingMessage.setHeader. This commit introduces a test for this case.

PR-URL: https://github.com/iojs/io.js/pull/970
Reviewed-By: Rod Vagg <rod@vagg.org>
Reviewed-By: Brendan Ashworth <brendan.ashworth@me.com>
9 years agobenchmark: check for wrk ahead of running benchmarks
Johan Bergström [Fri, 27 Feb 2015 02:37:46 +0000 (13:37 +1100)]
benchmark: check for wrk ahead of running benchmarks

PR-URL: https://github.com/iojs/io.js/pull/982
Reviewed-By: Rod Vagg <rod@vagg.org>
9 years agobuild: remove tools/wrk from the tree
Johan Bergström [Thu, 26 Feb 2015 23:00:07 +0000 (10:00 +1100)]
build: remove tools/wrk from the tree

wrk is an optional tool that some of the http benchmarks uses. The removal
doesn't affect any users. Developers are assumed to install it before running
the tests.

This change reduces the tarball by 5%

PR-URL: https://github.com/iojs/io.js/pull/982
Reviewed-By: Rod Vagg <rod@vagg.org>
9 years agoevents: optimize listener array cloning
Brian White [Thu, 5 Mar 2015 03:21:47 +0000 (22:21 -0500)]
events: optimize listener array cloning

This both switches to a single algorithm for array cloning and also
speeds up (by ~100% in the ee-listeners-many benchmark) the
"many elements"  case that was previously handled by
`array.slice()`.

PR-URL: https://github.com/iojs/io.js/pull/1050
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Julian Duque <julianduquej@gmail.com>
9 years agotest: split parts out of host-headers test into its own test
Johan Bergström [Wed, 4 Mar 2015 22:36:47 +0000 (09:36 +1100)]
test: split parts out of host-headers test into its own test

this makes the separation between http and https testing cleaner

PR-URL: https://github.com/iojs/io.js/pull/1049
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Shigeki Ohtsu <ohtsu@iij.ad.jp>
9 years agotest: refactor all tests that depends on crypto
Johan Bergström [Wed, 4 Mar 2015 01:11:21 +0000 (12:11 +1100)]
test: refactor all tests that depends on crypto

we had a few ways versions of looking for support before executing a test. this
commit unifies them as well as add the check for all tests that previously
lacked them. found by running `./configure --without-ssl && make test`. also,
produce tap skip output if the test is skipped.

PR-URL: https://github.com/iojs/io.js/pull/1049
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Shigeki Ohtsu <ohtsu@iij.ad.jp>
9 years agotest: check for openssl cli and provide path if it exists
Johan Bergström [Wed, 4 Mar 2015 01:09:07 +0000 (12:09 +1100)]
test: check for openssl cli and provide path if it exists

the previous version checked if io.js was compiled with openssl support which
isn't really relevant since we're starting a http server. we on the other hand
need an openssl-cli which may or may not exist.

PR-URL: https://github.com/iojs/io.js/pull/1049
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Shigeki Ohtsu <ohtsu@iij.ad.jp>
9 years agotest: remove unused https imports
Johan Bergström [Wed, 4 Mar 2015 00:40:20 +0000 (11:40 +1100)]
test: remove unused https imports

PR-URL: https://github.com/iojs/io.js/pull/1049
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Shigeki Ohtsu <ohtsu@iij.ad.jp>
9 years agotest: introduce a helper that checks if crypto is available
Johan Bergström [Wed, 4 Mar 2015 00:36:07 +0000 (11:36 +1100)]
test: introduce a helper that checks if crypto is available

since this applies to tls and https (among other things), it'll be used
for those tests as well. if we decouple the build system to somehow support
crypto but not tls, we could refine this.

PR-URL: https://github.com/iojs/io.js/pull/1049
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Shigeki Ohtsu <ohtsu@iij.ad.jp>
9 years agotest: don't assume process.versions.openssl always is available
Johan Bergström [Wed, 4 Mar 2015 00:35:56 +0000 (11:35 +1100)]
test: don't assume process.versions.openssl always is available

PR-URL: https://github.com/iojs/io.js/pull/1049
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Shigeki Ohtsu <ohtsu@iij.ad.jp>
9 years agobuffer: add indexOf() method
Trevor Norris [Thu, 22 Jan 2015 23:44:13 +0000 (15:44 -0800)]
buffer: add indexOf() method

Add Buffer#indexOf(). Support strings, numbers and other Buffers. Also
included docs and tests.

Special thanks to Sam Rijs <srijs@airpost.net> for first proposing this
change.

PR-URL: https://github.com/iojs/io.js/pull/561
Reviewed-by: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Brian White <mscdex@mscdex.net>
Reviewed-By: Chris Dickinson <christopher.s.dickinson@gmail.com>
9 years agourl: throw for invalid values to url.format
Christian Tellnes [Tue, 3 Mar 2015 03:41:07 +0000 (04:41 +0100)]
url: throw for invalid values to url.format

`'use strict'` changes the behavior for `Function.prototype.call` when
the context is `undefined`. In earlier versions of node the value
`undefined` would make `url.format` look for fields in the global scope.

The docs states that `url.format` takes a parsed URL object and returns
a formatted URL string. So with this change it will now throw for other
values.

The exception is if the input is a string. Then it will call `url.parse`
on the string and then format it. The reason for that is that you can
call `url.format` on strings to clean up potentially wonky urls.

Fixes: https://github.com/iojs/io.js/issues/1033
PR-URL: https://github.com/iojs/io.js/pull/1036
Reviewed-By: Trevor Norris <trev.norris@gmail.com>
Reviewed-By: Brian White <mscdex@mscdex.net>
Reviewed-By: Julian Duque <julianduquej@gmail.com>
9 years agodoc: add TC meeting 2015-02-18 minutes
Rod Vagg [Wed, 4 Mar 2015 06:29:53 +0000 (22:29 -0800)]
doc: add TC meeting 2015-02-18 minutes

PR-URL: https://github.com/iojs/io.js/pull/1051
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
9 years agotimer: Improve performance of callbacks
Ruben Verborgh [Sat, 10 Jan 2015 15:49:21 +0000 (16:49 +0100)]
timer: Improve performance of callbacks

setImmediate, setTimeout, and setInterval were called in an inefficient
way, especially in the presence of arguments.  This optimization
improves their performance, with special cases for up to 4 arguments.
Performance of setImmediate increases with 35%, setInterval with 60%,
setTimeout with 70%.

PR-URL: https://github.com/iojs/io.js/pull/406
Reviewed-by: Trevor Norris <trev.norris@gmail.com>
Reviewed-by: Christian Tellnes <christian@tellnes.com>
9 years agodoc: remove cjihrig from TC
cjihrig [Wed, 4 Mar 2015 14:13:40 +0000 (09:13 -0500)]
doc: remove cjihrig from TC

PR-URL: https://github.com/iojs/io.js/pull/1056
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
9 years agosrc: fix compiler warning in smalloc.cc
Ben Noordhuis [Wed, 4 Mar 2015 11:07:36 +0000 (12:07 +0100)]
src: fix compiler warning in smalloc.cc

Fix the following compiler warning by static_casting the enum values
to an uint32_t:

    ../src/smalloc.cc: In function 'void
    node::smalloc::Initialize(v8::Handle<v8::Object>,
                              v8::Handle<v8::Value>,
                              v8::Handle<v8::Context>)':
    ../src/smalloc.cc:601:203: warning: enumeral and non-enumeral type
    in conditional expression
        EXTERNAL_ARRAY_TYPES(V)

PR-URL: https://github.com/iojs/io.js/pull/1055
Reviewed-By: Vladimir Kurchatkin <vladimir.kurchatkin@gmail.com>
9 years agosmalloc: export constants from C++
Vladimir Kurchatkin [Sun, 22 Feb 2015 20:01:08 +0000 (23:01 +0300)]
smalloc: export constants from C++

PR-URL: https://github.com/iojs/io.js/pull/920
Reviewed-By: Chris Dickinson <christopher.s.dickinson@gmail.com>
Reviewed-By: Trevor Norris <trev.norris@gmail.com>
9 years agosmalloc: validate arguments in js
Vladimir Kurchatkin [Sun, 22 Feb 2015 18:07:57 +0000 (21:07 +0300)]
smalloc: validate arguments in js

PR-URL: https://github.com/iojs/io.js/pull/920
Reviewed-By: Chris Dickinson <christopher.s.dickinson@gmail.com>
Reviewed-By: Trevor Norris <trev.norris@gmail.com>
9 years agohttp: optimize outgoing requests
Brendan Ashworth [Sun, 25 Jan 2015 23:26:40 +0000 (15:26 -0800)]
http: optimize outgoing requests

This commit does some small optimization changes on
`lib/_http_outgoing.js`. These include switching from `while` loops to
`for` loops, moving away from `util` to `typeof` checks, and removing
dead code. It also includes variable caches to avoid lookups and
generic style changes. All in all, much faster execution.

It gets an across the board increase in req/sec on the benchmarks,
from my experience about a 10% increase.

PR-URL: https://github.com/iojs/io.js/pull/605
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Nicu Micleușanu <micnic90@gmail.com>
Reviewed-By: Christian Vaagland Tellnes <christian@tellnes.com>
Reviewed-By: Brian White <mscdex@mscdex.net>
9 years agotls: accept empty `net.Socket`s
Fedor Indutny [Tue, 3 Mar 2015 20:17:43 +0000 (15:17 -0500)]
tls: accept empty `net.Socket`s

Accept `new net.Socket()` as a `socket` option to `tls.connect()`
without triggering an assertion error in C++.

This is done by wrapping it into a JSStream to ensure that there will be
a handle at the time of wrapping the socket into TLSSocket.

Fix: https://github.com/iojs/io.js/issues/987
PR-URL: https://github.com/iojs/io.js/pull/1046
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Rod Vagg <rod@vagg.org>
9 years agotest: fix racey-ness in tls-inception
Fedor Indutny [Tue, 3 Mar 2015 09:10:28 +0000 (04:10 -0500)]
test: fix racey-ness in tls-inception

Fix test failure on FreeBSD and SmartOS, which happens due to a bad
timing:

    events.js:141
          throw er; // Unhandled 'error' event
                ^
    Error: read ECONNRESET
        at exports._errnoException (util.js:734:11)
        at TLSWrap.onread (net.js:538:26)

The outer `net.conncet()` socket stays alive after the inner socket is
gone. This happens because `.pipe()`'s implementation does not `destroy`
the source side when the destination has emitted `close`.

Fix: https://github.com/iojs/io.js/issues/1012
PR-URL: https://github.com/iojs/io.js/pull/1040
Reviewed-By: Chris Dickinson <christopher.s.dickinson@gmail.com>
9 years agonode: improve performance of nextTick
Trevor Norris [Fri, 27 Feb 2015 05:41:54 +0000 (22:41 -0700)]
node: improve performance of nextTick

Couple micro optimizations to improve performance of process.nextTick().
Removes ~60ns of execution time.

Also added small threshold to test that allows timer to fire early on
the order if microseconds.

PR-URL: https://github.com/iojs/io.js/pull/985
Reviewed-By: Vladimir Kurchatkin <vladimir.kurchatkin@gmail.com>
9 years agodoc: fix child_process heading depth
Sam Roberts [Tue, 3 Mar 2015 05:10:19 +0000 (21:10 -0800)]
doc: fix child_process heading depth

Addition of the Sync/Async headers didn't indent all the sub-headers.

Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
PR-URL: https://github.com/iojs/io.js/pull/1038

9 years agotest: fix test-fs-access when uid is 0
Johan Bergström [Tue, 3 Mar 2015 05:29:03 +0000 (16:29 +1100)]
test: fix test-fs-access when uid is 0

Superusers can open files with W_OK permission even though their
mode is set to 0444. This commit makes the test attempt to change
its uid to nobody on non-Windows platforms. Patch originally from
https://github.com/joyent/node/commit/28d0cbbd.

Fixes: https://github.com/iojs/io.js/issues/1031
PR-URL: https://github.com/iojs/io.js/pull/1037
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
9 years agotest: make destroyed-socket-write2.js more robust
Michael Dawson [Mon, 23 Feb 2015 19:13:20 +0000 (14:13 -0500)]
test: make destroyed-socket-write2.js more robust

test/simple/test-http-destroyed-socket-write2.js validates
that you get an appropriate error when trying to write to
a request when the response on the other side has been destroyed.

The test uses http.request to get a request and then keeps writing
to it until either it hits 128 writes or gets the expected error.
Since the writes are asynchronous we see that the writes just end
up adding events to the event loop, which then later get processed
once the connection supporting the request is fully ready.

The test is timing dependent and if takes too long for the connection
to be made the limit of 128 writes is exceeded and the test fails.
The fact that the test allows a number of writes is probably to allow
some delay for the connection to be ready for writing.

On AIX, in the default configuration using the loopback interface
is slower and the test fails because the delay is such that many
more writes can be queued up before the connection takes place.
If we use the host ip instead of defaulting to the loopback then
the test passes.

The test needs to be made more robust to delays. Since each write
simply enqueues an additional write to the event queue there is
probably no point in doing the second write until the first has
completed. This patch schedules the next write when the first one
completes and allows the test to pass even if it takes longer for
the connection to be ready for writing

PR-URL: https://github.com/joyent/node/pull/9270
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Timothy J Fontaine <tjfontaine@gmail.com>
9 years agodoc: add explanations for querystring
Robert Kowalski [Sat, 21 Feb 2015 00:03:14 +0000 (01:03 +0100)]
doc: add explanations for querystring

 - add an article: `decode a non-utf8 string`
 - explain default and fallback behaviour of `querystring.unescape`

PR-URL: https://github.com/joyent/node/pull/9259
Reviewed-by: Julien Gilli <julien.gilli@joyent.com>
9 years agodoc: fix default value of opts.decodeURIComponent
h7lin [Fri, 13 Feb 2015 03:59:11 +0000 (11:59 +0800)]
doc: fix default value of opts.decodeURIComponent

In the documentation for querystring.parse, the documentation mentions
that the default value for options.decodeURIComponent is the
decodeURIComponent function, but it's actually the querystring.unescape
function.

PR-URL: https://github.com/joyent/node/pull/9259
Reviewed-by: Julien Gilli <julien.gilli@joyent.com>
9 years agotests: fix race in test-http-curl-chunk-problem
Julien Gilli [Fri, 27 Feb 2015 23:48:34 +0000 (15:48 -0800)]
tests: fix race in test-http-curl-chunk-problem

This test setups two event listeners: one on a child process' exit event
, another for the same child process' stdandard output's 'data' event.
The data even listener writes to a stream, and the exit event listener
ends it.

Because the exit event can be emitted before the data event, there is a
chance that something will be written to the stream after it's ended,
and that an error is thrown.

This change makes the test end the stream in the listener for the child
process' standard output's end event, which is guaranteed to be emitted
after the last data event, thus avoiding the race.

PR: https://github.com/joyent/node/pull/9301
Reviewed-by: Bert Belder <bertbelder@gmail.com>
Reviewed-by: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-by: Colin Ihrig <cjihrig@gmail.com>
9 years agodoc: add missing newline in CHANGELOG
Rod Vagg [Tue, 3 Mar 2015 01:49:24 +0000 (17:49 -0800)]
doc: add missing newline in CHANGELOG

Pushed without review

9 years agoWorking on v1.4.4
Rod Vagg [Tue, 3 Mar 2015 00:48:12 +0000 (16:48 -0800)]
Working on v1.4.4

9 years ago2015-03-02 io.js v1.4.3 Release v1.4.3
Rod Vagg [Tue, 3 Mar 2015 00:45:43 +0000 (16:45 -0800)]
2015-03-02 io.js v1.4.3 Release

Notable changes:

* stream: Fixed problems for platforms without `writev()` support,
  particularly Windows. Changes introduced in 1.4.1, via
  https://github.com/iojs/io.js/pull/926, broke some
  functionality for these platforms, this has now been addressed.
  https://github.com/iojs/io.js/pull/1008 (Fedor Indutny)
* arm: We have the very beginnings of ARMv8 / ARM64 / AARCH64
  support. An upgrade to OpenSSL 1.0.2 is one requirement for full
  support. https://github.com/iojs/io.js/pull/1028
  (Ben Noordhuis)
* Add new collaborator: Julian Duque @julianduque

9 years agobuild: add basic arm64 support
Ben Noordhuis [Mon, 2 Mar 2015 20:35:29 +0000 (21:35 +0100)]
build: add basic arm64 support

This commit adds basic arm64 support to the build.  Building the bundled
openssl is disabled pending an upgrade to openssl 1.2, the currently
bundled version has some hand-rolled assembly that is 32 bits only.

PR-URL: https://github.com/iojs/io.js/pull/1028
Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
Reviewed-By: Rod Vagg <rod@vagg.org>
9 years agosrc: fix --without-ssl build
Ben Noordhuis [Mon, 2 Mar 2015 22:18:44 +0000 (23:18 +0100)]
src: fix --without-ssl build

Don't include tls_wrap.h in stream_base.cc.  It's not used and it breaks
the build when --without-ssl is passed to configure.

Fixes the following build error:

    In file included from ../src/tls_wrap.h:5:0,
                     from ../src/stream_base.cc:10:
    ../src/node_crypto.h:20:25: fatal error: openssl/ssl.h:
    No such file or directory
     #include <openssl/ssl.h>

PR-URL: https://github.com/iojs/io.js/pull/1027
Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
9 years agodoc: update AUTHORS list
Rod Vagg [Sun, 1 Mar 2015 19:37:55 +0000 (06:37 +1100)]
doc: update AUTHORS list

Update AUTHORS file using tools/update-authors.sh

PR-URL: https://github.com/iojs/io.js/pull/1018
Reviewed-By: Chris Dickinson <christopher.s.dickinson@gmail.com>
9 years agodoc: add julianduque as collaborator
Julian Duque [Sun, 1 Mar 2015 22:57:12 +0000 (17:57 -0500)]
doc: add julianduque as collaborator

PR-URL: https://github.com/iojs/io.js/pull/1021
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Christian Tellnes <christian@tellnes.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Rod Vagg <r@va.gg>
9 years agodoc: fix typos and sources in WORKING_GROUPS.md
&! (bitandbang) [Mon, 2 Mar 2015 00:31:05 +0000 (19:31 -0500)]
doc: fix typos and sources in WORKING_GROUPS.md

`therefor` is a typo of `therefore`, and was fixed. There were also two
places where the website WG was directly linked, where they should have
put the WG's name/repo; that was fixed as well.

PR-URL: https://github.com/iojs/io.js/pull/1022
Reviewed-By: Mikeal Rogers <mikeal.rogers@gmail.com>
Reviewed-By: Christian Tellnes <christian@tellnes.no>
9 years agodoc: Clean up net.Socket
Ryan Scheel [Wed, 25 Feb 2015 11:24:45 +0000 (03:24 -0800)]
doc: Clean up net.Socket

net.Socket::connect(options[, connectListener]) was missing, with the
relevant details found on the net.connect function. I moved the
appropriate documentation over and then rewrote the documentation for
the function to say that it just creates a socket and calls the connect
method on it. I also changed the other net.Socket::connect variants to
say they are like the options version but called with a specific
options object.

net.connect and other functions were called methods even though they
don't use the `this` binding, so I changed method to function where
appropriate.

Finally, I added a missing period to the end of the module summary.
It's not really related to the rest of the changes, but benjamingr
noticed it.

PR-URL: https://github.com/iojs/io.js/pull/951
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
Reviewed-By: Christian Tellnes <christian@tellnes.no>
9 years agodoc: suggest alternatives to deprecated APs
Benjamin Gruenbaum [Sat, 28 Feb 2015 20:30:03 +0000 (22:30 +0200)]
doc: suggest alternatives to deprecated APs

At the moment users who want to use `fs.exists` get a warning that the
method is deprecated but do not get offered an alternative in the page.
This PR suggests `fs.stat` and `fs.access` as alternatives while
keeping the warning about the use case in place.

Fixes: https://github.com/iojs/io.js/issues/1002
PR-URL: https://github.com/iojs/io.js/pull/1007
Reviewed-By: Brian White <mscdex@mscdex.net>
Reviewed-By: Christian Tellnes <christian@tellnes.no>
9 years agostream_base: `.writev()` has limited support
Fedor Indutny [Sat, 28 Feb 2015 21:46:43 +0000 (00:46 +0300)]
stream_base: `.writev()` has limited support

Only TCP and JSStream do support `.writev()` on all platforms at the
moment. Ensure that it won't be enabled everywhere.

Fix: https://github.com/iojs/io.js/issues/995
PR-URL: https://github.com/iojs/io.js/pull/1008
Reviewed-by: Bert Belder <bertbelder@gmail.com>
Reviewed-By: Rod Vagg <rod@vagg.org>
9 years agoWorking on v1.4.3
Rod Vagg [Sat, 28 Feb 2015 21:16:32 +0000 (15:16 -0600)]
Working on v1.4.3

9 years ago2015-02-28 io.js v1.4.2 Release v1.4.2
Rod Vagg [Sat, 28 Feb 2015 21:12:58 +0000 (15:12 -0600)]
2015-02-28 io.js v1.4.2 Release

Notable changes:

* tls: A typo introduced in the TLSWrap changes in
  https://github.com/iojs/io.js/pull/840 only encountered as a bug on
  Windows was not caught by the io.js CI system due to problems with the
  Windows build script and the Windows CI slave configuration, see
  Fixed in https://github.com/iojs/io.js/pull/994 &
  https://github.com/iojs/io.js/pull/1004 (Fedor Indutny)
* npm: Upgrade npm to 2.6.1. See
  https://github.com/npm/npm/blob/master/CHANGELOG.md#v260-2015-02-12
  for details.
* Add new collaborators:
  - Robert Kowalski (@robertkowalski)
  - Christian Vaagland Tellnes (@tellnes)
  - Brian White (@mscdex)

9 years agotls: proxy `handle.reading` back to parent handle
Fedor Indutny [Sat, 28 Feb 2015 18:14:07 +0000 (21:14 +0300)]
tls: proxy `handle.reading` back to parent handle

Fix: https://github.com/iojs/io.js/issues/995
PR-URL: https://github.com/iojs/io.js/pull/1004
Reviewed-By: Rod Vagg <rod@vagg.org>
9 years agoinstall: new performance counters provider guid
Russell Dempsey [Sat, 28 Feb 2015 14:54:13 +0000 (08:54 -0600)]
install: new performance counters provider guid

The io.js performance counters manifest conflicts with the one from
node, causing installer failures. Change the GUID in the io.js
performance counters manifest file to resolve the conflicts.

Fixes: https://github.com/iojs/io.js/issues/524
PR: https://github.com/iojs/io.js/pull/1001
Reviewed-by: Bert Belder <bertbelder@gmail.com>
9 years agotls: fix typo `handle._reading` => `handle.reading`
Fedor Indutny [Fri, 27 Feb 2015 23:07:25 +0000 (02:07 +0300)]
tls: fix typo `handle._reading` => `handle.reading`

The problem does not manifest itself on unixes, because
`uv_read_start()` always return 0 there. However on Windows on a second
call `uv_read_start()` returns `UV__EALREADY` destroying all sockets on
a read attempt.

Set `.reading` property that is already handled by `net.js` code.

Fix: https://github.com/iojs/io.js/issues/988
PR-URL: https://github.com/iojs/io.js/pull/994
Reviewed-By: Chris Dickinson <christopher.s.dickinson@gmail.com>
9 years agohttp: don't confuse automatic headers for others
Christian Tellnes [Fri, 13 Feb 2015 02:20:06 +0000 (03:20 +0100)]
http: don't confuse automatic headers for others

If you set a custom http header which includes eg. the string `Date`,
then http will not automatically send the `Date` header.

This is also true for other automatic http headers.

PR-URL: https://github.com/iojs/io.js/pull/828
Reviewed-By: Brendan Ashworth <brendan.ashworth@me.com>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
9 years agobuild: improve vcbuild.bat
Bert Belder [Sat, 28 Feb 2015 01:42:49 +0000 (02:42 +0100)]
build: improve vcbuild.bat

* Support running multiple test suites (e.g. vcbuild test-simple test-gc).
* Removed a nonexisting test suite (test-uv).
* Removed superfluous extra layer of argument parsing.
* Fix the node-weak build.

R=@rvagg

PR-URL: https://github.com/iojs/io.js/pull/998
Reviewed-By: Rod Vagg <rod@vagg.org>
9 years agobuild: reduce tarball size by 8-10%
Johan Bergström [Thu, 26 Feb 2015 00:08:16 +0000 (11:08 +1100)]
build: reduce tarball size by 8-10%

Slim the tarballs further by removing examples, documentation and test for
third party libraries. Also switch to checkout-index versus archive so we avoid
using tar.

PR-URL: https://github.com/iojs/io.js/pull/961
Reviewed-By: Trevor Norris <trev.norris@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Rod Vagg <rod@vagg.org>
9 years agodoc: minor formatting fixes.
Tim Oxley [Sat, 28 Feb 2015 00:00:13 +0000 (18:00 -0600)]
doc: minor formatting fixes.

PR-URL: https://github.com/iojs/io.js/pull/996
Reviewed-By: Rod Vagg <rod@vagg.org>
9 years agodoc: update stability index
Chris Dickinson [Wed, 25 Feb 2015 00:15:26 +0000 (16:15 -0800)]
doc: update stability index

This simplifies the stability index to 4 levels:

0 - deprecated
1 - experimental / feature-flagged
2 - stable
3 - locked

Domains has been downgraded to deprecated, assert has been
downgraded to stable. Timers and Module remain locked. All
other APIs are now stable.

PR-URL: https://github.com/iojs/io.js/pull/943
Fixes: https://github.com/iojs/io.js/issues/930
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: Vladimir Kurchatkin <vladimir.kurchatkin@gmail.com>
9 years agodoc: add robertkowalski as collaborator
Robert Kowalski [Thu, 26 Feb 2015 22:43:29 +0000 (23:43 +0100)]
doc: add robertkowalski as collaborator

PR-URL: https://github.com/iojs/io.js/pull/977
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Christian Tellnes <christian@tellnes.com>
Reviewed-By: Rod Vagg <r@va.gg>
9 years agodeps: make node-gyp work with io.js
cjihrig [Fri, 27 Feb 2015 18:40:58 +0000 (13:40 -0500)]
deps: make node-gyp work with io.js

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>
9 years agodeps: upgrade npm to 2.6.1
Forrest L Norvell [Fri, 27 Feb 2015 13:39:59 +0000 (05:39 -0800)]
deps: upgrade npm to 2.6.1

PR-URL: https://github.com/iojs/io.js/pull/990
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
9 years agodoc: update os.markdown
Benjamin Gruenbaum [Thu, 26 Feb 2015 21:57:45 +0000 (23:57 +0200)]
doc: update os.markdown

Some doc update based on improvement ideas I remember from when I used this module in node:

 - Mention windows returns 0 for `nice` values (which is obvious, but io makes no attempt to calculate something similar or return undefined, 0 is returned)
 - Mention platform and arch are aliases for `process` properties.
 - Document possible return values where appropriate, add examples in others.
 - Rename title in order to match other titles in the navigation.
 - Fix line that was over 80 characters long.

PR-URL: https://github.com/iojs/io.js/pull/976
Reviewed-By: Christian Tellnes <christian@tellnes.no>
Reviewed-By: Brian White <mscdex@mscdex.net>
Reviewed-By: Vladimir Kurchatkin <vladimir.kurchatkin@gmail.com>
9 years agosrc: add check for already defined macro NOMINMAX
Pavel Medvedev [Fri, 27 Feb 2015 07:00:43 +0000 (10:00 +0300)]
src: add check for already defined macro NOMINMAX

In order to avoid Visual C++ warning C4005 about macro redefinition
when node.h is included in another project.

PR-URL: https://github.com/iojs/io.js/pull/986
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
9 years agodoc: add roadmap, i18n, tracing, evangelism WGs
Mikeal Rogers [Sat, 21 Feb 2015 00:07:35 +0000 (16:07 -0800)]
doc: add roadmap, i18n, tracing, evangelism WGs

PR-URL: https://github.com/iojs/io.js/pull/911
Reviewed-By: Chris Dickinson <christopher.s.dickinson@gmail.com>
Reviewed-by: Bert Belder <bertbelder@gmail.com>
9 years agodoc: document roadmap, workgroups
Mikeal Rogers [Thu, 26 Feb 2015 22:48:07 +0000 (23:48 +0100)]
doc: document roadmap, workgroups

PR: https://github.com/iojs/io.js/pull/886
Reviewed-By: Chris Dickinson <christopher.s.dickinson@gmail.com>
Reviewed-by: Bert Belder <bertbelder@gmail.com>
9 years agodoc: add tellnes as collaborator
Christian Tellnes [Thu, 26 Feb 2015 20:04:40 +0000 (21:04 +0100)]
doc: add tellnes as collaborator

PR-URL: https://github.com/iojs/io.js/pull/973
Reviewed-By: Chris Dickinson <christopher.s.dickinson@gmail.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Brian White <mscdex@mscdex.net>
9 years agodoc: add mscdex as collaborator
Brian White [Thu, 26 Feb 2015 20:04:27 +0000 (15:04 -0500)]
doc: add mscdex as collaborator

PR-URL: https://github.com/iojs/io.js/pull/972
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
9 years agoWorking on v1.4.2
Rod Vagg [Thu, 26 Feb 2015 19:40:25 +0000 (13:40 -0600)]
Working on v1.4.2

9 years ago2015-02-26 io.js v1.4.1 Release v1.4.1
Rod Vagg [Wed, 25 Feb 2015 19:18:53 +0000 (13:18 -0600)]
2015-02-26 io.js v1.4.1 Release

Notable changes:

* process / promises: An'unhandledRejection' event is now emitted on
  process whenever a Promise is rejected and no error handler is
  attached to the Promise within a turn of the event loop. A
  'rejectionHandled' event is now emitted whenever a Promise was
  rejected and an error handler was attached to it later than after an
  event loop turn. See the process documentation for more detail.
  https://github.com/iojs/io.js/pull/758 (Petka Antonov)
* streams: you can now use regular streams as an underlying socket for
  tls.connect() https://github.com/iojs/io.js/pull/758 (Fedor Indutny)
* http: A new 'abort' event emitted when a http.ClientRequest is
  aborted by the client. https://github.com/iojs/io.js/pull/945
  (Evan Lucas)
* V8: Upgrade V8 to 4.1.0.21. Includes an embargoed fix, details
  should be available at
  https://code.google.com/p/chromium/issues/detail?id=430201
  when embargo is lifted. A breaking ABI change has been held back
  from this upgrade, possibly to be included when io.js merges V8 4.2.
  See https://github.com/iojs/io.js/pull/952 for discussion.
* npm: Upgrade npm to 2.6.0. Includes features to support the new
  registry and to prepare for npm@3. See npm CHANGELOG.md
  https://github.com/npm/npm/blob/master/CHANGELOG.md#v260-2015-02-12
  for details.
* libuv: Upgrade to 1.4.2. See libuv ChangeLog
  https://github.com/libuv/libuv/blob/v1.x/ChangeLog for details of
  fixes.

9 years agodoc: fix process.stdout reference to console.log
Brendan Ashworth [Thu, 26 Feb 2015 05:31:42 +0000 (21:31 -0800)]
doc: fix process.stdout reference to console.log

This commit changes the word introduction to the code block to be
explicitly correct, and changes the `d` variable to `msg` for clarity.

PR-URL: https://github.com/iojs/io.js/pull/964
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: Nicu Micleușanu <micnic90@gmail.com>
Reviewed-By: Vladimir Kurchatkin <vladimir.kurchatkin@gmail.com>
9 years agodoc: link & formatting of SHAs in commit list
Tim Oxley [Thu, 26 Feb 2015 12:15:40 +0000 (06:15 -0600)]
doc: link & formatting of SHAs in commit list

This adds more consistency, plus links to commits are particularly useful.

PR-URL: https://github.com/iojs/io.js/pull/967
Reviewed-By: Rod Vagg <rod@vagg.org>
9 years agodoc: fix 'dhparam' description of tls.createServer
silverwind [Thu, 26 Feb 2015 14:36:41 +0000 (15:36 +0100)]
doc: fix 'dhparam' description of tls.createServer

fixes #958

Fixes: https://github.com/iojs/io.js/issues/958
PR-URL: https://github.com/iojs/io.js/pull/968
Reviewed-By: Shigeki Ohtsu <ohtsu@iij.ad.jp>
9 years agodeps: update libuv to 1.4.2
Ben Noordhuis [Thu, 26 Feb 2015 10:27:18 +0000 (11:27 +0100)]
deps: update libuv to 1.4.2

PR-URL: https://github.com/iojs/io.js/pull/966
Reviewed-By: Fedor Indutny <fedor@indutny.com>
9 years agosrc: fix intermittent SIGSEGV in resolveTxt
Evan Lucas [Wed, 25 Feb 2015 22:59:29 +0000 (16:59 -0600)]
src: fix intermittent SIGSEGV in resolveTxt

Fixes a SIGSEGV by making sure `txt_chunk` is not empty before setting
it on `txt_records`

PR-URL: https://github.com/iojs/io.js/pull/960
Reviewed-By: Rod Vagg <rod@vagg.org>
9 years agosrc: fix domains + --abort-on-uncaught-exception
Chris Dickinson [Sun, 22 Feb 2015 22:54:25 +0000 (14:54 -0800)]
src: fix domains + --abort-on-uncaught-exception

If run with --abort-on-uncaught-exception, V8 will abort the process
whenever it does not see a JS-installed CatchClause in the stack. C++
TryCatch clauses are ignored. Domains work by setting a FatalException
handler which is ignored when running in abort mode.

This patch modifies MakeCallback to call its target function through a
JS function that installs a CatchClause and manually calls _fatalException
on error, if the process is both using domains and is in abort mode.

Semver: patch
PR-URL: https://github.com/iojs/io.js/pull/922
Fixes: https://github.com/iojs/io.js/issues/836
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
9 years agohttp: emit abort event from ClientRequest
Evan Lucas [Wed, 25 Feb 2015 00:11:11 +0000 (18:11 -0600)]
http: emit abort event from ClientRequest

ClientRequest will now emit an abort event the first time abort()
is called.

Semver: Minor
Fixes: https://github.com/joyent/node/issues/9278
PR-URL: https://github.com/iojs/io.js/pull/945
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Chris Dickinson <christopher.s.dickinson@gmail.com>
9 years agostream_base: remove static JSMethod declarations
Fedor Indutny [Wed, 25 Feb 2015 17:43:14 +0000 (20:43 +0300)]
stream_base: remove static JSMethod declarations

Move JS methods to the stream_base-inl.h and thus define them on each
use of `StreamBase::AddMethods`. Inline `AddMethods` itself, so that
there won't be any need in a static declaration in stream_base.cc.

NOTE: This basically allows using this API in user-land, though, some
polishing is required before releasing it.

PR-URL: https://github.com/iojs/io.js/pull/957
Reviewed-By: Chris Dickinson <christopher.s.dickinson@gmail.com>
9 years agodeps: revert v8 abi change
Ben Noordhuis [Wed, 25 Feb 2015 11:18:50 +0000 (12:18 +0100)]
deps: revert v8 abi change

Undo the ABI (but not API) change to NamedPropertyHandlerConfiguration.
This avoids a NODE_MODULE_VERSION bump and forcing everyone to recompile
their add-ons, at the cost of increasing the delta with upstream V8.

This commit effectively backs out 4.1.0.16, the release that introduced
the ABI change (and nothing else.)

PR-URL: https://github.com/iojs/io.js/pull/952
Reviewed-By: Fedor Indutny <fedor@indutny.com>
Reviewed-By: Rod Vagg <rod@vagg.org>
9 years agodeps: fix postmortem support in v8
Fedor Indutny [Tue, 3 Feb 2015 13:59:19 +0000 (16:59 +0300)]
deps: fix postmortem support in v8

Fix: iojs/io.js#461
PR-URL: https://github.com/iojs/io.js/pull/706
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
9 years agodeps: upgrade v8 to 4.1.0.21
Ben Noordhuis [Wed, 25 Feb 2015 10:41:56 +0000 (11:41 +0100)]
deps: upgrade v8 to 4.1.0.21

PR-URL: https://github.com/iojs/io.js/pull/952
Reviewed-By: Fedor Indutny <fedor@indutny.com>
Reviewed-By: Rod Vagg <rod@vagg.org>
9 years agodeps: update libuv to 1.4.1
Ben Noordhuis [Tue, 24 Feb 2015 20:03:00 +0000 (21:03 +0100)]
deps: update libuv to 1.4.1

PR-URL: https://github.com/iojs/io.js/pull/940
Reviewed-By: Fedor Indutny <fedor@indutny.com>
Reviewed-By: Rod Vagg <rod@vagg.org>
9 years agodoc: document 'unhandledRejection' and 'rejectionHandled'
Benjamin Gruenbaum [Wed, 25 Feb 2015 00:44:10 +0000 (02:44 +0200)]
doc: document 'unhandledRejection' and 'rejectionHandled'

Documents the new unhandled rejection detection API.

Documents the new unhandledRejection/rejectionHandled events in the process
docuemntation. As agreed on in this issue:

https://github.com/iojs/io.js/issues/256#event-241385784

PR-URL: https://github.com/iojs/io.js/pull/946
Reviewed-By: Domenic Denicola <domenic@domenicdenicola.com>
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
9 years agodoc: update documentation.markdown for io.js.
Ryan Scheel [Wed, 25 Feb 2015 11:00:49 +0000 (03:00 -0800)]
doc: update documentation.markdown for io.js.

PR-URL: https://github.com/iojs/io.js/pull/950
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>