[squid-dev] [PATCH] Do not hide important/critical messages

Alex Rousskov rousskov at measurement-factory.com
Mon Apr 11 14:59:36 UTC 2016


On 04/09/2016 10:42 PM, Amos Jeffries wrote:
> On 29/03/2016 12:44 p.m., Alex Rousskov wrote:
>> unpatched Squid console only says:
>>
>>   2016/03/27 14:19:48.297| SECURITY ALERT: By user agent:
>>   2016/03/27 14:19:48.297| SECURITY ALERT: on URL: dut70.test:443
>>
>> A patched Squid produces the expected three console lines:
>>
>>   2016/03/27 15:25:42| SECURITY ALERT: Host header forgery detected...
>>   2016/03/27 15:25:42| SECURITY ALERT: By user agent:
>>   2016/03/27 15:25:42| SECURITY ALERT: on URL: dut70.test:443

>> If this v3.5 patch is accepted in principle, I hope somebody volunteers
>> a trunk port (which should not be difficult).


> This appears to be just a polished implementation of the intended
> debugs() original design. So in principle its already acceptible.

> To get into v3.5 it does need to go in through v4 first though.


Sounds good to me. Does anybody want to volunteer to port these fixes to
trunk?


Thank you,

Alex.



More information about the squid-dev mailing list