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

HttpComponentsMessageSender inintuitive constructor

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1.3
    • Fix Version/s: 2.1.4
    • Component/s: Core
    • Labels:
      None

      Description

      HttpComponentsMessageSender has two constructors, with and without HttpClient, and the one with HttpClient omits adding a SoapRemoveHeaderInterceptor.

      This breaks in org.apache.http.protocol.RequestContent with
      new ProtocolException("Content-Length header already present"),
      but only in some containers (oc4j) and not in others (jetty, eclipe); this makes the issue hard to debug.

      Adding a note to the javadoc would help the developer pick the proper constructor to avoid this issue.

      git pull request coming up.

        Issue Links

          Activity

          ekonijn Erik van Konijnenburg created issue -
          Hide
          arjen.poutsma Arjen Poutsma added a comment -
          Show
          arjen.poutsma Arjen Poutsma added a comment - Git pull request at https://github.com/SpringSource/spring-ws/pull/5
          arjen.poutsma Arjen Poutsma made changes -
          Field Original Value New Value
          Assignee Arjen Poutsma [ arjen.poutsma ]
          arjen.poutsma Arjen Poutsma made changes -
          Fix Version/s 2.1.4 [ 14119 ]
          arjen.poutsma Arjen Poutsma made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          arjen.poutsma Arjen Poutsma made changes -
          Link This issue relates to SWS-838 [ SWS-838 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Resolved Resolved
          10d 21h 44m 1 Arjen Poutsma 03/Jul/13 3:08 AM

            People

            • Assignee:
              arjen.poutsma Arjen Poutsma
              Reporter:
              ekonijn Erik van Konijnenburg
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: