Gisle Aas wrote:
> 
> Looks good, but to me it feels like the correct place for this hack is
> in the LWP::Protocol::http module.  Not that I actually expect us
> start proxying over something else than HTTP anytime soon, but just in
> case.  The LWP design allow any protocol module to proxy.

Ah, yes.  I hadn't gotten far enough to grok the system architecture. 
The
funny thing is that I wanted to poke around and see how this was handled
for password protected pages :)

> This is the patch I checked in:

Great!  Those of us stuck behind auth-proxies thank you :)

I got one report that the $p_auth scalar should be unescaped also.  His
username has embedded '\'.

> Thanks!

Happy to be of use.

> Regards,
> Gisle

John

Reply via email to