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

net/rpc: client's DialHTTPPath() treats http CONNECT response incorrectly #38297

Open
AnimusPEXUS opened this issue Apr 7, 2020 · 1 comment
Open

Comments

@AnimusPEXUS
Copy link

@AnimusPEXUS AnimusPEXUS commented Apr 7, 2020

https://tools.ietf.org/html/rfc7231#section-4.3.6 - says: 'Any 2xx (Successful) response indicates that the sender (and all inbound proxies) will switch to tunnel mode immediately after the
blank line that concludes the successful response's header section;'

but DialHTTPPath() requires exactly resp.Status == "200 Connected to Go RPC"

https://github.com/golang/go/blob/master/src/net/rpc/client.go#L258
if err == nil && resp.Status == connected {
https://github.com/golang/go/blob/master/src/net/rpc/server.go#L688
var connected = "200 Connected to Go RPC"

@andybons andybons changed the title net/rpc client's DialHTTPPath() treats http CONNECT response incorrectly net/rpc: client's DialHTTPPath() treats http CONNECT response incorrectly Apr 8, 2020
@andybons
Copy link
Member

@andybons andybons commented Apr 8, 2020

@andybons andybons added this to the Unplanned milestone Apr 8, 2020
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.

None yet
2 participants