[Bug 204] New: dbparser docs not clear for @NUMBER@
https://bugzilla.balabit.com/show_bug.cgi?id=204 Summary: dbparser docs not clear for @NUMBER@ Product: syslog-ng Version: 3.3.x Platform: PC OS/Version: Linux Status: NEW Severity: normal Priority: unspecified Component: packaging AssignedTo: folti@balabit.hu ReportedBy: erempel@uvic.ca Type of the Report: --- Estimated Hours: 0.0 the @NUMBER@ parser should state that a leading minus sign is handled for non-hex numbers and that commas are not permitted. Alternatively, making @NUMBER@ accept commas would be nice, or allowing parameters for additional characters, although comma would be the only one I could think of. -- Configure bugmail: https://bugzilla.balabit.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are watching all bug changes.
https://bugzilla.balabit.com/show_bug.cgi?id=204 Balazs Scheidler <bazsi@balabit.hu> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |bazsi@balabit.hu --- Comment #1 from Balazs Scheidler <bazsi@balabit.hu> 2012-10-13 10:33:53 --- comma is locale dependant. some languages use ',' as thousands separator and '.' as decimal separator others use '.' for thousands separator, and comma as decimal separator. Yet others (like Hungarian) use space for thousands separator and comma for decimals. This is a big can of worms and I think it would be quite problematic when the input is a comma separated list of numbers. I'd rather allow a @REGEXP@ parser instead, which when used deeply in the pattern may be good enough performance wise. -- Configure bugmail: https://bugzilla.balabit.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are watching all bug changes.
https://bugzilla.balabit.com/show_bug.cgi?id=204 --- Comment #2 from Balazs Scheidler <bazsi@balabit.hu> 2012-10-13 13:32:06 --- And here goes the @PCRE@ parser (against 3.4 this time, but probably can be backported) commit 4900a9cc84b7e8395c00ae2210fdff721dab89ea Author: Balazs Scheidler <bazsi@balabit.hu> Date: Sat Oct 13 13:29:50 2012 +0200 db-parser: added @PCRE@ parser This patch implements the @PCRE@ parser in db-parser() patterns. With this you can use the power of regexps once the heavylifting of identifying the potential patterns happened by the radix implementation. Syntax: @PCRE:name:regexp@ As the name suggests regexp is a Perl-Compatible Regular Expression as implemented by the PCRE library. Signed-off-by: Balazs Scheidler <bazsi@balabit.hu> -- Configure bugmail: https://bugzilla.balabit.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are watching all bug changes.
https://bugzilla.balabit.com/show_bug.cgi?id=204 --- Comment #3 from Evan Rempel <erempel@uvic.ca> 2012-10-13 19:17:27 --- I agree with what you have said. The documentation still needs to be updated to state that the leading hyphen is accepted for non-hex numbers. -- Configure bugmail: https://bugzilla.balabit.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are watching all bug changes.
https://bugzilla.balabit.com/show_bug.cgi?id=204 Robert Fekete <frobert@balabit.hu> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |frobert@balabit.hu --- Comment #4 from Robert Fekete <frobert@balabit.hu> 2012-10-15 16:59:20 --- Thank you very much for the report, we'll update the docs as soon as we can. -- Configure bugmail: https://bugzilla.balabit.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are watching all bug changes.
https://bugzilla.balabit.com/show_bug.cgi?id=204 Robert Fekete <frobert@balabit.hu> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution| |FIXED Status|NEW |RESOLVED --- Comment #5 from Robert Fekete <frobert@balabit.hu> 2012-11-10 21:06:45 --- Should be fixed in the current 3.4 draft guide (see https://lists.balabit.hu/pipermail/syslog-ng/2012-November/019666.html). I hope to backport it to 3.3 as well, but that's not likely to happen in the near future. Regards, Robert -- Configure bugmail: https://bugzilla.balabit.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are watching all bug changes.
participants (1)
-
bugzilla＠bugzilla.balabit.com