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/http2: init canonHeader before use it #33287

Open
detailyang opened this issue Jul 26, 2019 · 3 comments

Comments

@detailyang
Copy link

commented Jul 26, 2019

No description provided.

@gopherbot

This comment has been minimized.

Copy link

commented Jul 26, 2019

Change https://golang.org/cl/187658 mentions this issue: x/net/http2: init canonHeader at first

@agnivade

This comment has been minimized.

Copy link
Member

commented Jul 26, 2019

Hi @detailyang, thank you for sending these changes. However, for small fixes like these, there is no need to open an issue, you can directly send a change. Just something to keep in mind for future :)

@detailyang

This comment has been minimized.

Copy link
Author

commented Jul 26, 2019

Gotcha :) I will keep in mind.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.