[SLING-6703] Sling Post Servlet: Do not hide original exception in AbstractPostResponse.setError

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: Servlets Post 2.3.14
• Fix Version/s: Servlets Post 2.3.16
• Component/s: Servlets
• Assignee: Konrad Windszus
• Reporter: Konrad Windszus
• Created: 23/Mar/17 14:21
• Updated: 11/Jul/17 14:14
• Resolved: 24/Mar/17 10:39

Description

Currently AbstractPostResponse.setError (https://github.com/apache/sling/blob/4df9ab2d6592422889c71fa13afd453a10a5a626/bundles/servlets/post/src/main/java/org/apache/sling/servlets/post/AbstractPostResponse.java#L221) always ignores the given Throwable and just creates a new generic SlingException.

To e.g. allow SlingPostProcessor to throw meaningful exceptions which appear in the response body, the given exception should not be wrapped but just the given throwable's message text should be given out in the document.

Issue Links

• blocks: SLING-6515 Sling Post Processor considering validation model as well
• is broken by: SLING-4415 :applyTo should not display changeLog (when operation fails)