Skip to content
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

Seeing failed to get reader and to write msg errors #191

Closed
gandaldf opened this issue Feb 17, 2020 · 6 comments · Fixed by #193
Closed

Seeing failed to get reader and to write msg errors #191

gandaldf opened this issue Feb 17, 2020 · 6 comments · Fixed by #193
Labels

Comments

@gandaldf
Copy link

gandaldf commented Feb 17, 2020

Hi, I just updated your library to the last version; I tried a basic example with two Websocket connection and get these errors in my Go debug console:

failed to get reader: received close frame: status = StatusGoingAway and reason = ""
on page load (when HTML page try to open the Websocket connection).

and:

write: failed to write msg: failed to close writer: failed to write fin frame: failed to flush: write tcp 127.0.0.1:7070->127.0.0.1:61615: write: broken pipe
on writing (when my Go code try to send a message to the HTML page).

These errors pop out with Firefox (73) and Chrome (79), but I think they are mostly "warnings" because everything seems to work.
Anyway, I never had these errors in older versions of your library and my frontend code is still the same.
Can you investigate, please?

@gandaldf gandaldf changed the title Broken pipe error Warnings: failed to get reader and to write msg Feb 17, 2020
@nhooyr
Copy link
Contributor

nhooyr commented Feb 17, 2020

StatusGoingAway means the WebSocket was closed by the browser as the window was closed.

The second error means a Write was in progress when the connection was closed.

Both seem normal to me.

@nhooyr
Copy link
Contributor

nhooyr commented Feb 17, 2020

I'd recommend ignoring read/write errors as there's really nothing you can do and they're unlikely to be useful in debugging.

@nhooyr nhooyr closed this as completed Feb 17, 2020
@nhooyr
Copy link
Contributor

nhooyr commented Feb 17, 2020

It is definitely bizarre that you only get these errors with the latest version though.

If you downgrade, what happens?

@nhooyr nhooyr reopened this Feb 17, 2020
@nhooyr
Copy link
Contributor

nhooyr commented Feb 17, 2020

Also what version were you on last?

@nhooyr nhooyr changed the title Warnings: failed to get reader and to write msg Seeing failed to get reader and to write msg errors Feb 17, 2020
@nhooyr
Copy link
Contributor

nhooyr commented Feb 20, 2020

Feel free to reopen if you have any further issues.

@nhooyr nhooyr closed this as completed Feb 20, 2020
@nhooyr
Copy link
Contributor

nhooyr commented Feb 20, 2020

Another user has reported this, fixing. @coadler

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants