[LOG4J2-2420] RequestContextFilter logging cleanup

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: Log4j-Audit 1.0.0
• Fix Version/s: Log4j-Audit 1.0.1
• Component/s: Log4j-Audit
• Assignee: Unassigned
• Reporter: Andrei Ivanov
• Created: 24/Aug/18 12:24
• Updated: 23/Sep/18 17:51
• Resolved: 22/Sep/18 06:14

Description

The RequestContextFilter logs some details that should be done on DEBUG or TRACE instead of INFO.

INFO  [org.apache.logging.log4j.audit.rest.RequestContextFilter] - Starting request {}/productrepository/2594
INFO  [org.apache.logging.log4j.audit.rest.RequestContextFilter] - Request /productrepository/2594 completed in 0.139279500 seconds

There's even a small bug on the 1st line where concatenation was used.
The calculation of the request duration should also be moved in an if block, to avoid it if it won't be logged.

Same for the Maven plugin, I think it should print the name of the generated classes only on debug.

Issue Links

• links to: GitHub Pull Request #6