Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-11299

ldap field sync problem (prevents userdata to get from ldap)

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.8.2
    • Fix Version/s: 1.7.2, 1.8.2, 1.9
    • Component/s: Authentication
    • Labels:
      None
    • Environment:
      debian etch, apache
    • Database:
      MySQL
    • Affected Branches:
      MOODLE_18_STABLE
    • Fixed Branches:
      MOODLE_17_STABLE, MOODLE_18_STABLE, MOODLE_19_STABLE

      Description

      I'm using authldap module, and set up successfully. Can logon to ldap, and so on. But if I set the userdata fields to get from the ldap, it simply don't get down.

      I figured out that tha problem caused by an invalid (empty) search filter in auth.php (moodle/auth/ldap) at line 139 the original code:

      1. if (!$user_info_result = ldap_read($ldapconnection, $user_dn, $this->config->objectclass, $search_attribs)) { # return false; // error! # }

      It is commented out by me and hardcoded the following:
      if (!$user_info_result = ldap_read($ldapconnection, $user_dn, '(objectclass=VirtualMailAccount)', $search_attribs))

      { return false; // error! }

      this way data can come down, because objectclass is the correct value...

      Please correct the bug, so after I upgrade, there will be no need such hardcoded stuff...

        Attachments

          Activity

            People

            Assignee:
            iarenaza Iñaki Arenaza
            Reporter:
            eliast Éliás Tamás
            Tester:
            Nobody
            Participants:
            Component watchers:
            Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              30/Mar/07