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: Client doesn't send body until ExpectContinueTimeout expires #49677

Open
mhr3 opened this issue Nov 19, 2021 · 2 comments · May be fixed by golang/net#118
Open

x/net/http2: Client doesn't send body until ExpectContinueTimeout expires #49677

mhr3 opened this issue Nov 19, 2021 · 2 comments · May be fixed by golang/net#118

Comments

@mhr3
Copy link

@mhr3 mhr3 commented Nov 19, 2021

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

$ go version
go version go1.17.1 linux/amd64

Does this issue reproduce with the latest release?

This is a problem with x/net master

What did you do?

Set a non-zero timeout for ExpectContinueTimeout on http.Transport and sent a POST request with Expect: 100-continue over h2.

What did you expect to see?

The request uploads the data and finishes.

What did you see instead?

The data doesn't start uploading until after the specified timeout expires, adding extra latency.

@mhr3 mhr3 linked a pull request that will close this issue Nov 19, 2021
@heschi heschi added this to the Go1.18 milestone Nov 22, 2021
@heschi
Copy link
Contributor

@heschi heschi commented Nov 22, 2021

cc @neild

Loading

@gopherbot
Copy link

@gopherbot gopherbot commented Nov 29, 2021

Change https://golang.org/cl/363914 mentions this issue: net/http2: Fix handling of expect continue

Loading

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

3 participants