| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| |
|
|
|
|
|
|
|
| |
* Documentation upgrades
* Fix glob bug which prevents proper README publishing
* node-gyp upgrade to 0.13
* Documentation updates
* Add --save-exact to save an exact dep (instead of a range)
* alias 't' to 'test'
|
| |
|
|
|
|
|
|
|
| |
Turn off -Werror when building V8, it hits -Werror=unused-local-typedefs
with g++ 4.8. The warning itself is harmless so don't abort the build.
This was originally implemented in commit d2ab314e back in 2011 but the
build process has gone through a few iterations since then, that change
no longer works.
|
| |
|
|
| |
The argument of process.stdin's readable event handler is not used.
|
| |
|
|
|
|
|
| |
`crypto.getCiphers()` and `crypto.getHashes()` should prefer lower-case
variants of names, but should not introduce them.
fix #7282
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Fix the following valgrind warning:
Conditional jump or move depends on uninitialised value(s)
at 0x7D64E7: v8::internal::GlobalHandles::IterateAllRootsWithClassIds(v8::internal::ObjectVisitor*) (global-handles.cc:613)
by 0x94DCDC: v8::internal::NativeObjectsExplorer::FillRetainedObjects() (profile-generator.cc:2849)
# etc.
This was fixed upstream in r12903 and released in 3.15.2 but that commit
was never back-ported to the 3.14 branch that node.js v0.10 uses.
The code itself works okay; this commit simply shuffles the clauses in
an `if` statement to check that the node is in use before checking its
class id (which is uninitialized if the node is not in use.)
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When sending a socket to a child process via IPC pipe,
`child_process.js` picks a raw UV handle from `_handle` property, sends
it, and assigns `null` to the property. Sending the same socket twice
was resulting in a runtime error, since we weren't handling the empty
`_handle` case.
In case of `null` `_handle` we should send just a plain text message
as passed it was passed to `.send()` and ignore the handle, letting
users handle such cases themselves instead of throwing the error at
runtime.
fix #5469
|
| |
|
|
|
| |
Added test-cluster-send-handle-twice.js testing to send a handle
twice to the parent process.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
It's currently not really possible to compile native add-ons with
-fvisibility=hidden because that also hides the struct containing
the module definition.
The NODE_MODULE() and NODE_MODULE_DECL() macros are structured in
a way that makes it impossible to add a visibility attribute manually
so there is no escape hatch there.
That's why this commit adds an explicit visibility attribute to
the module definition. It doesn't help with node.js releases that
are already out there but at least it improves the situation going
forward.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
If two timers run on the same tick, and the first timer uses a domain,
and then catches an exception and disposes of the domain, then the
second timer never runs. (And even if the first timer does not dispose
of the domain, the second timer could run under the wrong domain.)
This happens because timer.js uses "process.nextTick()" to schedule
continued processing of the timers for that tick. However, there was
an exception inside a domain, then "process.nextTick()" runs under
the domain of the first timer function, and will do nothing if
the domain has been disposed.
To avoid this, we temporarily save the value of "process.domain"
before calling nextTick so that it does not run inside any domain.
|
| |
|
|
|
|
|
|
| |
Previously if you cached process.nextTick and then require('domain')
subsequent nextTick() calls would not be caught because enqueued
functions were taking the wrong path. This keeps nextTick to a single
function reference and changes the implementation details after domain
has been required.
|
| |
|
|
|
| |
This is just the test portion from #7012 / #7189,
but targetted for the v0.10 branch.
|
| |
|
|
|
|
|
|
|
|
|
| |
This makes it so that the user may pass in a
`createConnection()` option, and they don't have
to pass `agent: false` at the same time.
Also adding a test for the `createConnection` option,
since none was in place before.
See #7014.
|
| |
|
|
|
|
| |
Ensure that the behavior of `assert.deepEqual` does not depend on
argument ordering when comparing an `arguments` object with a
non-`arguments` object.
|
| | |
|
| |
|
|
| |
Update assert.throws() and assert.doesNotThrow() docs
|
| | |
|
| | |
|
| | |
|
| |
|
|
|
| |
`node.d` should use `psinfo.d` instead of `procfs.d` and have statically
defined architecture on FreeBSD.
|
| | |
|
| | |
|
| |\ |
|
| | |
| |
| |
| | |
This were accidentally moved during the website refactor
|
| |/
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* uv: Upgrade to v0.10.25 (Timothy J Fontaine)
* npm: upgrade to 1.4.3 (isaacs)
* v8: support compiling with VS2013 (Fedor Indutny)
* cares: backport TXT parsing fix (Fedor Indutny)
* crypto: throw on SignFinal failure (Fedor Indutny)
* crypto: update root certificates (Ben Noordhuis)
* debugger: Fix breakpoint not showing after restart (Farid Neshat)
* fs: make unwatchFile() insensitive to path (iamdoron)
* net: do not re-emit stream errors (Fedor Indutny)
* net: make Socket destroy() re-entrance safe (Jun Ma)
* net: reset `endEmitted` on reconnect (Fedor Indutny)
* node: do not close stdio implicitly (Fedor Indutny)
* zlib: avoid assertion in close (Fedor Indutny)
|
| | |
|
| |
|
|
|
| |
Makes clear that the behaviour of stdout is blocking
in Linux/Unix even when they refer to pipes.
|
| | |
|
| | |
|
| | |
|
| |
|
|
|
|
|
|
|
|
| |
The reason this wasn't working was because after restart, when restoring
breakpoints the scripts wasn't loaded, so the breakpoint.script was
undefined. As a fix I added another check to use breakpoint.scriptReq
instead of breakpoint.script, which is the same except when the
breakpoint is a function.
fixes #7027
|
| | |
|
| |
|
|
|
|
| |
zlib should not crash in `close()` if the write is still in progress.
fix #7101
|
| |
|
|
|
|
|
| |
The man page was accidentally removed in 37376de for the website
refactor, bring it back.
Fixes #7117
|
| |
|
|
|
|
|
| |
- Removes 'npm publish -f'
- Documentation
- Bug-fixes
- Update license etc to refer to npm, Inc. rather than @isaacs personally
|
| |
|
|
|
| |
The website will no longer be living in the source repository instead
it can be found at http://github.com/joyent/node-website
|
| |
|
|
| |
fix #7074
|
| |
|
|
| |
fix #7015
|
| |
|
|
|
| |
Consolidate buffer bounds checking logic into Buffer namespace and use
it consistently throughout the source.
|
| | |
|
| | |
|
| |
|
|
| |
This reverts commit 56e80a37e0df0d131d3a3ad6426d52f887ef8e94.
|
| |
|
|
| |
This reverts commit 2ee86c624ecd6b9dbaad10989143325fc64778cd.
|
| |
|
|
|
|
| |
Don't use argument as callback if it's not a valid callback function.
Throw a valid exception instead explaining the issue. Adds to #7070
("DNS — Throw meaningful error(s)").
|
| |
|
|
|
|
| |
Mitigat C++-land assertion error, add test accordingly.
fix #7070
|
| | |
|
| | |
|
| |
|
|
|
|
|
| |
References for err.signal and err.code should be error.signal and
error.code.
Fixes joyent/node#6862
|
| |
|
|
|
|
|
|
| |
Update the list of root certificates in src/node_root_certs.h with
tools/mk-ca-bundle.pl and update src/node_crypto.cc to make use of
the new format.
Fixes #6013.
|
| |
|
|
| |
`path.extname` returns only the last extension
|
| |
|
|
|
| |
Add enough arguments to `NODE_NET_SOCKET_READ()` and
`NODE_NET_SOCKET_WRITE()` stubs.
|
