Mahara

  1. Bug #1607669
  2. Comment #6

Comment 6 for bug 1607669

Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/6762
Committed: https://git.mahara.org/mahara/mahara/commit/b4298f9b4fc6edcb90c9914c6ad4af370b56a1a7
Submitter: Robert Lyon (<email address hidden>)
Branch: 16.04_STABLE

commit b4298f9b4fc6edcb90c9914c6ad4af370b56a1a7
Author: Ghada El-Zoghbi <email address hidden>
Date: Fri Jul 29 18:40:35 2016 +1000

bug#1607669: Validate the results of ldap_search() and ldap_list() when syncing users.

When retrieving data from the LDAP server, check the results retrieved from
ldap_search() and ldap_list(). If there is an error, do not continue.

Otherwise, the sync_users() will assume that zero users were retrieved
from LDAP. It will then deleted or suspended users when
'If a user is no longer present in LDAP' is set to either suspend
users or delete them.

behatnotneeded

Change-Id: Ib57901c61f769d96720cf932d2e0d7f643853a56
(cherry picked from commit 1daee33a1ef67a0e0fc0599c67de08896408519e)