Remove warning about ActiveDirectory on LDAP auth "Update user info on login"

Bug #1337664 reported by Aaron Wells
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
Medium
Aaron Wells

Bug Description

See https://bugs.launchpad.net/mahara/+bug/656679

In that bug, we added a warning message to the LDAP auth plugin's "update user info on login" checkbox, that tells people it may cause login problems in ActiveDirectory.

We subsequently pushed a patch which probably fixed the original problem that was causing the bug's behavior. But, we forgot to remove that warning about ActiveDirectory.

I've just verified, in testing with a user on myportfolio.school.nz, that activating this feature did not cause problems for them. So in light of the probably original problem being fixed, and firsthand verification that the feature works, I think it's fair to remove this warning.

Tags: auth ldap
Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

Patch for "master" branch: https://reviews.mahara.org/3478

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

Reviewed: https://reviews.mahara.org/3478
Committed: http://gitorious.org/mahara/mahara/commit/ed705c246bc6c259f3dd85b7aac6db3804838082
Submitter: Yuliya Bozhko (<email address hidden>)
Branch: master

commit ed705c246bc6c259f3dd85b7aac6db3804838082
Author: Aaron Wells <email address hidden>
Date: Fri Jul 4 14:09:40 2014 +1200

Remove unnecessary warning message in LDAP auth

Bug 1337664

Change-Id: I3f3bbb777144d5959844f5b7bafa139812af06b7

Changed in mahara:
status: In Progress → Fix Committed
Aaron Wells (u-aaronw)
Changed in mahara:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.