Moodle

LDAP Authentication with MS AD does not work when apostrophes are present in usernames

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 1.5.4, 1.6, 1.6.1
  • Fix Version/s: 1.8, 1.9
  • Component/s: Authentication
  • Labels:
    None
  • Environment:
    Windows 2000 IIS5, PHP 5.1.4, MySQL 5.0.22-community-nt, MS Active Directory

Description

Authentication using LDAP (Active Directory) fails when trying to login using a username containing an apostrophe (') (i.e. sean o'neill). Returns error stating invalid username and re-presents username with apostrpohe escaped i.e. sean o\'neill.

To resolve this, instructions provided by Iñaki Arenaza in Forum Post 54901 (http://moodle.org/mod/forum/discuss.php?d=54901) involved removing escape character () before submitting query to LDAP. This was modified in <moodle>\auth\ldap\lib.php

As part of this solution, the user profile page had to be modifed, as described in Forum Post 55083 (http://moodle.org/mod/forum/discuss.php?d=55083).

After applying the fix stated above, our production server was able to process usernames with apostrophes via LDAP.

To reproduce
===========

1. Create a new user in Active Directory with an apostrophe in the username (i.e. sean o'neill)
2. Wait for replication etc around network and then try to login with that username to moodle

Expected result
============

User is logged in to Moodle as per normal

Actual result
==========

Username/Password is invalid error.

Issue Links

Activity

Hide
Petr Škoda (skodak) added a comment -

this hould be fixed in 1.8dev cvs, could you please test it and confirm?

Show
Petr Škoda (skodak) added a comment - this hould be fixed in 1.8dev cvs, could you please test it and confirm?
Hide
Petr Škoda (skodak) added a comment -

closing now, should be fixed in 1.8
please file a new report for problems in 1.8 if needed

thanks for the report!

Show
Petr Škoda (skodak) added a comment - closing now, should be fixed in 1.8 please file a new report for problems in 1.8 if needed thanks for the report!

People

Vote (1)
Watch (2)

Dates

  • Created:
    Updated:
    Resolved: