[Fedora-directory-commits] console/src/com/netscape/management/client/console Console.java, 1.14, 1.15

[Richard Allen Megginson]

Author: rmeggins

Update of /cvs/dirsec/console/src/com/netscape/management/client/console
In directory cvs1.fedora.phx.redhat.com:/tmp/cvs-serv10703/console/src/com/netscape/management/client/console

Modified Files:
	Console.java 
Log Message:
Resolves: bug 430364
Bug Description: setup-ds-admin.pl does not correctly set the admin server ip address
Reviewed by: nkinder (Thanks!)
Fix Description: If the admin server advertises its IP address in its cn=config nsServerAddress, the console will attempt to use the address instead of hostname (which will break https, btw).  If the address is set to 0.0.0.0, clients cannot use this, so must fall back on the hostname.
Platforms tested: RHEL5
Flag Day: no
Doc impact: no



Index: Console.java
===================================================================
RCS file: /cvs/dirsec/console/src/com/netscape/management/client/console/Console.java,v
retrieving revision 1.14
retrieving revision 1.15
diff -u -r1.14 -r1.15
--- Console.java	15 Jul 2008 17:26:58 -0000	1.14
+++ Console.java	29 Jan 2009 21:23:59 -0000	1.15
@@ -266,8 +266,9 @@
              * nsserveraddress might not be defined, which means that the
              * admin server should listen on all interfaces rather than on
              * a specific one. Read serverhostname from the SIE entry.
+             * admin server uses 0.0.0.0 to mean listen on all interfaces
              */
-            if (host == null || host.trim().length() == 0) {
+            if ((host == null) || (host.trim().length() == 0) || host.equals("0.0.0.0")) {
                 LDAPEntry sieEntry = ldc.read(dn=adminServerDN, new String[] {"serverhostname"});
                 if (sieEntry == null) {
                     Debug.println("ERROR Console.getInstanceAdminURL: " +