James Server
  1. James Server
  2. JAMES-302

Functionality of org.apache.james.dnsserver.DNSServer.getByName(String) is not symetric to java.net.InetAddress.getByName(String)

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.2.0, 2.3.0, 2.3.1, 2.3.2
    • Fix Version/s: 3.0-M2
    • Component/s: DNSServer
    • Labels:
      None
    • Environment:
      Tested on WIN2000, JDK 1.4.1_01-b01

      Description

      org.apache.james.dnsserver.DNSServer.getByName(String) does not always return the same result as java.net.InetAddress.getByName(address). Sometimes an exception is thrown when the standard implementation does not.

      When passed a fully qualified domain name the results are the same. When passed a hostname or the special name 'localhost', a java.net.UnknownHostException is thrown by org.apache.james.dnsserver.DNSServer while java.net.InetAddress resolves the addresses correctly.

      This is a critical issue as in v2.2.0 java.net.InetAddress.getByName() has pretty thoroughly been replaced by org.apache.james.dnsserver.DNSServer.getByName(), but in the noted circumstances it doesn't perform the same. Dependent code breaks.

      Here are the contrasting examples...

      // FAILS
      String address = "localhost";
      java.net.InetAddress inetAddress = org.apache.james.dnsserver.DNSServer.getByName(address);
      return inetAddress;

      // FAILS
      String address = "hostname";
      java.net.InetAddress inetAddress = org.apache.james.dnsserver.DNSServer.getByName(address);
      return inetAddress;

      // SUCCEEDS
      String address = "localhost";
      java.net.InetAddress inetAddress = java.net.InetAddress.getByName(address);
      return inetAddress;

      // SUCCEEDS
      String address = "hostname";
      java.net.InetAddress inetAddress = java.net.InetAddress.getByName(address);
      return inetAddress;

        Issue Links

          Activity

            People

            • Assignee:
              Norman Maurer
              Reporter:
              Steve Brewin
            • Votes:
              3 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development