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

Comments

@wjdp
Copy link

@wjdp 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

This comment has been minimized.

Copy link
Contributor

@bradfitz bradfitz commented Oct 22, 2016

Can you try Go 1.7.3?

@bradfitz

This comment has been minimized.

Copy link
Contributor

@bradfitz bradfitz commented Oct 23, 2016

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

@bradfitz

This comment has been minimized.

Copy link
Contributor

@bradfitz bradfitz commented Nov 22, 2016

Looks like this was a dup of #16974

@bradfitz bradfitz closed this Nov 22, 2016
@wjdp

This comment has been minimized.

Copy link
Author

@wjdp 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.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.