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

AbstractAddressingEndpointMapping not calling SmartEndpointInterceptor.shouldIntercept while creating invocation chain

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Complete
    • Affects Version/s: 2.2.0.RELEASE
    • Fix Version/s: 2.2.1
    • Component/s: Core
    • Labels:
      None

      Description

      AbstractAddressingEndpointMapping adds all SmartEndpointInterceptors to its SoapEndpointInvocationChain without calling shouldIntercept() on each instance. Contract this to AbstractEndpointMapping which does call shouldIntercept() on each interceptor and only adds ones that return true to its chain.

      AbstractAddressingEndpointMapping should be enhanced to invoke shouldIntercept and only add the necessary SmartEndpointInterceptors to its SoapEndpointInvocationChain.

        Activity

        nstoddar Nate Stoddard created issue -
        Show
        arjen.poutsma Arjen Poutsma added a comment - PR at https://github.com/spring-projects/spring-ws/pull/27
        Show
        arjen.poutsma Arjen Poutsma added a comment - SO question at http://stackoverflow.com/questions/27276302/whats-the-proper-way-of-configuring-a-soapactionsmartendpointinterceptor-along/27291014
        Hide
        gregturn Greg Turnquist added a comment -

        @arjen No problem. Thanks!

        Show
        gregturn Greg Turnquist added a comment - @arjen No problem. Thanks!
        gregturn Greg Turnquist made changes -
        Field Original Value New Value
        Assignee Greg Turnquist [ gregturn ]
        gregturn Greg Turnquist made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        gregturn Greg Turnquist made changes -
        Fix Version/s 2.2.1 [ 14639 ]
        gregturn Greg Turnquist made changes -
        Status In Progress [ 3 ] Resolved [ 5 ]
        Resolution Complete [ 8 ]
        gregturn Greg Turnquist 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
        1d 5h 54m 1 Greg Turnquist 05/Dec/14 12:31 PM
        In Progress In Progress Resolved Resolved
        1m 22s 1 Greg Turnquist 05/Dec/14 12:33 PM
        Resolved Resolved Closed Closed
        109d 21h 57m 1 Greg Turnquist 25/Mar/15 10:30 AM

          People

          • Assignee:
            gregturn Greg Turnquist
            Reporter:
            nstoddar Nate Stoddard
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: