[SLING-1850] Make sure error response is sent back even if the response cannot be reset

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: Engine 2.2.0
• Component/s: Engine
• Assignee: Felix Meschberger
• Reporter: Felix Meschberger
• Created: 22/Oct/10 14:42
• Updated: 28/Jan/11 09:07
• Resolved: 22/Oct/10 14:55

Description

Along the lines of fixing SLING-1842 I also modified error handling to not send back any response data at all if the response has already been committed. Previously the error information was just written into the existing response regardless of whether the response has been committed or not.

The problem is in the SlingRequestProcessorImpl.handleError methods which do nothing if the response has already been committed. This is probably fundamentally wrong and must be fixed.

It is the task of the actual error handler to (a) check whether the response has already been committed and can be reset and (b) to decide whether to send back response data if the response is committed or not...