Skip to content

Multiple reverse proxy support#484

Closed
b10m wants to merge 3 commits intoPerlDancer:masterfrom
b10m:master
Closed

Multiple reverse proxy support#484
b10m wants to merge 3 commits intoPerlDancer:masterfrom
b10m:master

Conversation

@b10m
Copy link
Contributor

@b10m b10m commented Sep 28, 2013

Apache for one allows the X-Forwarded-Host to be a comma seperated
list rather than a single value. See:
http://httpd.apache.org/docs/2.2/mod/mod_proxy.html#x-headers

In a case of:

[interwebs] -> [external proxy] -> [internal proxy] -> [Dancer]

The X-Forward-Host will look like "domain.example, domain.example". This will then be used in, e.g. redirects:

Location: http://domain.example, domain.example/redirected_page

Menno Blom and others added 3 commits September 28, 2013 19:18
Apache for one allows the X-Forwarded-Host to be a comma seperated
list rather than a single value. See:
http://httpd.apache.org/docs/2.2/mod/mod_proxy.html#x-headers
When HTTP_REFERER is not set (which is a quite likely scenario), dancer
would die with:
    isa check for "referer" failed: undef is not a string!

This forces request->referer to return an empty string, rather than
undef (which is not a Str).
@b10m b10m closed this Nov 13, 2013
@xsawyerx
Copy link
Member

xsawyerx commented Dec 1, 2013

Why was this closed?

@b10m
Copy link
Contributor Author

b10m commented Dec 1, 2013

Restructuring commits. See #503 for the same new pull request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants