- Fix LDAP attributes case senitivity problems (#1485830)

release-0.6
alecpl 15 years ago
parent 6855ce6f70
commit 0131ec9724

@ -1,6 +1,7 @@
CHANGELOG RoundCube Webmail CHANGELOG RoundCube Webmail
=========================== ===========================
- Fix LDAP attributes case senitivity problems (#1485830)
- Fix LDAP addressbook browsing when only one directory is used (#1486022) - Fix LDAP addressbook browsing when only one directory is used (#1486022)
- Fix endless loop on error response for APPEND command (#1486060) - Fix endless loop on error response for APPEND command (#1486060)
- Don't require date.timezone setting in installer (#1485989) - Don't require date.timezone setting in installer (#1485989)

@ -55,9 +55,9 @@ class rcube_ldap extends rcube_addressbook
foreach ($p as $prop => $value) foreach ($p as $prop => $value)
if (preg_match('/^(.+)_field$/', $prop, $matches)) if (preg_match('/^(.+)_field$/', $prop, $matches))
$this->fieldmap[$matches[1]] = $value; $this->fieldmap[$matches[1]] = strtolower($value);
$this->sort_col = $p["sort"]; $this->sort_col = $p['sort'];
$this->connect(); $this->connect();
} }
@ -102,7 +102,7 @@ class rcube_ldap extends rcube_addressbook
$this->ready = true; $this->ready = true;
// User specific access, generate the proper values to use. // User specific access, generate the proper values to use.
if ($this->prop["user_specific"]) { if ($this->prop['user_specific']) {
// No password set, use the session password // No password set, use the session password
if (empty($this->prop['bind_pass'])) { if (empty($this->prop['bind_pass'])) {
$this->prop['bind_pass'] = $RCMAIL->decrypt($_SESSION['password']); $this->prop['bind_pass'] = $RCMAIL->decrypt($_SESSION['password']);
@ -166,7 +166,7 @@ class rcube_ldap extends rcube_addressbook
{ {
if ($this->conn) if ($this->conn)
{ {
@ldap_unbind($this->conn); ldap_unbind($this->conn);
$this->conn = null; $this->conn = null;
} }
} }
@ -255,8 +255,8 @@ class rcube_ldap extends rcube_addressbook
// we have a search result resource // we have a search result resource
if ($this->ldap_result && $this->result->count > 0) if ($this->ldap_result && $this->result->count > 0)
{ {
if ($this->sort_col && $this->prop['scope'] !== "base") if ($this->sort_col && $this->prop['scope'] !== 'base')
@ldap_sort($this->conn, $this->ldap_result, $this->sort_col); ldap_sort($this->conn, $this->ldap_result, $this->sort_col);
$start_row = $subset < 0 ? $this->result->first + $this->page_size + $subset : $this->result->first; $start_row = $subset < 0 ? $this->result->first + $this->page_size + $subset : $this->result->first;
$last_row = $this->result->first + $this->page_size; $last_row = $this->result->first + $this->page_size;
@ -381,13 +381,15 @@ class rcube_ldap extends rcube_addressbook
$res = null; $res = null;
if ($this->conn && $dn) if ($this->conn && $dn)
{ {
$this->ldap_result = @ldap_read($this->conn, base64_decode($dn), "(objectclass=*)", array_values($this->fieldmap)); $this->ldap_result = ldap_read($this->conn, base64_decode($dn), "(objectclass=*)", array_values($this->fieldmap));
$entry = @ldap_first_entry($this->conn, $this->ldap_result); $entry = @ldap_first_entry($this->conn, $this->ldap_result);
if ($entry && ($rec = ldap_get_attributes($this->conn, $entry))) if ($entry && ($rec = ldap_get_attributes($this->conn, $entry)))
{ {
$rec = array_change_key_case($rec, CASE_LOWER);
// Add in the dn for the entry. // Add in the dn for the entry.
$rec["dn"] = base64_decode($dn); $rec['dn'] = base64_decode($dn);
$res = $this->_ldap2result($rec); $res = $this->_ldap2result($rec);
$this->result = new rcube_result_set(1); $this->result = new rcube_result_set(1);
$this->result->add($res); $this->result->add($res);
@ -408,11 +410,10 @@ class rcube_ldap extends rcube_addressbook
{ {
// Map out the column names to their LDAP ones to build the new entry. // Map out the column names to their LDAP ones to build the new entry.
$newentry = array(); $newentry = array();
$newentry["objectClass"] = $this->prop["LDAP_Object_Classes"]; $newentry['objectClass'] = $this->prop['LDAP_Object_Classes'];
foreach ($save_cols as $col => $val) { foreach ($save_cols as $col => $val) {
$fld = "";
$fld = $this->_map_field($col); $fld = $this->_map_field($col);
if ($fld != "") { if ($fld != '') {
// The field does exist, add it to the entry. // The field does exist, add it to the entry.
$newentry[$fld] = $val; $newentry[$fld] = $val;
} // end if } // end if
@ -421,15 +422,15 @@ class rcube_ldap extends rcube_addressbook
// Verify that the required fields are set. // Verify that the required fields are set.
// We know that the email address is required as a default of rcube, so // We know that the email address is required as a default of rcube, so
// we will default its value into any unfilled required fields. // we will default its value into any unfilled required fields.
foreach ($this->prop["required_fields"] as $fld) { foreach ($this->prop['required_fields'] as $fld) {
if (!isset($newentry[$fld])) { if (!isset($newentry[$fld])) {
$newentry[$fld] = $newentry[$this->_map_field("email")]; $newentry[$fld] = $newentry[$this->_map_field('email')];
} // end if } // end if
} // end foreach } // end foreach
// Build the new entries DN. // Build the new entries DN.
$dn = $this->prop["LDAP_rdn"]."=".$newentry[$this->prop["LDAP_rdn"]].",".$this->prop['base_dn']; $dn = $this->prop['LDAP_rdn'].'='.$newentry[$this->prop['LDAP_rdn']].','.$this->prop['base_dn'];
$res = @ldap_add($this->conn, $dn, $newentry); $res = ldap_add($this->conn, $dn, $newentry);
if ($res === FALSE) { if ($res === FALSE) {
return false; return false;
} // end if } // end if
@ -455,9 +456,8 @@ class rcube_ldap extends rcube_addressbook
$replacedata = array(); $replacedata = array();
$deletedata = array(); $deletedata = array();
foreach ($save_cols as $col => $val) { foreach ($save_cols as $col => $val) {
$fld = "";
$fld = $this->_map_field($col); $fld = $this->_map_field($col);
if ($fld != "") { if ($fld != '') {
// The field does exist compare it to the ldap record. // The field does exist compare it to the ldap record.
if ($record[$col] != $val) { if ($record[$col] != $val) {
// Changed, but find out how. // Changed, but find out how.
@ -465,9 +465,9 @@ class rcube_ldap extends rcube_addressbook
// Field was not set prior, need to add it. // Field was not set prior, need to add it.
$newdata[$fld] = $val; $newdata[$fld] = $val;
} // end if } // end if
elseif ($val == "") { elseif ($val == '') {
// Field supplied is empty, verify that it is not required. // Field supplied is empty, verify that it is not required.
if (!in_array($fld, $this->prop["required_fields"])) { if (!in_array($fld, $this->prop['required_fields'])) {
// It is not, safe to clear. // It is not, safe to clear.
$deletedata[$fld] = $record[$col]; $deletedata[$fld] = $record[$col];
} // end if } // end if
@ -484,7 +484,7 @@ class rcube_ldap extends rcube_addressbook
$dn = base64_decode($id); $dn = base64_decode($id);
if (!empty($deletedata)) { if (!empty($deletedata)) {
// Delete the fields. // Delete the fields.
$res = @ldap_mod_del($this->conn, $dn, $deletedata); $res = ldap_mod_del($this->conn, $dn, $deletedata);
if ($res === FALSE) { if ($res === FALSE) {
return false; return false;
} // end if } // end if
@ -492,7 +492,7 @@ class rcube_ldap extends rcube_addressbook
if (!empty($replacedata)) { if (!empty($replacedata)) {
// Replace the fields. // Replace the fields.
$res = @ldap_mod_replace($this->conn, $dn, $replacedata); $res = ldap_mod_replace($this->conn, $dn, $replacedata);
if ($res === FALSE) { if ($res === FALSE) {
return false; return false;
} // end if } // end if
@ -500,7 +500,7 @@ class rcube_ldap extends rcube_addressbook
if (!empty($newdata)) { if (!empty($newdata)) {
// Add the fields. // Add the fields.
$res = @ldap_mod_add($this->conn, $dn, $newdata); $res = ldap_mod_add($this->conn, $dn, $newdata);
if ($res === FALSE) { if ($res === FALSE) {
return false; return false;
} // end if } // end if
@ -520,13 +520,13 @@ class rcube_ldap extends rcube_addressbook
{ {
if (!is_array($ids)) { if (!is_array($ids)) {
// Not an array, break apart the encoded DNs. // Not an array, break apart the encoded DNs.
$dns = explode(",", $ids); $dns = explode(',', $ids);
} // end if } // end if
foreach ($dns as $id) { foreach ($dns as $id) {
$dn = base64_decode($id); $dn = base64_decode($id);
// Delete the record. // Delete the record.
$res = @ldap_delete($this->conn, $dn); $res = ldap_delete($this->conn, $dn);
if ($res === FALSE) { if ($res === FALSE) {
return false; return false;
} // end if } // end if
@ -591,7 +591,6 @@ class rcube_ldap extends rcube_addressbook
return strtr($str, array('*'=>'\2a', '('=>'\28', ')'=>'\29', '\\'=>'\5c')); return strtr($str, array('*'=>'\2a', '('=>'\28', ')'=>'\29', '\\'=>'\5c'));
} }
} }
?>

Loading…
Cancel
Save