Re: [PATCH] Request URI logging for malformed requests

The last patch was not correct...
The temporary buffers are not freed.
Sorry, I should be more careful ...
I commit the fixed patch. I am also posting the fixed patch here.....

On 03/10/2011 12:11 PM, Amos Jeffries wrote:
> On 10/03/11 21:57, Tsantilas Christos wrote:
>> On 03/10/2011 10:18 AM, Amos Jeffries wrote:
>>> On 10/03/11 20:53, Tsantilas Christos wrote:
>>>> If there is not any objection I will commit the last patch to trunk...
>>>>
>>>>
>>>> On 03/07/2011 08:34 PM, Tsantilas Christos wrote:
>>>>> This is the second version of the patch.
>>>>> - The documentation in cf.data.pre fixed
>>>>> - The new rfc1738_do_escape is used
>>>>>
>>>>> Regards,
>>>>> Christos
>>>>
>>>
>>> The final version of your patch has not made it to the mailing list for
>>> audit yet. I'd like to see it before it goes in please.
>>
>> It is the malformed-URI-logging-v2.patch file attached in this mail:
>> http://www.mail-archive.com/squid-dev@squid-cache.org/msg15100.html
>>
>> I am attaching again the patch in this mail.
>
> Thanks. The earlier strangely failed to arrive here.
>
> +1. Looks great.
>
> Amos