- Mar 11, 2014
-
-
Brian White authored
-
- Mar 04, 2014
-
-
Brian White authored
-
Brian White authored
-
- Mar 03, 2014
-
-
Fedor Indutny authored
Conflicts: configure lib/_stream_readable.js lib/http.js src/node_dtrace.cc
-
- Mar 02, 2014
-
-
Fedor Indutny authored
Since `smalloc.h` is included in a `node_buffer.h`, including private headers in it is strictly prohibited. fix #7206
-
Fedor Indutny authored
When `setImmediate(cb)` is called in `beforeExit` event handler the consequent `uv_run(..., UV_RUN_NOWAIT)` may return `0`, even if there was some active handles at start. Fixes simple/test-beforeexit-event.js.
-
- Feb 28, 2014
-
-
Fedor Indutny authored
fix #7155
-
Sam Roberts authored
-
Ben Noordhuis authored
Unlike the 'exit' event, this event allows the user to schedule more work and thereby postpone the exit. That also means that the 'beforeExit' event may be emitted many times, see the attached test case for an example. Refs #6305.
-
Saúl Ibarra Corretgé authored
uv_loop_new and uv_loop_delete are considered deprecated now.
-
- Feb 27, 2014
-
-
Saúl Ibarra Corretgé authored
When the exit callback is fired for the child process we should close the handle associated with it.
-
Timothy J Fontaine authored
libuv gyp builds now require you to define the library disposition (static or shared). Also, libuv now supports vectored IO for file system reads and writes, update to those function signatures
-
Timothy J Fontaine authored
-
Nathan Rajlich authored
Between `ClientRequest` and `Agent`. The circular require was doing weird things at load time, like making the `globalAgent` property be `undefined` from within the context of the "_http_client" module. Removing the circular dependency completely fixes this. This commit effectively removes the undocumented `Agent#request()` and `Agent#get()` functions.
-
Nathan Rajlich authored
Don't invoke the `agent.requst()` or `agent.get()` functions directly. Instead, use the public API and pass the agent instance in as the `agent` option.
-
Nathan Rajlich authored
For the `request()` and `get()` functions. I could never really understand why these two functions go through agent first... Especially since the user could be passing `agent: false` or a different Agent instance completely, in which `globalAgent` will be completely bypassed. Moved the relevant logic from `Agent#request()` into the `ClientRequest` constructor. Incidentally, this commit fixes #7012 (which was the original intent of this commit).
-
Nathan Rajlich authored
See #7012.
-
Nathan Rajlich authored
This is just the test portion from #7012 / #7189, but targetted for the v0.10 branch.
-
Nathan Rajlich authored
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.
-
Trevor Norris authored
Rely on defined order of operations by closing the server and destorying the client socket, instead of when setImmediate fires
-
Alexis Campailla authored
Expose `setBlocking` on Pipe's and if a pipe is being created for stdio on windows then make the pipes blocking. This fixes test-stream2-stderr-sync.js on Windows. Fixes #3584
-
Alexis Campailla authored
One test case in test-stream2-stderr-sync.js was creating a TTY object using an undocumented constructor and passing in fd 2. However, this is running in a child process and fd 2 is actually a pipe, not a TTY. The constructor fails on Windows and causes the handle type to be left uninitialized, which later causes an assert to fail. On Unix, the constructor fails to retrieve the windows size but unlike on Windows, it just leaves the size fields undefined and continues with initializing the stream type, yielding a semi-usable object. I could make the Windows version match Unix behavior, but it seems to me that the test is relying on an implementation detail of an undocumented API, and the Unix behavior is not necessarily more correct than the Windows one. Thus it makes more sense to remove this test.
-
- Feb 26, 2014
-
-
Maxwell Krohn authored
NodeBIO::Gets was reading off the end of a buffer if it didn't find a "\n" before the EOF. This behavior was causing X509 certificates passed to `https.Agent` via the "ca" option to be silently discarded. It also was causing improper parsing of certs and keys passed to https.Agent, but those problems were worked around in cdde9a38. Backed out workaround in `lib/crypto.js` from ccde9a3, which now isn't needed. But keep the test introduced in that commit, which tests properly for this bug. This bug was first introduced in a58f93f Gist containing test code, bisection log, and notes: https://gist.github.com/maxtaco/9211605
-
Alexis Campailla authored
The linker was optimizing the static variables that were supposed to trigger module initialization. I am making them non-static, and dllexport so that they don't get optimized away. Fixes #7116
-
Alexis Campailla authored
Fix a bug introduced by 3da36fe0 of a missed early return of a handle that needed to be passed to listen. Fixes test-net-listen-fd0.js on Windows
-
Timothy J Fontaine authored
This ends up being too difficult to test across different deployments
-
Timothy J Fontaine authored
We were trying too hard to connect, and getting timeouts instead of the refusals, slow down how hard we try.
-
Timothy J Fontaine authored
-
Timothy J Fontaine authored
-
Timothy J Fontaine authored
-
Timothy J Fontaine authored
-
Timothy J Fontaine authored
-
Mike Pennisi authored
Ensure that the behavior of `assert.deepEqual` does not depend on argument ordering when comparing an `arguments` object with a non-`arguments` object.
-
- Feb 25, 2014
-
-
Timothy J Fontaine authored
When calling out to the openssl client handle the child closing and returning EPIPE on writes
-
Timothy J Fontaine authored
ESERVFAIL is also an acceptable error code when failing to resolve a domain.
-
Timothy J Fontaine authored
This test is particularly pathological, and requires a ton of time to run, we need to find a better way to manage it but in general this path is fairly safe these days.
-
Timothy J Fontaine authored
bufferSize is now a getter that shows all that has not been acknowledged by the os, as well as in the buffer state. The test is only looking to verify the js verified state.
-
Timothy J Fontaine authored
watch file will now generate an empty event when the file doesn't exist initially
-
Timothy J Fontaine authored
Scheduling of next ticks from within the next tick handler will result in a tight execution loop where a timer cannot break into. This test was invalid
-
Alexis Campailla authored
Environment doesn't have ThrowException, we meant isolate here. Introduced in commit 75adde07.
-