[SLING-11365] Improve the exception handling of the DistributionPublisher.execute

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Done
• Fix Version/s: Content Distribution Journal Core 0.2.0
• Component/s: Content Distribution
• Assignee: Unassigned
• Reporter: José Correia
• Created: 01/Jun/22 13:00
• Updated: 01/Jun/22 13:16
• Resolved: 01/Jun/22 13:16

Description

In https://github.com/apache/sling-org-apache-sling-distribution-journal/blob/master/src/main/java/org/apache/sling/distribution/journal/impl/publisher/DistributionPublisher.java#L272 we are currently wrapping the exception that happens while trying to distribute content, which is resulting in the exception message for example:

org.apache.sling.distribution.common.DistributionException: Can't distribute package with size greater than 1 Byte, actual 4055

 

We need to change this wrapping, in order to obtain a more structured message, including the same information that we are logging:

Failed to create content package for requestType=ADD, paths=[/content]. Error=Can't distribute package with size greater than 1 Byte, actual 4053

 

This will allow for the content distribution API to deal with this error in a cleaner manner and provide more data to API users.

Issue Links

• links to: GitHub Pull Request #110