-
Notifications
You must be signed in to change notification settings - Fork 300
ERR_STREAM_WRITE_AFTER_END #121
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
2nd time encountering it, same error.
|
Hey @joshxyzhimself I think it's fine keeping the issue open even though you don't have a repro. Maybe others can chime in with ideas / a repro. Is this with v1? |
Hi @porsager, thanks just reopened it. Currently using |
Encountered this again locally with
|
I'm starting to think this one is partly caused by me using
|
Don't have a repro too and not sure if it helps, but I got that error twice since setting the |
Hey @naumf, just noticed that my |
In my case, the error didn't show up since setting the |
I'll try that. The error message also implies that we're writing on the socket while it's ending, could be the case. |
Sure thing. Given the difficulty of reproducing the bug, this was the easier "fix" (workaround). It might not work for everyone though, as someone might need the |
Note: links below are at Usage of Line 184 in 0e91989
According to nodejs docs[1], calling the stream.write() method after calling stream.end() will raise an error. It seems that Line 318 in 0e91989
calls the Line 331 in 0e91989
which raises the error below which leads to so, maybe check if the socket is ending already before we call any References |
That would prevent the error from showing up, but I think we need to know what data is lost, should we queue it and retry writing to the socket once re-connected etc. |
it seems like setting |
Just encountered this while testing, will close for now as I'm not sure what caused it and I don't have a repro yet.
The text was updated successfully, but these errors were encountered: