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

An unknown error occurred - actions == null. Please see logs for more details.

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