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

h2c/dialer: continue to use bufio.Reader for reading #11

Merged
merged 1 commit into from Feb 26, 2018
Merged

h2c/dialer: continue to use bufio.Reader for reading #11

merged 1 commit into from Feb 26, 2018

Conversation

danp
Copy link
Collaborator

@danp danp commented Feb 26, 2018

Once a bufio.Reader is set up for use with net/http.ReadResponse, it
must continue to be used on the returned net.Conn. Wrap up the
bufio.Reader and the raw net.Conn we have and have Read on that read
from the bufio.Reader.


This change is Reviewable

Once a bufio.Reader is set up for use with net/http.ReadResponse, it
must continue to be used on the returned net.Conn. Wrap up the
bufio.Reader and the raw net.Conn we have and have Read on that read
from the bufio.Reader.
Copy link
Owner

@lstoll lstoll left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the contribution!

@lstoll lstoll merged commit 9334f17 into lstoll:master Feb 26, 2018
@danp danp deleted the dial-buffered-conn branch February 26, 2018 20:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants