Hi Chiths, yes, you're right, the code in Handler.java is obsolete and shouldn't be there. I'll fix this asap. Thanks for spotting this!
But the Handler class is originally designed to be a lightweight object as there are many handler objects created, so extending it to make it Loggable is not so desirable. Thanks Carsten > -----Original Message----- > From: Chitharanjan Das [mailto:[EMAIL PROTECTED]] > Sent: Saturday, March 30, 2002 6:10 AM > To: [EMAIL PROTECTED] > Subject: Attn : SunRise developers: Identified 1 bug and fixed it , > please clarify and do the same in the CVS > > > Hello, > In Handler .java in the method Configure, there is a section of > code, which is trying to build a fully referenced URI for the redirect > URI. This > Should not be done, the redirector takes care of this. > I have commented out the section and this has the label > /** Commented by Chiths > > And also I have added some System.out.println statements for debug > because Handler does not implement Loggable. > > Could the owner try and implement Loggable interface for handler as it > contains some juicy stuff, required for debugging.. > > I am attaching the modified Handler.java (CVS Head version). Please > view it and comment if this is a mistake. > > Thanks in advance, > > Regds, > Chiths > --------------------------------------------------------------------- Please check that your question has not already been answered in the FAQ before posting. <http://xml.apache.org/cocoon/faqs.html> To unsubscribe, e-mail: <[EMAIL PROTECTED]> For additional commands, e-mail: <[EMAIL PROTECTED]>
