Hi,

On Dec 18, 2007 1:16 PM, Felix Meschberger <[EMAIL PROTECTED]> wrote:

> ...(1) for status codes:...

I assume you mean status codes >= 400, right?

I'm ok with the proposed ScriptOrServlet resolution.

> ...(2) Likewise for Throwables it would be:
>
>     Class<?> tClass = throwable.getClass();
>     Servlet s = null;
>     while (s == null && tClass != Object.class) {
>         s = resolveScriptOrServlet(tClass);,,,

Shouldn't we use Class.getSimpleName() here?So that scripts are named
MyFunnyException instead of com.nowhere.somepackage.MyFunnyException?

>         tClass = tClass.getSuperclass();
>     }
>     if (s == null) {
>         s = defaultErrorHandlerServlet;
>     }...

Sounds good to me, but we'll have to be careful with exceptions that
occur while looking up exception scripts (I'm kind of noting the
obvious here ;-)

-Bertrand

Reply via email to