[SWS-576] sentMessageTracingLogger skips request message headerer content Created: 13/Oct/09  Updated: 04/May/12  Resolved: 10/Nov/09

Status: Closed
Project: Spring Web Services
Component/s: Core
Affects Version/s: 1.5.3, 1.5.8
Fix Version/s: 1.5.9

Type: Bug Priority: Minor
Reporter: Jakub Milkiewicz Assignee: Arjen Poutsma
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: 0d
Time Spent: 0.3h
Original Estimate: Not Specified

Widnows XP, Websphere, SAAJ 1.3 (sun implementation), MTOM enabled JAXB2 marshaller.


In org.springframework.ws.server.MessageDispatcher class sentMessageTracingLogger with trace level does not log the same request message as the one being logged by receivedMessageTracingLogger.
I've noticed that request message having <SOAP-ENV:Header> with wss4j stuff is properly logged by receivedMessageTracingLogger on trace level:

Received request [<SOAP-ENV:Envelope xmlns:SOAP-ENV="http://schemas.xmlsoap.org/soap/envelope/">
<wsse:Security xmlns:wsse="http://docs.oasis-open.org/wss/2004/01/oasis-200401-wss-wssecurity-secext-1.0.xsd" SOAP-ENV:mustUnderstand="1">
<wsse:UsernameToken xmlns:wsu="http://docs.oasis-open.org/wss/2004/01/oasis-200401-wss-wssecurity-utility-1.0.xsd" wsu:Id="UsernameToken-21398464">
<wsse:Password Type="http://docs.oasis-open.org/wss/2004/01/oasis-200401-wss-username-token-profile-1.0#PasswordText">a701050a4e03ef54319c319c9a06a895</wsse:Password>

Unfortunately sentMessageTracingLogger strips out content of <SOAP-ENV:Header> when logging request message:

Sent response [------=_Part_18_463346590.1255441172093
Content-Type: application/xop+xml; type="text/xml"; charset=utf-8

<SOAP-ENV:Envelope xmlns:SOAP-ENV="http://schemas.xmlsoap.org/soap/envelope/">
------=_Part_18_463346590.1255441172093--] for request [<SOAP-ENV:Envelope xmlns:SOAP-ENV="http://schemas.xmlsoap.org/soap/envelope/"><SOAP-ENV:Header>

It seems that wss4j headers are processed by WSS4j Interceptor and then somehow removed from requestMessage. Actually had no time to dig it further and check if custom headres are also stripped out.
Anyway i think that sentMessageTracingLogger should log original client request message.
I am attaching patch version of MessageDispatcher

Comment by Arjen Poutsma [ 10/Nov/09 ]


Comment by Arjen Poutsma [ 10/Nov/09 ]

Indeed, the WSS4J interceptors change the request content, getting rid of security headers. This conforms to the SOAP spec, which says that a processing node should remove the headers that belong to it after processing.

That said, this issue should be fixed now: we keep a reference to the request content as it came in.

Comment by Arjen Poutsma [ 19/Nov/09 ]


I think you forgot to attach your patch to this issue. We have some issues with our JIRA installation a few weeks ago, perhaps we lost your patch in the process. But let me know if the solution currently in svn is not working for you. A shame we didn't get to chat at Devoxx.



Comment by Jakub Milkiewicz [ 23/Nov/09 ]

I am quite sure i successfully attached patch but it does not matter now since it is nearly identical to yours.
I believe your code works as expected but I haven't tried it, since I am still using SWS 1.5.5 ( with my original patch applied).
It is all because of https://jira.springsource.org/browse/SWS-374 and Websphere. Actually i haven't checked myself if newer (above 1.5.5) versions of SWS work on Websphere but according to comments by Sebastien Dubois these do not work. I am planning to check it myself this week.
Back to devoxx: really ashame we had no time to chat but actually i have not seen you at all in Antwerp! I had a chat with Peter Meijer on Tuesday and have seen him quite regularly till Thursday afternoon at SpringSource booth but I haven't seen you at all during whole conference! During one of launch breaks I have met Tareq and have just asked him a couple of SWS related questions. I hope we will have a chat/beer during next SpringOne/geecon.

Comment by Arjen Poutsma [ 04/May/12 ]

Closing old issues

Generated at Sun Oct 21 23:56:49 UTC 2018 using JIRA 7.9.2#79002-sha1:3bb15b68ecd99a30eb364c4c1a393359bcad6278.