Conversation
Uses existing plack middlewares (ReverseProxy and ReverseProxyPath) to handle all request header modification when operating behing a proxy. Allows for variants (eg HTTP_X_FORWARDED_PROTOCOL) supported by Dancer(2) but not from those existing ReverseProxy middleware. Special cases REQUEST_BASE header to work with ReverseProxyPath so proxies from/to non-root paths "just work"(tm). (Alternate to #571.) As a middleware, devs get more flexability as to where to apply it; they can use Plack::Builder to wrap this around their app as well as further path/header altering middleware. This could be released as a seperate package; its not Dancer2 specific.
set behind_proxy => 1; We now 'use' the middleware directly rather than letting Plack::Builder load it for us. May allow earlier version of Plack to be used as a dependency.
xsawyerx
changed the title
cromedome
left a comment
cromedome
left a comment
There was a problem hiding this comment.
I really like how this came together. We removed a test in t/request.t and we don't have any tests for lib/Dancer2/Middleware/BehindProxy.pm. Asking here so we can capture this for our future selves: do we not need to test this going forward because of the changes, or should we re-implement the test we removed in another fashion? That's about my only concern moving forward with this.
| @@ -0,0 +1,81 @@ | |||
| use strict; | |||
There was a problem hiding this comment.
I'm not sure this test file is supposed to be part of this PR. It's causing the Appveyor tests to fail.
xsawyerx
force-pushed
the
pr/behind_proxy_middleware-rebased
branch
from
9d4d838 to
2faca00
Compare
| @@ -0,0 +1,58 @@ | |||
| package Dancer2::Middleware::BehindProxy; | |||
| # ABSTRACT: Support Dancer2 apps when operating behing a reverse proxy | |||
|
I still think this is a better way to handle the psgi It's taken some time, but I concur with the comments on the original PR about the namespace; something like |
3 participants
This is a rebased branch of #590. :)
It's only for review against the original PR. Please do not merge it.