- Jan 21, 2014
-
-
Alexis Campailla authored
This test was originally intended to guard against regressions for commit 16b59cbc. As such, it only needs to ensure that process exit has not been held up by the date cache timer, which would fire on the next second.
-
Alexis Campailla authored
The test needs a little more time to run so that it passes for all builds (eg: Windows, debug)
-
Alexis Campailla authored
We now wait to connect to the debuggee until we know that its error stream has data, to ensure that the output message "connecting..... ok" appears after "Debugger listening on port xyz" I also increased the test timeout to let the more complex tests finish in time on Windows This change fixes the following unit tests on Windows: test-debugger-repl.js test-debugger-repl-term.js test-debugger-repl-utf8.js test-debugger-repl-restart.js
-
Timothy J Fontaine authored
-
Timothy J Fontaine authored
-
Fedor Indutny authored
fix #6880
-
Fedor Indutny authored
fix #6889
-
- Jan 20, 2014
-
-
Ben Noordhuis authored
As of commit 39aa8940, SSLv2 support is disabled by default. Update the documentation to reflect that.
-
Ben Noordhuis authored
* Make it clear that ECDHE-RSA-AES128-SHA256 and AES128-GCM-SHA256 are TLS v1.2 ciphers. * Note that RC4 is under suspicion.
-
- Jan 18, 2014
-
-
Ben Noordhuis authored
addon_register_func and its cousin addon_context_register_func are type definitions, dllimport and dllexport are name mangling directives, i.e. they're quite unrelated concepts. MinGW complains about mixing them when cross-compiling native add-ons. Signed-off-by:
Trevor Norris <trev.norris@gmail.com> Signed-off-by:
Bert Belder <bertbelder@gmail.com>
-
Ben Noordhuis authored
Not all ciphers require the presence of a certificate. Remove the check in lib/_tls_wrap.js. Fixes #6887.
-
Ben Noordhuis authored
Before this commit, verification exceptions had err.message set to the OpenSSL error code (e.g. 'UNABLE_TO_VERIFY_LEAF_SIGNATURE'). This commit moves the error code to err.code and replaces err.message with a human-readable error. Example: // before { message: 'UNABLE_TO_VERIFY_LEAF_SIGNATURE' } // after { code: 'UNABLE_TO_VERIFY_LEAF_SIGNATURE', message: 'unable to verify the first certificate' } UNABLE_TO_VERIFY_LEAF_SIGNATURE is a good example of why you want this: the error code suggests that it's the last certificate that fails to validate while it's actually the first certificate in the chain. Going by the number of mailing list posts and StackOverflow questions, it's a source of confusion to many people.
-
- Jan 16, 2014
-
-
Trevor Norris authored
Conflicts: lib/domain.js
-
Ryan Graham authored
domain.create().exit() should not clear the domain stack if the domain instance does not exist within the stack. Signed-off-by:Trevor Norris <trev.norris@gmail.com>
-
isaacs authored
-
- Jan 14, 2014
-
-
Fedor Indutny authored
fix #6859
-
Fedor Indutny authored
-
Alexis Campailla authored
Killing the worker without ensuring the socket was closed was causing intermittent ECONNRESET errors.
-
Timothy J Fontaine authored
-
Timothy J Fontaine authored
Conflicts: deps/uv/ChangeLog deps/uv/build.mk deps/uv/src/version.c deps/uv/test/test-ipc.c deps/v8/src/objects.cc src/node.cc src/node_os.cc
-
Sam Roberts authored
spawn stdio options can be a 'stream', but the following code fails with "Incorrect value for stdio stream: [object Object]", despite being a stream. The problem is the test isn't really for a stream, its for an object with a numeric `.fd` property, and streams do not have an fd until their async 'open' event has occurred. This is reasonable, but was not documented. child_process.spawn('date', [], {stdio: [ 'ignore', fs.createWriteStream('out.txt',{flags:'a'}), 'ignore']}) -
Alexis Campailla authored
Killing the debuggee without first closing the socket can result in an ECONNRESET error.
-
Alex Kocharin authored
-
- Jan 13, 2014
-
-
Timothy J Fontaine authored
If node was compiled with --no-ifaddrs to support older operating systems, don't throw instead simply return an empty object Fixes #6846
-
- Jan 11, 2014
-
-
svenpanne@chromium.org authored
Keep the number of descriptors below DescriptorArray::kMaxNumberOfDescriptors even for accessors Review URL: https://codereview.chromium.org/11362182
-
Tom Gallacher authored
Fixes #6835
-
gluxon authored
-
- Jan 10, 2014
-
-
Ryan Graham authored
We want to clear the found domain and the domains after it. Signed-off-by:Trevor Norris <trev.norris@gmail.com>
-
Trevor Norris authored
Now will process all write() that were done on a single tick in a single writev().
-
Jeff Barczewski authored
The RR cluster scheduler replaces the normal StreamWrap handle. Because of this the AsyncListener method failed to be in place when domains were in use. The issue was resolved in 828f1455 by reverting having domains use AsyncListeners. Signed-off-by:
Trevor Norris <trev.norris@gmail.com>
-
Timothy J Fontaine authored
We are in an unrecoverable state if v8 throws a FatalError, actually ask the operating system to dump core in this case. Fixes #6836
-
Trevor Norris authored
Before when an AsyncListener object was created and the "create" callback returned a value, it was necessary to construct a new Object with the same callbacks but add a place for the new storage value. Now, instead, a separate storage array is kept on the context which is used for any return value of the "create" callback. This significantly reduces the number of Objects that need to be created. Also added a flags property to the context to quickly check if a specific callback was available either on the context or on the AsyncListener instance itself. Few other minor changes for readability that were difficult to separate into their own commit. This has not been optimized yet.
-
Trevor Norris authored
This is a slightly modified revert of bc39bdd9. Getting domains to use AsyncListeners became too much of a challenge with many edge cases. While this is still a goal, it will have to be deferred for now until more test coverage can be provided.
-
- Jan 09, 2014
-
-
Fedor Indutny authored
When not specified as a configure flag, and not derived from system configuration, `arm_float_abi` should be set to `'default'`. fix #6789
-
Fedor Indutny authored
Report path to executable and argv on error, stderr is not enough in many cases. fix #6796
-
Fedor Indutny authored
`x86masm.pl` was mistakenly using .486 instruction set, why `cpuid` (and perhaps others) are requiring .686 .
-
Lorenz Leutgeb authored
-
- Jan 08, 2014
-
-
Fedor Indutny authored
-
Timothy J Fontaine authored
-
Ben Noordhuis authored
Forcibly disable -Werror, the old { 'werror': '' } hack in node.gyp no longer works with newer versions of V8. We support a wide range of compilers, it's simply not feasible to squelch all warnings, never mind that the libraries in deps/ are not under our control. Fixes #6817.
-