[WICKET-3917] NPE with RequestLogger enabled and Https

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.5-RC5.1
• Fix Version/s: 1.5-RC6
• Component/s: wicket
• Assignee: Sven Meier
• Reporter: Ivan Vasilev
• Created: 19/Jul/11 09:36
• Updated: 28/Jul/11 20:43
• Resolved: 28/Jul/11 20:43

Description

There is a wicket application with https enabled:

setRootRequestMapper(new HttpsMapper(getRootRequestMapper(), new HttpsConfig()));

The RequestLogger is also enabled:

getRequestLoggerSettings().setRequestLoggerEnabled(true);

The bug can be reproduced by:
1. Try to access the page with "localhost/h" (deliberately excluding the https protocol prefix).
2. A NPE is thrown:

java.lang.NullPointerException
at org.apache.wicket.protocol.http.RequestLogger.getRequestHandlerString(RequestLogger.java:532)
at org.apache.wicket.protocol.http.RequestLogger.getDelegateString(RequestLogger.java:574)
at org.apache.wicket.protocol.http.RequestLogger.getRequestHandlerString(RequestLogger.java:556)
at org.apache.wicket.protocol.http.RequestLogger.logEventTarget(RequestLogger.java:513)
at org.apache.wicket.protocol.http.RequestLoggerRequestCycleListener.registerHandler(RequestLoggerRequestCycleListener.java:119)
at org.apache.wicket.protocol.http.RequestLoggerRequestCycleListener.onRequestHandlerResolved(RequestLoggerRequestCycleListener.java:64)