Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • N node-http-proxy
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 482
    • Issues 482
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 102
    • Merge requests 102
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • http ... PARTY!
  • node-http-proxy
  • Merge requests
  • !544

Fix for Issue#543 (name-based virtual host target)

  • Review changes

  • Download
  • Email patches
  • Plain diff
Closed Administrator requested to merge github/fork/marmor/patch-1 into caronte Jan 07, 2014
  • Overview 14
  • Commits 1
  • Pipelines 0
  • Changes 1

Created by: marmor

The "Host" header field that was created and sent by the proxy to the backends, was wrong, because it had been copied from the original headers that were received by the proxy.

This caused many problems; for example, when the backend web server served several websites on the same IP/port ("name-based virtual-host"), which is very common, a wrong Host header caused the request to fail or to bring the wrong site.

The patch overwrites the Host field by the real host/port of the backend.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: github/fork/marmor/patch-1