To update the patch for this feature, I have some comments regarding hno@'s
comment #5:

> 1. URL escaping is already provided by rfc1738_encode().

I'll follow your advise.

> 2. I would prefer redoing the core access controls to return other entities 
> than
> allow/deny to make things like this patch considerably cleaner. There is now
> quite many acl driven directives selecting various things, and all need 
> special
> code today..
>
>   tcp_outgoing_xxx
>   access_log
>   reply_body_max_size
>   always/never_direct could be joined into one if this was available

Should follow this for updating 'internal redirect'?

Perhaps that's easier with squid3 async framework, and would involve quite many
changes on squid2.

Let me know your opinion.

Regards,

-- 
Gonzalo A. Arana

Reply via email to