Skip to content

/ servo

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

Use NetworkConnector directly to account for replaced hosts #16131

Merged
merged 5 commits into from Mar 26, 2017
Merged

Use NetworkConnector directly to account for replaced hosts #16131

merged 5 commits into from Mar 26, 2017
+150 −159

Commits on Mar 26, 2017

  1. Rework replace_hosts and host_replacement 

    They do less cloning now.
    @nox
    nox committed Mar 26, 2017
    bc5c4fa

  2. Remove some useless Option<T> wrappers from ServoUrl methods

    @nox
    nox committed Mar 26, 2017
    54d37d9

  3. Use NetworkConnector directly to account for replaced hosts 

    This let us remove a hack where we had to replace the host in the request URL.
    @nox
    nox committed Mar 26, 2017
    92c4a43

  4. Use Url::port instead of Url::port_or_known_default to set Host 

    AFAIK, if there is no explicit port in the request URL, there should be
no explicit port in the Host header.
    @nox
    nox committed Mar 26, 2017
    b096bf4

  5. Put websocket_loader::init first in its module

    @nox
    nox committed Mar 26, 2017
    Loading status checks…
    8796a82
You can’t perform that action at this time.