[CXF-5504] AbstractLoggingInterceptor.getMessageLogger throws NPE when MultipleEndpointObserver is used

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.5
• Fix Version/s: 3.0
• Component/s: Core
• Estimated Complexity: Unknown
• Assignee: Daniel Kulp
• Reporter: Vadim Beilin
• Created: 20/Jan/14 17:40
• Updated: 10/Apr/17 19:04
• Resolved: 05/Apr/17 19:05

Description

The method AbstractLoggingInterceptor#getMessageLogger(Message) that was introduced by CXF-3888 assumes that message.getExchange().getEndpoint() is not null.

This assumption does not work if one uses org.apache.cxf.transport.MultipleEndpointObserver as the message observer, leading to NPE:

java.lang.NullPointerException: null
	at org.apache.cxf.interceptor.AbstractLoggingInterceptor.getMessageLogger(AbstractLoggingInterceptor.java:78) ~[cxf-2.5.9.jar:2.5.9]
	at org.apache.cxf.interceptor.LoggingInInterceptor.handleMessage(LoggingInInterceptor.java:73) ~[cxf-2.5.9.jar:2.5.9]
	at org.apache.cxf.phase.PhaseInterceptorChain.doIntercept(PhaseInterceptorChain.java:263) ~[cxf-2.5.9.jar:2.5.9]
	at org.apache.cxf.transport.MultipleEndpointObserver.onMessage(MultipleEndpointObserver.java:98) [cxf-2.5.9.jar:2.5.9]
...

Would it be possible to check first if endpoint is present?