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

Server error on client close. #1062

Closed
HeinOldewageRetro opened this issue Feb 3, 2017 · 4 comments
Closed

Server error on client close. #1062

HeinOldewageRetro opened this issue Feb 3, 2017 · 4 comments

Comments

@HeinOldewageRetro
Copy link

When running the helloworld example, the client executes successfully and without error.
The server gives an error :

transport: http2Server.HandleStreams failed to read frame: read tcp [::1]:50051->[::1]:58520: wsarecv: An existing connection was forcibly closed by the remote host.

This only happens when the client shuts down.

This might be fixed by issue #460

@ghost
Copy link

ghost commented Apr 12, 2017

Yes, and this seems only to happen under Windows. If I cross compile it to Linux it works without error.

@tamird
Copy link
Contributor

tamird commented Apr 12, 2017

Probably related to https://go-review.googlesource.com/c/39490/.

@hsaliak hsaliak added the bug label May 17, 2017
@fho
Copy link
Contributor

fho commented Jun 28, 2017

@linjohn also happens on Linux when I establish a connection and then call con.Close()
transport: http2Server.HandleStreams failed to read frame: read tcp [..]: read: connection reset by peer

@MakMukhi
Copy link
Contributor

MakMukhi commented Jun 28, 2017

This is not a bug but a log spam: there's a goroutine on the server that reads on the connection synchronously, when the connection breaks(con.Close()), this gorutine logs an error out. Now the connection breaking might be completely valid (not an error, like in this case), the goroutine logs it out anyway.
We have a PR that'll take care of this by adding verbosity levels to log.
I'm going to go ahead and close this issue since there're already several issues tracking the same thing. Please track the aforementioned PR for updates.

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

No branches or pull requests

6 participants