> 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]

Reply via email to