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

Document Spring-WS Security exception handling

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Complete
    • Affects Version/s: 1.5.7, 2.0 M3
    • Fix Version/s: 2.0 RC1, 1.5.10
    • Component/s: Security
    • Labels:
      None

      Description

      In SWS-549, the exceptionHandler property was introduced into the AbstractWSSecurityInterceptor.
      This creates an inconsistency with the overall exceptionhandling architecture in Spring WS.

      Instead of introducing the EndpointExceptionResolver in this Interceptor, would it not be better to just let the exception flow up to the MessageDispatcher, where the resolvers already are configured (with sensible defaults)

      And as this is not documented anywhere unlike the exception resolving in the MessageDispatcher (http://static.springsource.org/spring-ws/sites/1.5/reference/html/server.html#server-endpoint-exception-resolver), it is too easy to miss out on or forget this extra configuration step.

      In my opinion this could be done for both the client and endpoint handleRequest/handleResponse methods by just removing the catch clauses.
      I would be happy to contribute a patch for this if needed.

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        31m 17s 1 Arjen Poutsma 01/Nov/10 3:09 AM
        Resolved Resolved Reopened Reopened
        1h 2m 1 Arjen Poutsma 01/Nov/10 4:11 AM
        Reopened Reopened In Progress In Progress
        8d 21h 37m 1 Arjen Poutsma 10/Nov/10 1:49 AM
        In Progress In Progress Resolved Resolved
        37m 57s 1 Arjen Poutsma 10/Nov/10 2:27 AM
        Resolved Resolved Closed Closed
        541d 4h 36m 1 Arjen Poutsma 04/May/12 7:03 AM

          People

          • Assignee:
            arjen.poutsma Arjen Poutsma
            Reporter:
            hurragutt Paul Nyheim
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - Not Specified
              Not Specified
              Remaining:
              Remaining Estimate - 0d
              0d
              Logged:
              Time Spent - 0.15d
              0.15d