|
From: Lionel B. <lio...@bo...> - 2005-06-23 14:45:29
|
Gianpaolo Del Matto wrote the following on 23.06.2005 16:14 :
>
>-----BEGIN PGP SIGNED MESSAGE-----
>Hash: SHA1
>
>Hello Lionel
>
>My appologies, I am too late with my feedback
>before you release 1.6 :-/
>
>I found the issue with sqlgrey-logparser.pl and
>verified it with the 1.6 release.
>
>Problem was that it would not match anything taken
>from /var/log/maillog.
>I though first it had to be a FreeBSD specific bug,
>maybe from a different syslog format or such.
>
>However it turned out, that the regexp on line 207
>(1.6's logparser) did not match my line because my
>hostname contains dashes.
>
>So I changed the line
>
>m/^(\w{3} [\d ]\d \d\d:\d\d:\d\d) \w+ $self->{programname}: (\w+):
>(.*)$/o
>to
>m/^(\w{3} [\d ]\d
>\d\d:\d\d:\d\d)\s[\S]+\s$self->{programname}:\s(\w+):\s(.*)$/o
>
>This would also work, however I found myself safer using the \s to
>mark the whitespaces...
>m/^(\w{3} [\d ]\d \d\d:\d\d:\d\d) [\S]+ $self->{programname}: (\w+):
>(.*)$/o
>
>This should match any non-whitespace string enclosed by whitespaces
>after
>the initial date string followed by programname.
>
>
>
Thanks, I will release 1.6.1 and 1.7.1 with the fix.
Lionel
|