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
Strip port suffix from Host header if the protocol is known #588
Labels
Comments
sindresorhus
added
enhancement
This change will extend Got features
✭ help wanted ✭
labels
Aug 24, 2018
Wireshark log:
|
Ok, great. We should at least implement the same tests as https://github.com/request/request/pull/2904/files#diff-5db92d37ce5156ccbf7499555bdd0bf9R183 so we don't accidentally regress in the future. |
Will do. |
sindresorhus
pushed a commit
that referenced
this issue
Aug 25, 2018
szmarczak
added a commit
to szmarczak/got
that referenced
this issue
Aug 28, 2018
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
See request/request#2904
The text was updated successfully, but these errors were encountered: