[Fedora-directory-commits] ldapserver/ldap/admin/src/scripts repl-monitor.pl, 1.1, 1.2 template-repl-monitor-cgi.pl.in, 1.4, NONE

Author: nhosoi Update of /cvs/dirsec/ldapserver/ldap/admin/src/scripts In directory cvs-int.fedora.redhat.com:/tmp/cvs-serv18485 Modified Files: repl-monitor.pl Removed Files: template-repl-monitor-cgi.pl.in Log Message: Resolves: #273501 Summary: Replication monitor CGI seems misconfigured (comment #1) 1) get_replica: missing return value for the success case 2) support the case "shadowport" is not passed Index: repl-monitor.pl =================================================================== RCS file: /cvs/dirsec/ldapserver/ldap/admin/src/scripts/repl-monitor.pl,v retrieving revision 1.1 retrieving revision 1.2 diff -u -r1.1 -r1.2 --- repl-monitor.pl 8 May 2007 17:22:33 -0000 1.1 +++ repl-monitor.pl 5 Sep 2007 16:35:28 -0000 1.2 @@ -441,6 +441,8 @@ } $conn->close; + + return 0; } # @@ -817,7 +819,11 @@ return $i if ($servers[$i] =~ /$hostnode($domainpattern)*:\d*=$shadowport\D/i); } - push (@servers, "$host:$port=$shadowport:$binddn:$bindpwd:$bindcert"); + if ($shadowport) { + push (@servers, "$host:$port=$shadowport:$binddn:$bindpwd:$bindcert"); + } else { + push (@servers, "$host:$port:$binddn:$bindpwd:$bindcert"); + } return $i; } --- template-repl-monitor-cgi.pl.in DELETED ---