sling-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jörg Hoh (JIRA) <j...@apache.org>
Subject [jira] [Created] (SLING-7552) SlingPostServlet error handling still insufficient
Date Tue, 20 Mar 2018 15:20:00 GMT
Jörg Hoh created SLING-7552:
-------------------------------

             Summary: SlingPostServlet error handling still insufficient
                 Key: SLING-7552
                 URL: https://issues.apache.org/jira/browse/SLING-7552
             Project: Sling
          Issue Type: Improvement
            Reporter: Jörg Hoh


At the moment the default errorhandling of Sling [1] cannot be used for errors caused and
handled by the SlingPostServlet itself. It will always return its own custom output without
the chance of customizing it. Although Antonio and Justing worked in SLING-2156 to improve
this situation, it still requires extra work (implementing a PostResponseWithErrorHandling).
It would be better if the output could be customized by the "standard error handling".

 

How to reproduce:
 * create an error handling script in /apps/sling/servlet/errorhandler/default.jsp which creates
some random output.
 * Validate this script config by doing a request which causes some exception (do not use
the SlingPostServlet here)
 * Do a POST to the Sling instance which results in an exception (e.g. due to insufficient
permissions.
 * The output of the second call is completely determined by the SlingPostServlet, the default
error handling does not kick in.

Proposed solution:

* The SlingPostServlet should not swallow the exception and handle it by itself, but rather
re-throw it, so the standard error handling is triggered.

 

[1] http://sling.apache.org/documentation/the-sling-engine/errorhandling.html



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Mime
View raw message