Skip to content
This repository was archived by the owner on Sep 11, 2020. It is now read-only.

plumbing: transport/http, Close http.Body reader when needed#657

Merged
mcuadros merged 1 commit intosrc-d:masterfrom
ajnavarro:fix/close-http-response
Nov 24, 2017
Merged

plumbing: transport/http, Close http.Body reader when needed#657
mcuadros merged 1 commit intosrc-d:masterfrom
ajnavarro:fix/close-http-response

Conversation

@ajnavarro
Copy link
Contributor

If we don't do that, all the goroutines opened by the Body Reader are never closed, causing a 'fatal: morestack on g0' error.

@ajnavarro
Copy link
Contributor Author

Related issue: src-d/borges#186

@ajnavarro ajnavarro changed the title Close http.Body reader when needed [transport.http] Close http.Body reader when needed Nov 24, 2017
@mcuadros mcuadros changed the title [transport.http] Close http.Body reader when needed plumbing: transport/http, Close http.Body reader when needed Nov 24, 2017
If we don't do that, all the goroutines opened by the Body Reader are never closed, causing a 'fatal: morestack on g0' error.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants