On Dec 19, 2007, at 2:00 PM, Grzegorz Kossakowski wrote:
The check has been introduced in order to handle polymorphism of servlet services
Shouldn't it explicitly check for 404 only? And, for Cocoon servlet, for unhandled ResourceNotFoundException?
Any of 1XX-3XX, 5XX are clearly not an indication for calling super, IIUC. Seems that only 404 ('The server has not found anything matching the Request-URI.') should be.
Even then, if 'super' is not available, it should simply return that 404 without messing with it... Right? There should be no wrappedResponse.reset() is super is null. There also should be no "throw null;", if there were no ServletException.
Vadim
