You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
uv_write() for pipes on Windows does not always call callback #1729
Hi everyone! I'm currently debugging some of my Node work, and I'm pretty sure uv_write() callbacks for Windows pipes don't always get called (even though the other side of the pipe fully received the data).
I'll try to get a pure libuv reproduction together, but the easiest way to reproduce would be something along the lines of adding a fprintf(stderr, "foobar\n"); to Node's LibuvStreamWrap::AfterWrite(), then running a test like test\parallel\test-cluster-cwd.js -- it writes data, the other side receives it, and it does use uv_write() for writing, but there's no extra output.
@bzoz I don’t think that’s a solution, no… the writes should still be reported back as finished, even without an explicit disconnect, right? That’s true for all other kinds of libuv streams
I mean, it looks like the process exits before callbacks can be called. If you drop the process.disconnect() altogether, the fprints also start to appear (but the test never finishes).
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
Activity
bzoz commentedon Feb 20, 2018
@addaleax if you change slaves
process.sendtothe
fprints start to appear. Does this makes sense, or do you still think there is some issue here?addaleax commentedon Feb 20, 2018
@bzoz I don’t think that’s a solution, no… the writes should still be reported back as finished, even without an explicit disconnect, right? That’s true for all other kinds of libuv streams
bzoz commentedon Feb 20, 2018
I mean, it looks like the process exits before callbacks can be called. If you drop the
process.disconnect()altogether, thefprints also start to appear (but the test never finishes).stale commentedon Aug 10, 2019
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.