#592 Log severity for all audit events, not just warnings

Future
closed
nobody
5
2013-11-17
2012-02-22
Richard Barnett
No

DefaultLogger.addError() only logs AuditEvent.severityLevel if it's WARNING.
This makes it hard to find events of severity ERROR in eg mvn checkstyle:check output.
Log severityLevel in all cases. Patch attached (based on 5.3).

(I know that DefaultLogger is an AuditListener & it's possible to add new AuditListeners, but is it possible to replace DefaultLogger? I also know that checkstyle:check allows you to control the severity threshold, though it doesn't seem possible from the command-line.)

Discussion

  • Patch to DefaultLogger 5.3

     
    Attachments
  • ...and of course that patch breaks 2/3 of your unit tests :-(
    If you think this feature request is worthwhile I'll fix them.

     
  • Ivan Sopov
    Ivan Sopov
    2013-11-17

    • status: open --> closed
    • Group: --> Future