[SLING-12133] Include time spend in authentication in request log

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: Engine 2.15.8
• Component/s: Engine
• Assignee: Carsten Ziegeler
• Reporter: Carsten Ziegeler
• Created: 08/Nov/23 07:09
• Updated: 17/Nov/23 13:44
• Resolved: 10/Nov/23 15:13

Description

The request logging is currently triggered by a servlet filter. Filters run after authentication and therefore the time spent in authentication is not included. In addition time spent in anything before authentication (like preprocessors) is not included either.
We should change RequestLoggerFilter from a filter to an early running Preprocessor. However,currently the RequestLoggerFilter has two purposes - one is the logging and the other one is preparing request and response objects. We need to split that functionality and keep it in the filter - Preprocessors run for every http context, not just Sling so we need to avoid that this code runs outside of Sling's context.