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

net/http: add CONNECT bidi example #28178

Open
bradfitz opened this Issue Oct 12, 2018 · 2 comments

Comments

Projects
None yet
2 participants
@bradfitz
Member

bradfitz commented Oct 12, 2018

https://go-review.googlesource.com/c/go/+/123156 is removing some documentation from http.Transport that says not to use CONNECT requests with Transport because it's adding CONNECT support.

We should also add examples. CL 123156 has a test which is close at least, but could be cleaned up and be made less unit-testy.

@bradfitz bradfitz added this to the Go1.12 milestone Oct 12, 2018

@bradfitz bradfitz self-assigned this Oct 12, 2018

@fengzixu

This comment has been minimized.

fengzixu commented Oct 14, 2018

I would like to handle it. Could you give this issue to me? @bradfitz

@bradfitz

This comment has been minimized.

Member

bradfitz commented Oct 15, 2018

Go for it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment