Summary | doesn't properly quote |
Queue | Turba |
Queue Version | HEAD |
Type | Bug |
State | Resolved |
Priority | 1. Low |
Owners | chuck (at) horde (dot) org |
Requester | ldillon (at) sk-tech (dot) com |
Created | 01/13/2005 (7484 days ago) |
Due | |
Updated | 02/09/2005 (7457 days ago) |
Assigned | 01/15/2005 (7482 days ago) |
Resolved | 02/07/2005 (7459 days ago) |
Github Issue Link | |
Github Pull Request | |
Milestone | |
Patch | No |
State ⇒ Resolved
Assigned to Chuck Hagenbuch
State ⇒ Feedback
DNs in the LDAP driver, which should cover the case you found plus a
bunch of others that we weren't properly catching before:
http://cvs.horde.org/diff.php/turba/lib/Driver/ldap.php?r1=1.56&r2=1.57&ty=u
If this works for you, I'll merge it into the next stable Turba release.
State ⇒ Unconfirmed
Priority ⇒ 1. Low
Type ⇒ Bug
Summary ⇒ doesn't properly quote
Queue ⇒ Turba
doesn't properly quote the string. The following fixes this:
--- /tmp/ldap.php.org 2005-01-13 08:37:13.727402078 -0600
+++ ldap.php 2005-01-13 08:31:39.689948716 -0600
@@ -290,7 +290,11 @@
$newrdn = '';
foreach ($this->_params['dn'] as $param) {
if (isset($attributes[$param])) {
- $newrdn .= $param . '=' . $attributes[$param] . ',';
+ if (strpos($attributes[$param],',') === false) {
+ $newrdn .= $param . '=' .
$attributes[$param] . ',';
+ } else {
+ $newrdn .= $param . '="' . $attributes[$param] . '",';
+ }
}
}
$newrdn = substr($newrdn, 0, -1);