----- Original Message -----
From: "Craig R. McClanahan" <[EMAIL PROTECTED]>
To: <[EMAIL PROTECTED]>
Sent: Tuesday, February 26, 2002 9:15 AM
Subject: Another 1.1 TODO


> Something we should include on the 1.1 TODO list is to ensure that you can
> put struts.jar (and the appropriate commons-*.jar files it depends on)
> into a container-provided shared class loader (such as the "lib" directory
> in Tomcat 4.0.x) instead of requiring that they be included in the
> /WEB-INF/lib directory.  This will provide more flexibility for those
> deploying multiple Struts-based apps on the same container.

I agree that this would be a nice-to-have for 1.1. Pre-beta? Post-beta?

>
> Principally, this means that whenever the Struts code does a
> Class.forName() on an application-provided class, it should be modified to
> look in the webapp class loader first:
>
>   ClassLoader webappClassLoader =
>     Thread.currentThread().getContextClassLoader();
>
> which is guaranteed to work correctly on Servlet 2.3 containers (required
> in the spec), and in practice seems to work correctly on all containers
> that matter.  (If we're creating objects with Digester, this probably
> means using the factory create rule rather than the object create rule so
> that the object creation factory can do the right thing.)

What are the implications of this last comment for existing applications
that use the Digester to load their own additional configuration files? Will
they still work, or will they require the same change?

>
> I will put this on my list unless someone else wants to take a crack at
> it.

I'm happy for you to take care of this. ;-) I have my own list of
fixes/changes I'd like to get in, on the top of which is the customisation
of action mappings (which I'd really like some feedback on, if you have a
sec!).

--
Martin Cooper


>
> Craig
>
>
>
> --
> To unsubscribe, e-mail:
<mailto:[EMAIL PROTECTED]>
> For additional commands, e-mail:
<mailto:[EMAIL PROTECTED]>
>


--
To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>

Reply via email to