summaryrefslogtreecommitdiffstats
path: root/ldap/admin/src/scripts/FileConn.pm
diff options
context:
space:
mode:
authorRich Megginson <rmeggins@redhat.com>2007-11-14 17:53:44 +0000
committerRich Megginson <rmeggins@redhat.com>2007-11-14 17:53:44 +0000
commitb0b81ec8ce6496d205ddc02b0c26aac19b4af828 (patch)
tree6f21b21dfda3c2ba35c7c563a676cf6094b6a6f2 /ldap/admin/src/scripts/FileConn.pm
parentbc83e3255b4b0ebe31d9f42ad57fbea0bc49e056 (diff)
downloadds-b0b81ec8ce6496d205ddc02b0c26aac19b4af828.tar.gz
ds-b0b81ec8ce6496d205ddc02b0c26aac19b4af828.tar.xz
ds-b0b81ec8ce6496d205ddc02b0c26aac19b4af828.zip
Resolves: bug 197997
Bug Description: PTA config parsing broken Reviewed by: nhosoi (Thanks!) Fix Description: The problem is that it is very difficult to use a comma as a delimiter between the url and the optional settings. This is because the suffix may contain many commas. The argument string may look like this: ldap://host1:port1 host2:port2 .... hostN:portN/a,long,suffix1:a,long,suffix2;....;a,long,suffixN optional,numeric,settings The ldap url may not contain any spaces after the hostlist - the suffixlist part must contain only url encoded spaces if the suffix actually has a space in it. So the solution is to use a space to separate the url from the options list. The parser looks for the first space after the last "/" in the url. This should be ok - at least it will not break the most common use of pta, which is to allow the config DS admin user to log into servers that do not have the o=NetscapeRoot. setup will use something like this: ldap://configdshost:configdsport/o=NetscapeRoot with not optional settings - this should parse just fine with the new code. Platforms tested: RHEL5 x86_64 Flag Day: no Doc impact: no QA impact: should be covered by regular nightly and manual testing New Tests integrated into TET: none
Diffstat (limited to 'ldap/admin/src/scripts/FileConn.pm')
0 files changed, 0 insertions, 0 deletions