(RADIATOR) <Log SQL> on Radiator 3.6

[Mike McCauley]

Hello Jacob,


On Sat, 20 Sep 2003 09:34 am, Jacob Leaver wrote:
> I was putting together a troubleshooting tool for our staff, and was
> looking to use <Log SQL> to help with that.
> I was having a devil of a time getting it to work for me, so I took a look
> at the code.
> Line 54(ish) of LogSQL.pm reads:
>
>     # Catch recursion
>     return if $in_log ++;
>
> Would this not always be true?   

No, it is a post-increment. The value of the statement is the value before 
incrementing. LogSQL really does work.

Cheers.


> Perhaps this needs to read more like:
>
>     # Catch recursion
>     return if $in_log > 1;
>     $in_log ++;
>
> At least, when I updated the code to read as above, logs started happening.
>
> Jacob
>
>
> ===
> Archive at http://www.open.com.au/archives/radiator/
> Announcements on radiator-announce at open.com.au
> To unsubscribe, email 'majordomo at open.com.au' with
> 'unsubscribe radiator' in the body of the message.

-- 
Mike McCauley                               mikem at open.com.au
Open System Consultants Pty. Ltd            Unix, Perl, Motif, C++, WWW
24 Bateman St Hampton, VIC 3188 Australia   http://www.open.com.au
Phone +61 3 9598-0985                       Fax   +61 3 9598-0955

Radiator: the most portable, flexible and configurable RADIUS server 
anywhere. SQL, proxy, DBM, files, LDAP, NIS+, password, NT, Emerald, 
Platypus, Freeside, TACACS+, PAM, external, Active Directory, EAP, TLS, 
TTLS, PEAP etc on Unix, Windows, MacOS etc.

===
Archive at http://www.open.com.au/archives/radiator/
Announcements on radiator-announce at open.com.au
To unsubscribe, email 'majordomo at open.com.au' with
'unsubscribe radiator' in the body of the message.