Hi > Am 12.11.2014 um 13:43 schrieb [email protected]: > > On Mon, Nov 10, 2014 at 1:50 PM, Bertrand Delacretaz > <[email protected]> wrote: >> >> ...in some >> cases Sling returns a 200 status if the error script fails, as in my >> SLING-4143 "example 2". I think we should at least fix that one... > > I have fixed that now, details in SLING-4143 - my changes only cause a > 500 status if the error handling script or servlet throws an Exception > or Error.
Sorry 'bout that, but I have to reopen: You must not report the failing error handler to the client but only the original issue. You may certainly log the failing error handler. Consider this case: * Client browses non-existing /a/b/xyz.html * Not found, hence 404 * 404 handler fails due to some bad programming * client gets 500 I don’t think that is ok. Regards Felix
