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

x/net/http2: write on closed buffer #17543

Closed
wjdp opened this issue Oct 21, 2016 · 4 comments
Closed

x/net/http2: write on closed buffer #17543

wjdp opened this issue Oct 21, 2016 · 4 comments
Labels
FrozenDueToAge WaitingForInfo Issue is not actionable because of missing required information, which needs to be provided.

Comments

@wjdp
Copy link

wjdp commented Oct 21, 2016

What version of Go are you using (go version)?

go version go1.6.2 linux/amd64

What operating system and processor architecture are you using (go env)?

GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/will/local/htmltest"
GORACE=""
GOROOT="/usr/lib/go-1.6"
GOTOOLDIR="/usr/lib/go-1.6/pkg/tool/linux_amd64"
GO15VENDOREXPERIMENT="1"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0"
CXX="g++"
CGO_ENABLED="1"

What did you do?

Perform a sequence of http.Get-s to the https://photos.smugmug.com domain for jpg images.

resp, err := http.Get(url)
if err != nil {
  log.Println(err.Error())
} else {
  log.Println(url)
}

Output: https://gist.github.com/wjdp/ba545c7ae1925e3be1b10f05425abeda

What did you expect to see?

No errors, 200 response codes

What did you see instead?

Majority were error free, a random subset return write on closed buffer (see http2.log in gist)

When running with GODEBUG=http2client=0 no errors were seen (see http1.log in gist)

@bradfitz
Copy link
Contributor

Can you try Go 1.7.3?

@bradfitz bradfitz added the WaitingForInfo Issue is not actionable because of missing required information, which needs to be provided. label Oct 22, 2016
@bradfitz
Copy link
Contributor

Also, can you share a more complete program? Your sample code doesn't seem to match the output.

@bradfitz
Copy link
Contributor

Looks like this was a dup of #16974

@wjdp
Copy link
Author

wjdp commented Nov 22, 2016

Sorry for not getting back to you on this! If this is still an issue in 1.7 I'll run into it again in a few weeks, will post here I do.

@golang golang locked and limited conversation to collaborators Nov 22, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
FrozenDueToAge WaitingForInfo Issue is not actionable because of missing required information, which needs to be provided.
Projects
None yet
Development

No branches or pull requests

3 participants