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.

Sign up for GitHub

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

Jump to bottom

#515, #2894 Strip port suffix from Host header if the protocol is known. #2904

Merged
merged 5 commits into from Jul 18, 2018
Merged

#515, #2894 Strip port suffix from Host header if the protocol is known. #2904

merged 5 commits into from Jul 18, 2018

Commits on Apr 4, 2018

  1. Strip port suffix from Host header if the protocol is known. 

    This partially revert ff6d6c6, and still works for IPv6 addresses as well.
    @paambaati
    paambaati committed Apr 4, 2018
    Copy the full SHA
    41cc05f View commit details
    Browse the repository at this point in the history

Commits on Apr 10, 2018

  1. Port is a string out of url.parse(). 

    See https://nodejs.org/api/url.html#url_url_port
    @paambaati
    paambaati committed Apr 10, 2018
    Copy the full SHA
    1e79977 View commit details
    Browse the repository at this point in the history

  2. Port is a string out of url.parse(). 

    See https://nodejs.org/api/url.html#url_url_port
    @paambaati
    paambaati committed Apr 10, 2018
    Copy the full SHA
    ab2d63d View commit details
    Browse the repository at this point in the history

  3. Add tests for the new Host header changes.

    @paambaati
    paambaati committed Apr 10, 2018
    Copy the full SHA
    237231e View commit details
    Browse the repository at this point in the history

Commits on May 16, 2018

  1. Merge branch 'master' into master

    @paambaati
    paambaati committed May 16, 2018
    Copy the full SHA
    d149dd7 View commit details
    Browse the repository at this point in the history