From bloritsch@infoplanning.com  Tue Dec 12 16:22:17 2000
Return-Path: <bloritsch@infoplanning.com>
Mailing-List: contact cocoon-dev-help@xml.apache.org; run by ezmlm
Delivered-To: mailing list cocoon-dev@xml.apache.org
Received: (qmail 93220 invoked from network); 12 Dec 2000 16:22:17 -0000
Received: from fw.infoplanning.net (HELO infoplanning.com) (@209.8.58.131)
  by locus.apache.org with SMTP; 12 Dec 2000 16:22:17 -0000
Received: (qmail 26227 invoked from network); 12 Dec 2000 16:29:00 -0000
Received: from unknown (HELO johnnybravo) (192.168.0.189)
  by inet with SMTP; 12 Dec 2000 16:29:00 -0000
Message-ID: <013701c06457$6dda9db0$bd00a8c0@infoplanning.com>
From: "Berin Loritsch" <bloritsch@infoplanning.com>
To: <cocoon-dev@xml.apache.org>
References: <20001212160805.72689.qmail@locus.apache.org> <012f01c06456$538c42c0$bd00a8c0@infoplanning.com>
Subject: Re: cvs commit: xml-cocoon/src/org/apache/cocoon Cocoon.java
Date: Tue, 12 Dec 2000 11:20:23 -0500
MIME-Version: 1.0
Content-Type: text/plain;
	charset="Windows-1252"
Content-Transfer-Encoding: 7bit
X-Priority: 3
X-MSMail-Priority: Normal
X-Mailer: Microsoft Outlook Express 5.50.4133.2400
X-MimeOLE: Produced By Microsoft MimeOLE V5.50.4133.2400
X-Spam-Rating: locus.apache.org 1.6.2 0/1000/N


----- Original Message ----- 
From: "Berin Loritsch" <bloritsch@infoplanning.com>
To: <cocoon-dev@xml.apache.org>
Sent: Tuesday, December 12, 2000 11:12 AM
Subject: Re: cvs commit: xml-cocoon/src/org/apache/cocoon Cocoon.java


> ----- Original Message ----- 
> From: <greenrd@locus.apache.org>
> To: <xml-cocoon-cvs@apache.org>
> Sent: Tuesday, December 12, 2000 11:08 AM
> Subject: cvs commit: xml-cocoon/src/org/apache/cocoon Cocoon.java
> 
> 
> > greenrd     00/12/12 08:08:04
> > 
> >    public class Cocoon extends HttpServlet implements Defaults {
> >   +
> >   +    // Quick workaround for Websphere bug
> >   +    static {
> >   +      try {
> >   +        Class.forName ("com.ibm.servlet.classloader.Handler");
> >   +      }
> >   +      catch (Throwable t) {} // ignore
> >   +    }
> >    
> >        Engine engine = null;
> >        String message = null;
> >   
> 
> This was so unnecessary and unscalable.  We have the attribute
> force-load for this express purpose.  All we have to do is add
> the proper attribute to the servlet init parameters and all is
> well with the world!  That is why I committed that change for
> parsing the "force-load" parameter!
> 
> And as a general course of action: NEVER completely ignore an
> exception--log it at the very least!

I just noticed, that this is for Cocoon 1.  See if you can back-port
the force-load parameter handling from Cocoon 2.  That way, if we
have problems with other servlet engines, we don't have to change
the code.

The exception should count.  If the parameter force-load is encountered,
and Cocoon can't load the class, then we should throw a ServletException
and abort.  If the parameter is not encountered, then it should not
be processed.