At 08:53 26.09.2002 -0700, you wrote: >Paul, thanks. I will review the change and apply. I'll need to create a >test case first to reproduce the problem before applying the patch.
Absolutely. >Ceki, I don't know when we are planning to release 1.2.7, but this fix >should probably be included? I'll work on it tonight before looking at the >merge. Indeed. >-Mark > > > -----Original Message----- > > From: Paul Austin [mailto:[EMAIL PROTECTED]] > > Sent: Thursday, September 26, 2002 8:47 AM > > To: Log4J Developers List > > Subject: RE: log4j with xerces 2.1.0 > > > > > > Mark, > > > > Yes it should be backwards compatible as this is the standard > > way in JAXP to > > look up resources using either a public or system id. > > > > The resolver class can also be extended if you wanted to > > support a public id > > like the following to look up the DTD. > > > > -//Apache Software Group//DTD Log4j 1.4//EN > > > > Paul > > > > -----Original Message----- > > From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED]] > > Sent: September 25, 2002 10:43 PM > > To: Log4J Developers List > > Subject: RE: log4j with xerces 2.1.0 > > > > > > Hi Paul, > > > > Appreciate the patch submission. Your fix is backward compatible with > > earlier versions of xerces? > > > > thanks, > > -Mark > > > > > -----Original Message----- > > > From: Paul Austin [mailto:[EMAIL PROTECTED]] > > > Sent: Wednesday, September 25, 2002 11:41 AM > > > To: [EMAIL PROTECTED] > > > Subject: log4j with xerces 2.1.0 > > > > > > > > > All, > > > > > > With xerces 2.1.0 the DOMConfigurator class fails to load the XML > > > file as it > > > cannot find the log4j.dtd. > > > > > > Looking into this I saw that the original code was setting a > > > systemId on the > > > InputSource for the document. Further investigation has > > shown that this is > > > used if the input source does not have a reader or input stream > > > not instead > > > of using the dtd system id. (i.e. This would only be used > > to load the XML > > > document not the dtd). > > > > > > The correct way to do this is to add an entity resolver > > which will look at > > > either the systemId or the publicId and return an input source > > > for the dtd. > > > I have modified the DOMConfigurator to remove the old code > > and add the new > > > Log4jEntityResolver class which will return the dtd for > > log4j. I would > > > propose that this class be extended in the future to > > support a well known > > > public id for the configuration files as well. > > > > > > Can someone add this to cvs for the project > > > > > > Paul Austin > > > Galdos Systems Inc.(tm) > > > [EMAIL PROTECTED] > > > Tel: +1 (604) 484-2761 > > > Fax: +1 (604) 484-2755 > > > http://www.galdosinc.com/ > > > > > > Privileged or confidential information may be contained in this > > > message. If > > > this message was not intended for you, destroy it and notify us > > > immediately. > > > Opinions, conclusions, recommendations, and other > > information presented in > > > this message are not given or necessarily endorsed by my > > employer or firm. > > > > > > > > > > > > -- > > 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]> > > > >-- >To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]> >For additional commands, e-mail: <mailto:[EMAIL PROTECTED]> -- Ceki TCP implementations will follow a general principle of robustness: be conservative in what you do, be liberal in what you accept from others. -- Jon Postel, RFC 793 -- To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
