Uploaded image for project: 'Spring Web Services'
  1. Spring Web Services
  2. SWS-465

Optional WS-Addressing request headers being treated as mandatory

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.5.5
    • Fix Version/s: 1.5.6
    • Component/s: Core
    • Labels:
      None
    • Environment:
      Windows XP, JDK 1.5

      Description

      Using SimpleActionEndpointMapping results in wsa:MessageAddressingHeaderRequired errors being returned from a request unless that request specifies
      a wsa:To and wsa:MessageID header. According to the WS-Addressing standard, only the wsa:Action header is mandatory.

      The problem appears to lie in the hasRequiredProperties() method of org.springframework.ws.soap.addressing.MessageAddressingProperties. This method insists on

      1. wsa:To being present
      2. wsa:MessageID being present if either ReplyTo or FaultTo have been specified.

      In the case of (2) above I was able to trigger the fault by omitting all of the MessageID, ReplyTo and FaultTo headers in my request, so it may be that the server-side stack has inserted a default "anonymous endpoint" value into one or both of the latter and that is triggering the error.

        Activity

        wabrit Alan Boshier created issue -
        arjen.poutsma Arjen Poutsma made changes -
        Field Original Value New Value
        Fix Version/s 1.5.6 [ 11141 ]
        arjen.poutsma Arjen Poutsma made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        Hide
        arjen.poutsma Arjen Poutsma added a comment - - edited

        Unfortunately, the required elements of the Message Addressing Properties are version-dependent. For instance, the http://www.w3.org/Submission/2004/SUBM-ws-addressing-20040810 version of the spec does require Action and To. So I guess i'll move the hasRequiredProperties() code over to AddressingVersion.

        Show
        arjen.poutsma Arjen Poutsma added a comment - - edited Unfortunately, the required elements of the Message Addressing Properties are version-dependent. For instance, the http://www.w3.org/Submission/2004/SUBM-ws-addressing-20040810 version of the spec does require Action and To. So I guess i'll move the hasRequiredProperties() code over to AddressingVersion.
        Hide
        arjen.poutsma Arjen Poutsma added a comment - - edited

        With regard to the ReplyTo header, I believe that it defaults to "anonymous" when not given (in WS-Addressing 1.0, see http://www.w3.org/TR/ws-addr-core/#formreplymsg). So even when no ReplyTo or FaultTo is given in the request, a MessageID header is still required. The only way around this is to specify a "none" ReplyTo header.

        Show
        arjen.poutsma Arjen Poutsma added a comment - - edited With regard to the ReplyTo header, I believe that it defaults to "anonymous" when not given (in WS-Addressing 1.0, see http://www.w3.org/TR/ws-addr-core/#formreplymsg ). So even when no ReplyTo or FaultTo is given in the request, a MessageID header is still required. The only way around this is to specify a "none" ReplyTo header.
        Hide
        arjen.poutsma Arjen Poutsma added a comment -

        I believe this issue has been fixed. Could you please try a recent snapshot as of tomorrow, and see if it works for you?

        Show
        arjen.poutsma Arjen Poutsma added a comment - I believe this issue has been fixed. Could you please try a recent snapshot as of tomorrow, and see if it works for you?
        arjen.poutsma Arjen Poutsma made changes -
        Resolution Fixed [ 1 ]
        Status In Progress [ 3 ] Resolved [ 5 ]
        Hide
        arjen.poutsma Arjen Poutsma added a comment -

        Closing old issues

        Show
        arjen.poutsma Arjen Poutsma added a comment - Closing old issues
        arjen.poutsma Arjen Poutsma made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open In Progress In Progress
        2h 32m 1 Arjen Poutsma 19/Dec/08 1:56 AM
        In Progress In Progress Resolved Resolved
        37d 20h 57m 1 Arjen Poutsma 25/Jan/09 10:53 PM
        Resolved Resolved Closed Closed
        1194d 8h 9m 1 Arjen Poutsma 04/May/12 7:03 AM

          People

          • Assignee:
            arjen.poutsma Arjen Poutsma
            Reporter:
            wabrit Alan Boshier
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: