summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge remote-tracking branch 'upstream/v0.10'merge-testTimothy J Fontaine2014-01-22189-5728/+1281
|\ | | | | | | | | Conflicts: node.gyp
| * gyp: fix non-ninja buildFedor Indutny2014-01-201-1/+1
| |
| * npm: Upgrade to v1.3.24isaacs2014-01-19183-5712/+1190
| |
| * blog: nodejs v0.12 roadmap updateTimothy J Fontaine2014-01-161-0/+52
| |
| * gyp: fix `ninja` build on linuxFedor Indutny2014-01-162-6/+13
| | | | | | | | fix #6679
| * doc: clarify Windows signal sending emulationSam Roberts2014-01-161-4/+10
| |
| * child_process: fix spawn() optional argumentsSam Roberts2014-01-161-2/+10
| | | | | | | | | | | | | | 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.
| * doc: describe child_process.fork() silent optionSam Roberts2014-01-161-2/+4
| |
| * doc: child_process.execFile arguments are optionalSam Roberts2014-01-161-1/+1
| |
* | test: fix array sorting bugBen Noordhuis2014-01-221-1/+4
| | | | | | | | | | `a === a.sort()` is always true because Array#sort() does an in-place sort. Make a copy of the array first.
* | crypto: support custom pbkdf2 digest methodsBen Noordhuis2014-01-224-20/+86
| | | | | | | | | | | | | | | | | | | | 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.
* | node: remove asyncStackTrevor Norris2014-01-211-36/+12
| | | | | | | | | | | | | | | | | | 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>
* | node: only run same AL once on errorTrevor Norris2014-01-212-1/+57
| | | | | | | | | | | | Should have been included with 60fcc11 as it is the same type of fix. Signed-off-by: Forrest L Norvell <ogd@aoaioxxysz.net>
* | path: improve POSIX path.join() performanceJo Liss2014-01-211-9/+22
| | | | | | | | | | | | | | | | | | | | | | | | 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>
* | crypto: add newline to cert and key if not presentFedor Indutny2014-01-222-3/+87
| | | | | | | | | | | | | | | | 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
* | crypto: throw only in direct C++ methodsFedor Indutny2014-01-222-121/+158
| | | | | | | | | | | | | | | | | | 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
* | node: `EmitExit` should not call `exit()`Fedor Indutny2014-01-222-4/+7
| | | | | | | | | | Before this commit `RunAtExit` and `env->Dispose()` were never reached, because `EmitExit` was always colling `exit`.
* | addons: build and test examplesFedor Indutny2014-01-2210-117/+277
| | | | | | | | fix #6910
* | text: give more time to test-next-tick-error-spinAlexis Campailla2014-01-211-1/+1
| | | | | | | | | | The previous timeout was too short for certain execution conditions (Windows, debug build, first execution).
* | test: give test-net-GH-5504 more time to runAlexis Campailla2014-01-211-1/+1
| | | | | | | | | | On Windows debug builds, the test was failing because the timeout was too short.
* | node: compare AsyncListener instances, not uid'sTrevor Norris2014-01-211-3/+3
| | | | | | | | | | | | | | 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>
* | node: ensure same AL inst only runs onceTrevor Norris2014-01-212-2/+51
| | | | | | | | | | | | | | | | 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>
* | async_wrap/timers: remove Add/RemoveAsyncListenerTrevor Norris2014-01-2112-485/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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>
* | crypto: remove most of the `node_isolate` usesFedor Indutny2014-01-201-115/+116
| | | | | | | | | | | | All C++ code should be using `args.GetIsolate()` or `env->isolate()`. Using static `node_isolate` var limits possible future functionality (like multi-isolate support).
* | test: relax timing in test-http-exit-delayAlexis Campailla2014-01-201-7/+8
| | | | | | | | | | | | | | | | 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.
* | test: debug-signal-cluster increase timeoutsAlexis Campailla2014-01-201-2/+2
| | | | | | | | | | The test needs a little more time to run so that it passes for all builds (eg: Windows, debug)
* | debug client: connect after child is readyAlexis Campailla2014-01-202-10/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | test: move debugger repl into own sectionTimothy J Fontaine2014-01-2012-2/+18
| |
* | test: refactor to use common testcfgTimothy J Fontaine2014-01-205-519/+152
| |
* | test: ignore tests when built without OpenSSL CLIFedor Indutny2014-01-2012-21/+56
| | | | | | | | fix #6880
* | lib: introduce `.setMaxSendFragment(size)`Fedor Indutny2014-01-205-0/+112
| | | | | | | | fix #6889
* | doc: tls: note that SSLv2 is disabled by defaultBen Noordhuis2014-01-201-3/+4
| | | | | | | | | | As of commit 39aa894, SSLv2 support is disabled by default. Update the documentation to reflect that.
* | doc: tls: clarify server cipher listBen Noordhuis2014-01-201-3/+9
| | | | | | | | | | | | | | * Make it clear that ECDHE-RSA-AES128-SHA256 and AES128-GCM-SHA256 are TLS v1.2 ciphers. * Note that RC4 is under suspicion.
* | src: don't mark addon_register_func as dllimportBen Noordhuis2014-01-171-2/+2
| | | | | | | | | | | | | | | | | | | | 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>
* | tls: make cert/pfx optional in tls.createServer()Ben Noordhuis2014-01-172-13/+6
| | | | | | | | | | | | | | Not all ciphers require the presence of a certificate. Remove the check in lib/_tls_wrap.js. Fixes #6887.
* | tls: show human-readable error messagesBen Noordhuis2014-01-174-28/+72
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
* | Merge remote-tracking branch 'origin/v0.10'Trevor Norris2014-01-1512-63/+210
|\ \ | |/ | | | | | | Conflicts: lib/domain.js
| * domains: exit() only affects active domainsRyan Graham2014-01-152-5/+41
| | | | | | | | | | | | | | 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>
| * blog: TJ is the new node core project leadisaacs2014-01-151-0/+54
| |
| * gyp: fix build with python 2.6Fedor Indutny2014-01-131-1/+2
| | | | | | | | fix #6859
| * deps: update gyp to 1eae492bFedor Indutny2014-01-138-56/+113
| |
* | test: terminate gracefully in cluster-net-sendAlexis Campailla2014-01-131-0/+3
| | | | | | | | | | Killing the worker without ensuring the socket was closed was causing intermittent ECONNRESET errors.
* | Merge remote-tracking branch 'upstream/v0.10'Timothy J Fontaine2014-01-131-1/+3
|\ \ | |/
| * doc: streams must be open to be passed to childSam Roberts2014-01-131-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | 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']})
* | Merge remote-tracking branch 'upstream/v0.10'Timothy J Fontaine2014-01-13223-444/+749
|\ \ | |/ | | | | | | | | | | | | | | | | | | 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
| * test: close debug client in test-debugger-clientAlexis Campailla2014-01-131-3/+8
| | | | | | | | | | Killing the debuggee without first closing the socket can result in an ECONNRESET error.
| * src: return empty set on ENOSYS for interfacesTimothy J Fontaine2014-01-121-2/+5
| | | | | | | | | | | | | | If node was compiled with --no-ifaddrs to support older operating systems, don't throw instead simply return an empty object Fixes #6846
| * v8: backport codereview.chromium.org/11362182svenpanne@chromium.org2014-01-101-1/+3
| | | | | | | | | | | | | | Keep the number of descriptors below DescriptorArray::kMaxNumberOfDescriptors even for accessors Review URL: https://codereview.chromium.org/11362182
| * doc: Fix argument typo in SimpleProtocol examplegluxon2014-01-101-1/+1
| |
| * src: OnFatalError handler must abort()Timothy J Fontaine2014-01-092-1/+40
| | | | | | | | | | | | | | We are in an unrecoverable state if v8 throws a FatalError, actually ask the operating system to dump core in this case. Fixes #6836