Petals CDK

Remove or refactor Exceptions around Fault

Details

  • Type: Task Task
  • Status: Resolved Resolved
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 5.4.3
  • Fix Version/s: 5.5.0
  • Component/s: Utils
  • Security Level: Public
  • Description:
    Hide

    As per PETALSDISTRIB-133, technical errors should be kept as errors and not transformed to fault.

    Now that PETALSCDK-130 is closed, FaultException, DocumentException, SOAP11FaultServerException and SOAP11FaultClientException should be removed from the CDK.

    Note: these exception were used previously by components to build faults that were coherent with their WSDL: for that case, we should provide utils to create Fault from a Document or other XML in-memory representation of a fault.
    But that shouldn't be done using Java exceptions (except maybe for the case of the POJO that seems to be relying on it... maybe we should move that "exception-based fault handling" logic to the pojo if we think current users of the component are exploiting them! (or provides an easy to use alternative)).

    Show
    As per PETALSDISTRIB-133, technical errors should be kept as errors and not transformed to fault. Now that PETALSCDK-130 is closed, FaultException, DocumentException, SOAP11FaultServerException and SOAP11FaultClientException should be removed from the CDK. Note: these exception were used previously by components to build faults that were coherent with their WSDL: for that case, we should provide utils to create Fault from a Document or other XML in-memory representation of a fault. But that shouldn't be done using Java exceptions (except maybe for the case of the POJO that seems to be relying on it... maybe we should move that "exception-based fault handling" logic to the pojo if we think current users of the component are exploiting them! (or provides an easy to use alternative)).
  • Environment:
    -

Issue Links

Activity

Victor NOËL made changes - Thu, 21 May 2015 - 17:11:53 +0200
Field Original Value New Value
Status New [ 10000 ] Open [ 10002 ]
Priority Major [ 3 ]
Assignee Christophe DENEUX [ cdeneux ] Victor NOËL [ vnoel ]
Victor NOËL made changes - Thu, 21 May 2015 - 17:12:12 +0200
Link This issue depends on PETALSBCSOAP-145 [ PETALSBCSOAP-145 ]
Victor NOËL made changes - Thu, 21 May 2015 - 17:12:23 +0200
Link This issue depends on PETALSSEJSR-28 [ PETALSSEJSR-28 ]
Victor NOËL made changes - Thu, 21 May 2015 - 17:13:46 +0200
Link This issue blocks PETALSDISTRIB-133 [ PETALSDISTRIB-133 ]
Victor NOËL made changes - Fri, 22 May 2015 - 13:00:38 +0200
Link This issue depends on PETALSCDK-130 [ PETALSCDK-130 ]
Victor NOËL made changes - Fri, 22 May 2015 - 13:04:12 +0200
Link This issue depends on PETALSSEACTIVITI-3 [ PETALSSEACTIVITI-3 ]
Victor NOËL made changes - Fri, 22 May 2015 - 16:52:51 +0200
Link This issue depends on PETALSSEPOJO-22 [ PETALSSEPOJO-22 ]
Victor NOËL made changes - Fri, 22 May 2015 - 17:03:02 +0200
Link This issue depends on PETALSBCHTTP-1 [ PETALSBCHTTP-1 ]
Victor NOËL made changes - Fri, 22 May 2015 - 17:06:34 +0200
Link This issue blocks PETALSCDK-133 [ PETALSCDK-133 ]
Victor NOËL made changes - Wed, 15 Jul 2015 - 13:39:36 +0200
Status Open [ 10002 ] In Progress [ 10003 ]
Victor NOËL made changes - Wed, 15 Jul 2015 - 13:40:21 +0200
Status In Progress [ 10003 ] Resolved [ 10004 ]
Fix Version/s 5.5.0 [ 10406 ]
Resolution Fixed [ 1 ]
Victor NOËL made changes - Wed, 15 Jul 2015 - 13:48:35 +0200
Link This issue depends on PETALSBCSOAP-145 [ PETALSBCSOAP-145 ]

People

Dates

  • Created:
    Thu, 21 May 2015 - 17:09:26 +0200
    Updated:
    Wed, 15 Jul 2015 - 13:48:35 +0200
    Resolved:
    Wed, 15 Jul 2015 - 13:40:21 +0200