diff options
| author | Rich Megginson <rmeggins@redhat.com> | 2007-11-14 17:53:44 +0000 |
|---|---|---|
| committer | Rich Megginson <rmeggins@redhat.com> | 2007-11-14 17:53:44 +0000 |
| commit | b0b81ec8ce6496d205ddc02b0c26aac19b4af828 (patch) | |
| tree | 6f21b21dfda3c2ba35c7c563a676cf6094b6a6f2 /ldap/admin/src/scripts/FileConn.pm | |
| parent | bc83e3255b4b0ebe31d9f42ad57fbea0bc49e056 (diff) | |
| download | ds-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
