Hi Costin, I'm in the process of porting the Tomcat 3.2 exception propagation changes to Tomcat 3.3 that were made to Handler.java, ServletWrapper.java, and RequestDispatcherImpl.java. I was trying to review behavior as I do so to see what can be improved or cleaned up. One thing that looks like in can be removed is the synchronization found in ServletWrapper.doInit(). If you were to sync block here, it would mean that the preServletInit() interceptors had been called twice. It looks like the "init" synchronization in Handler.service() should suffice. Would you agree? I was going to port the default error response changes next. Part of that involved adding an errorURI property to ResponseImpl.java. It is used to carry the "error URI" from lower levels in case it is needed by the top level handler that outputs the default error response. It there a better way to transport this "error URI" that setting it on the response? Would using a note be consistent with the design? One more question, should ServletWrapper.init() be changed to call postServletInit() interceptors, even if doInit() throws an exception? If init() fails, I would think that the interceptors should be made aware of this fact. Thanks, Larry
