| Commit message (Collapse) | Author | Age | Files | Lines |
|\
| |
| |
| |
| | |
Conflicts:
node.gyp
|
| | |
|
| | |
|
| | |
|
| |
| |
| |
| | |
fix #6679
|
| | |
|
| |
| |
| |
| |
| |
| |
| | |
Spawn's arguments were documented to be optional, as they are for the
other similar child_process APIs, but the code was missing. Result was
`child_process.spawn('node', {})` errored when calling slice() on an
Object, now it behaves as the documentation said it would.
|
| | |
|
| | |
|
| |
| |
| |
| |
| | |
`a === a.sort()` is always true because Array#sort() does an in-place
sort. Make a copy of the array first.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Make the HMAC digest method configurable. Update crypto.pbkdf2() and
crypto.pbkdf2Sync() to take an extra, optional digest argument.
Before this commit, SHA-1 (admittedly the most common method) was used
exclusively.
Fixes #6553.
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Now that the context stores the active execution stack, and because
removeAsyncListener() always removed the AsyncListener from the queue
and the stack, there's no need to keep a stack around anymore. Instead
the active asyncQueue and the currentContext is able to handle it all.
Signed-off-by: Forrest L Norvell <ogd@aoaioxxysz.net>
|
| |
| |
| |
| |
| |
| | |
Should have been included with 60fcc11 as it is the same type of fix.
Signed-off-by: Forrest L Norvell <ogd@aoaioxxysz.net>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Performance gains are ~4x (~1.5us), but still much slower than a naive
approach. There is some duplicate work done between join(), normalize()
and normalizeArray() so additional optimizations are possible.
Note that this only improves the POSIX implementation.
Thanks to @isaacs and @othiym23 for helping with this optimization.
Signed-off-by: Trevor Norris <trev.norris@gmail.com>
|
| |
| |
| |
| |
| |
| |
| |
| | |
After one of OpenSSL updates we have stopped accepting PEM private keys
and certificates that doesn't end with a newline (`\n`) character.
Handle this regression in `crypto.js` to make less trouble to our users.
fix #6892
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Do not throw in internal C++ methods, that clobbers logic and may lead
to the situations, where both exception was thrown and the value was
returned (via `args.GetReturnValue().Set()`). That doesn't play nicely
with v8.
fix #6912
|
| |
| |
| |
| |
| | |
Before this commit `RunAtExit` and `env->Dispose()` were never reached,
because `EmitExit` was always colling `exit`.
|
| |
| |
| |
| | |
fix #6910
|
| |
| |
| |
| |
| | |
The previous timeout was too short for certain execution conditions
(Windows, debug build, first execution).
|
| |
| |
| |
| |
| | |
On Windows debug builds, the test was failing because the timeout
was too short.
|
| |
| |
| |
| |
| |
| |
| | |
Now that process.createAsyncListener() returns a unique object instance
it is no longer necessary to compare the uid's of the objects.
Signed-off-by: Timothy J Fontaine <tjfontaine@gmail.com>
|
| |
| |
| |
| |
| |
| |
| |
| | |
It was possible that the same AL instance was run twice if it were both
attached to the currentContext then again added to the new asyncQueue
generated for the new stack.
Signed-off-by: Timothy J Fontaine <tjfontaine@gmail.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The ability to add/remove an AsyncListener to an object after its
creation was an artifact of trying to get AL working with the domain
module. Now that is no longer necessary and other features are going to
be implemented that would be affected by this functionality. So the code
will be removed for now to simplify the implementation process.
In the future this code will likely be reintroduced, but after some
other more important matters have been addressed.
None of this functionality was documented, as is was meant specifically
for domain specific implementation work arounds.
Signed-off-by: Timothy J Fontaine <tjfontaine@gmail.com>
|
| |
| |
| |
| |
| |
| | |
All C++ code should be using `args.GetIsolate()` or `env->isolate()`.
Using static `node_isolate` var limits possible future functionality
(like multi-isolate support).
|
| |
| |
| |
| |
| |
| |
| |
| | |
This test was originally intended to guard against regressions for
commit 16b59cbc74c8fe2f8b30f3af4c2f885b7bfb6030.
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.
|
| |
| |
| |
| |
| | |
The test needs a little more time to run so that it passes for all
builds (eg: Windows, debug)
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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
|
| | |
|
| | |
|
| |
| |
| |
| | |
fix #6880
|
| |
| |
| |
| | |
fix #6889
|
| |
| |
| |
| |
| | |
As of commit 39aa894, SSLv2 support is disabled by default. Update
the documentation to reflect that.
|
| |
| |
| |
| |
| |
| |
| | |
* Make it clear that ECDHE-RSA-AES128-SHA256 and AES128-GCM-SHA256 are
TLS v1.2 ciphers.
* Note that RC4 is under suspicion.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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>
|
| |
| |
| |
| |
| |
| |
| | |
Not all ciphers require the presence of a certificate. Remove the
check in lib/_tls_wrap.js.
Fixes #6887.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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.
|
|\ \
| |/
| |
| |
| | |
Conflicts:
lib/domain.js
|
| |
| |
| |
| |
| |
| |
| | |
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>
|
| | |
|
| |
| |
| |
| | |
fix #6859
|
| | |
|
| |
| |
| |
| |
| | |
Killing the worker without ensuring the socket was closed
was causing intermittent ECONNRESET errors.
|
|\ \
| |/ |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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']})
|
|\ \
| |/
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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
|
| |
| |
| |
| |
| | |
Killing the debuggee without first closing the socket can result
in an ECONNRESET error.
|
| |
| |
| |
| |
| |
| |
| | |
If node was compiled with --no-ifaddrs to support older operating
systems, don't throw instead simply return an empty object
Fixes #6846
|
| |
| |
| |
| |
| |
| |
| | |
Keep the number of descriptors below
DescriptorArray::kMaxNumberOfDescriptors even for accessors
Review URL: https://codereview.chromium.org/11362182
|
| | |
|
| |
| |
| |
| |
| |
| |
| | |
We are in an unrecoverable state if v8 throws a FatalError, actually
ask the operating system to dump core in this case.
Fixes #6836
|