[Catalyst] Re: [Catalyst-dev] Proxy patches

[Marlon Bailey]

X-Forwarded-Port: 1234; should be X-Forwarded-Host-Port: 1234 per RFC.
The reasoning for this is: X-Forwarded-Port is already being heavily
used in things such as mod_proxy and Python's twisted web, etc..etc.. to
mean the client's remote port.  These differ in situations where you
have something like a Pound box feeding Perlbal for ssl requests, the
Perlbal Proxy will be on port 80, but the client's remote will be 143.  

_Marlon_

On Sun, 2007-06-17 at 18:17 -0500, Jonathan Rockway wrote:
> On Sunday 17 June 2007 02:17:32 pm Dave Rolsky wrote:
> > The last round had one bug in it, in that it broke tests in
> > t/live_component_controller_args.t that expected the path to include a
> > doubled slash. I'm not sure this is a sane test (a doubled slash probably
> > should not be relevant), but I don't want to get too sidetracked.
> >
> > Also, per jrockway's IRC request, I've separated the patches by distro.
> 
> Thanks.  All of these were applied as r6477.  Yay for "svk push --lump" :)
> 
> To those wanting to test, the patches are in branches ending 
> with "proxystuff", like branches/Catalyst-Runtime-proxystuff.  I've 
> personally tested the new code with perlbal + dev server, and it's working 
> wonderfully with this perlbal config:
> 
>   CREATE POOL myapp
>     POOL myapp ADD 127.0.0.1:3010
>     POOL myapp ADD 127.0.0.1:3011
>     POOL myapp ADD 127.0.0.1:3012
>     POOL myapp ADD 127.0.0.1:3013
>   
>   CREATE SERVICE balancer
>     SET listen          = 0.0.0.0:1234
>     SET role            = reverse_proxy
>     SET pool            = myapp
>     SET persist_client  = on
>     SET persist_backend = on
>   ENABLE balancer
>   HEADER balancer insert X-Forwarded-Host: foo.jrock.us
>   HEADER balancer insert X-Forwarded-Port: 1234
> 
> And of course, it works fine without a proxy in front too :)
> 
> Thanks, Dave!
> 
> Regards,
> Jonathan Rockway
> -- 
> package JAPH;use Catalyst qw/-Debug/;($;=JAPH)->config(name => do {
> $,.=reverse qw[Jonathan tsu rehton lre rekca Rockway][$_].[split //,
> ";$;"]->[$_].q; ;for 1..4;$,=~s;^.;;;$,});$;->setup;
> 
> _______________________________________________
> List: Catalyst at lists.rawmode.org
> Listinfo: http://lists.rawmode.org/mailman/listinfo/catalyst
> Searchable archive: http://www.mail-archive.com/catalyst@lists.rawmode.org/
> Dev site: http://dev.catalyst.perl.org/
> 


-- 
This message has been scanned for viruses and
dangerous content by MailScanner, and is
believed to be clean.