Shannon -jj Behrens wrote:
> On Thu, Jul 3, 2008 at 2:05 PM, Ian Bicking <[EMAIL PROTECTED]> wrote:
>> Jonathan Vanasco wrote:
>>> On Jul 3, 4:42 pm, Wichert Akkerman <[EMAIL PROTECTED]> wrote:
>>>> Possibly better:
>>>>
>>>>   request.environ.get("X_FORWARDED_FOR", request.environ["REMOTE_ADDR"])
>>> maybe someone can make a middleware or pylons patch + config setting
>>> that migrates X_FORWARDED_FOR to REMOTE_ADDR
>>>
>>> this is the one i maintain for mod_perl
>>>      
>>> http://search.cpan.org/dist/Apache2-xForwardedFor/lib/Apache2/xForwardedFor.pm
>> I was going to say that PrefixMiddleware already did this, but
>> apparently it didn't.  It does now in Paste Deploy trunk.
> 
> I remember submitting a pretty decent patch for this a long time ago.

I think we had philosophical differences about whether to read the 
headers in attributes, or rewrite the environment, and I was on the 
rewrite side.  And... wasn't it X-Forwarded-Server?

-- 
Ian Bicking : [EMAIL PROTECTED] : http://blog.ianbicking.org

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"pylons-discuss" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at 
http://groups.google.com/group/pylons-discuss?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to