Noel, after your commit I'm getting the following build error:
compile: Compiling James Java sources Compiling 228 source files to C:\cvsssh\james-branch_2_1_fcs\build\classes C:\cvsssh\james-branch_2_1_fcs\src\java\org\apache\james\dnsserver\DNSServer.java:257: lookupRecords(org.xbill.DNS.Name,int,byte) in org.xbill.DNS.Cache cannot be applied to (org.xbill.DNS.Name,int,int) SetResponse cached = cache.lookupRecords(name, type, dnsCredibility); ^ Note: Some input files use or override a deprecated API. Note: Recompile with -deprecation for details. 1 error BUILD FAILED file:C:/cvsssh/james-branch_2_1_fcs/build.xml:274: Compile failed; see the compiler error output for details. Vincenzo > -----Original Message----- > From: Noel J. Bergman [mailto:[EMAIL PROTECTED] > Sent: marted́ 13 aprile 2004 3.50 > To: James Developers List > Subject: RE: MxSorter and changes to DNSServer.java > > > > I haven't looked at the org.xbill.DNS.Address but it > > sounds like a good solution. > > Agreed. I'll look at making that change. > > > I have put it on my todo-list to go through the code and > > weed-out InetAddress, just haven't gotten around to it yet. > > I just did: > > $ grep --recursive "InetAddress\.get.*ByName" src/java/ -l > src/java/org/apache/james/James.java > src/java/org/apache/james/core/AbstractJamesService.java > src/java/org/apache/james/dnsserver/DNSServer.java > src/java/org/apache/james/fetchmail/MessageProcessor.java > > src/java/org/apache/james/transport/mailets/RemoteDeliverySocketFactory.java > src/java/org/apache/james/transport/mailets/RemoteDelivery.java > src/java/org/apache/james/transport/matchers/InSpammerBlacklist.java > src/java/org/apache/james/util/NetMatcher.java > > James, AbstractJamesService, and RemoteDeliverySocketFactory were all local > stuff. > > I've updated DNSServer, MessageProcessor, RemoteDelivery, > InSpammerBlacklist, and NetMatcher it use org.xbill.DNS.Address for the two > static methods. I fully qualified the name to (a) make it easier to find, > and (b) because it was necessary in some cases to distinguish from > javax.mail.Address, anyway. Of course, I just realized that like an idiot, > I just committed the changes before testing them. :-( I'll be doing that > immediately. > > Another side-effect of the changes since yours and Richard's is that by > passing an IP address to JavaMail, we eliminate its lookup, which would also > have used InetAddress' caching. > > I find it bothersome that Sun's DNS service for JNDI apparently ignores the > TTL provided by the DNS server. > > --- Noel > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]