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

2016-06-15 Thread Eliezer Croitoru
@lists.squid-cache.org Subject: Re: [squid-dev] [PATCH] Do not hide important/critical messages On 12/04/2016 2:59 a.m., Alex Rousskov wrote: > 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: >>

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

2016-06-15 Thread Amos Jeffries
On 12/04/2016 2:59 a.m., Alex Rousskov wrote: > 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

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

2016-05-20 Thread Amos Jeffries
On 20/05/2016 3:36 a.m., Eduard Bagdasaryan wrote: > Hello, > > 2016-05-19 16:54 GMT+03:00 Amos Jeffries : >> Thanks. But ... I just tried to apply this and got lots of rejects in >> Debugs.h and debug.cc on the latest trunk. > > Reworked against the latest r14673, sorry.

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

2016-05-19 Thread Amos Jeffries
On 19/05/2016 10:01 p.m., Eduard Bagdasaryan wrote: > Hello, > > This is a trunk port for Alex's v3.5 reentrant debugging fix > . > Thanks. But ... I just tried to apply this and got lots of rejects in Debugs.h and

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

2016-05-19 Thread Eduard Bagdasaryan
Hello, This is a trunk port for Alex's v3.5 reentrant debugging fix . Regards, Eduard. Do not allow low-level debugging to hide important/critical messages. Removed debugs() side effects that inadvertently resulted in

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

2016-04-11 Thread Alex Rousskov
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

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

2016-04-09 Thread Amos Jeffries
On 29/03/2016 12:44 p.m., Alex Rousskov wrote: > Hello, > > While working on Squid bug 4465 (Header forgery detection leads to > crash), I noticed that Squid may hide important/critical debugs() > messages from the admin if those messages are assembled using code that > also uses debugs().

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

2016-03-28 Thread Alex Rousskov
Hello, While working on Squid bug 4465 (Header forgery detection leads to crash), I noticed that Squid may hide important/critical debugs() messages from the admin if those messages are assembled using code that also uses debugs(). For example, unpatched Squid console only says: 2016/03/27