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

stream-tester sometimes doesn't read back its response body, maybe? #29

Closed
iameli opened this issue Feb 12, 2020 · 2 comments
Closed

stream-tester sometimes doesn't read back its response body, maybe? #29

iameli opened this issue Feb 12, 2020 · 2 comments

Comments

@iameli
Copy link
Contributor

iameli commented Feb 12, 2020

Forking discussion from internal Discord channel — this is what I've found for at least one 400 response, but perhaps this is just what happened when an instance was restarting or some such?

https://my.papertrailapp.com/groups/16613582/events?focus=1164038057298141210&selected=1164038057298141210

2020-02-11T23:58:42.000Z nginx-ingress-controller-74956b95cf-8vxwp chi 2020/02/11 23:58:42 
[info] 396#396: *217918 client prematurely closed stream: only 0 out of 387468 bytes of request body 
received, client: 10.140.19.8, server: chi-broadcaster-charmander.livepeer-ac.live, request: "POST 
/live/af1e01d8-6fb0-42c3-b7a5-9b20cada2e0c/44.ts HTTP/2.0", host: "chi-broadcaster-
charmander.livepeer-ac.live"
@darkdarkdragon
Copy link
Contributor

@iameli Cool, we hit a bug in Golang's HTTP library 😄
golang/go#32441
There is open PR with the fix https://golang.org/cl/210123 , but it sits there for a while, so it is unknown when it will get to the Golang's release.
Will disable HTTP2 for the stream tester deployment for now, let see how it will go.

@iameli
Copy link
Contributor Author

iameli commented Feb 13, 2020

@darkdarkdragon Yep, this seems to be fixed. Thanks!

@iameli iameli closed this as completed Feb 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants