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

LDAP tries to update external when set not to.

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 1.7.3, 1.8.3, 1.9
    • Fix Version/s: 1.7.3, 1.8.3, 1.9
    • Component/s: Authentication
    • Labels:
      None
    • Affected Branches:
      MOODLE_17_STABLE, MOODLE_18_STABLE, MOODLE_19_STABLE
    • Fixed Branches:
      MOODLE_17_STABLE, MOODLE_18_STABLE, MOODLE_19_STABLE

      Description

      Just found a duplicate on my system of a user and tried to alter the firstname of the unused one so I could identify and remove it. I got the error

      Could not update external LDAP, check your server logs (or similar).

      Server error logs say nothing. Does it mean the LDAP server logs?

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              mattgibson Matt Gibson added a comment -

              XAMPP, win2k3 BTW

              Show
              mattgibson Matt Gibson added a comment - XAMPP, win2k3 BTW
              Hide
              iarenaza Iñaki Arenaza added a comment -

              Hi Matt,

              could you try the attached patch? I think this should fix this issue, and provide better logging for the more general case of 'Could not update external LDAP' (which should help diagnose the real origin of the problem).

              Saludos. Iñaki.

              Show
              iarenaza Iñaki Arenaza added a comment - Hi Matt, could you try the attached patch? I think this should fix this issue, and provide better logging for the more general case of 'Could not update external LDAP' (which should help diagnose the real origin of the problem). Saludos. Iñaki.
              Hide
              iarenaza Iñaki Arenaza added a comment -

              Hi Matt,

              have you had a chance to test the patch?

              Saludos. Iñaki.

              Show
              iarenaza Iñaki Arenaza added a comment - Hi Matt, have you had a chance to test the patch? Saludos. Iñaki.
              Hide
              mattgibson Matt Gibson added a comment -

              I've been driving myself nuts trying to get a patch applied on a windows box GNUwin says its garbage.

              Still trying

              Show
              mattgibson Matt Gibson added a comment - I've been driving myself nuts trying to get a patch applied on a windows box GNUwin says its garbage. Still trying
              Hide
              mattgibson Matt Gibson added a comment -

              Tried everything. There must be another way.

              Show
              mattgibson Matt Gibson added a comment - Tried everything. There must be another way.
              Hide
              iarenaza Iñaki Arenaza added a comment -

              Hi Matt,

              it seems your version of patch is choking on the Unix line fomat of the patch file. You can overcome this by opening the file in Wordpad, then saving is as 'Text document - MSDOS format'.

              If this doesn't fix it, then send me an email at iarenuno_at_eteo_dot_mondragon_dot_edu with your ldap/auth.php file, and I'll send you back the file fully patched for you to test it.

              Saludos. Iñaki.

              Show
              iarenaza Iñaki Arenaza added a comment - Hi Matt, it seems your version of patch is choking on the Unix line fomat of the patch file. You can overcome this by opening the file in Wordpad, then saving is as 'Text document - MSDOS format'. If this doesn't fix it, then send me an email at iarenuno_at_eteo_dot_mondragon_dot_edu with your ldap/auth.php file, and I'll send you back the file fully patched for you to test it. Saludos. Iñaki.
              Hide
              mattgibson Matt Gibson added a comment -

              Got it. seems to be fine now.

              Show
              mattgibson Matt Gibson added a comment - Got it. seems to be fine now.
              Hide
              iarenaza Iñaki Arenaza added a comment -

              Matt,

              did the patch fixed the real issue?

              Saludos. Iñaki.

              Show
              iarenaza Iñaki Arenaza added a comment - Matt, did the patch fixed the real issue? Saludos. Iñaki.
              Hide
              mattgibson Matt Gibson added a comment -

              Not sure what the 'real issue' was, but the thing I was trying to do i.e. deleting the duplicate user account, was not a problem after applying the patch.

              Thanks for making the patch BTW, much appreciated.

              Show
              mattgibson Matt Gibson added a comment - Not sure what the 'real issue' was, but the thing I was trying to do i.e. deleting the duplicate user account, was not a problem after applying the patch. Thanks for making the patch BTW, much appreciated.
              Hide
              iarenaza Iñaki Arenaza added a comment -

              Closing now. Thanks for the report and testing!

              Saludos. Iñaki.

              Show
              iarenaza Iñaki Arenaza added a comment - Closing now. Thanks for the report and testing! Saludos. Iñaki.

                People

                • Votes:
                  0 Vote for this issue
                  Watchers:
                  1 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    11/Oct/07