Moodle

Enable user signup with Active Directory (via LDAP).

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 1.8
  • Fix Version/s: 1.9
  • Component/s: Authentication
  • Labels:
    None
  • Environment:
    MS Active Directory
  • Affected Branches:
    MOODLE_18_STABLE
  • Fixed Branches:
    MOODLE_19_STABLE

Description

Currently you users can only signup using the email-based authentication plugin. The proposed patch allows the LDAP authentication plugin to be used, and it specifically adds support to enable user creation in MS Active Directory.

It's being used in production in at least a couple of production sites (see http://moodle.org/mod/forum/discuss.php?d=57378#p314361).

Saludos. Iñaki.

  1. ad-user-creation-18.diff
    28/Apr/07 10:55 PM
    9 kB
    Iñaki Arenaza
  2. ad-user-creation-18-incr.diff
    02/May/07 3:25 AM
    7 kB
    Iñaki Arenaza
  3. ad-user-creation-18-incr.diff
    01/May/07 3:31 PM
    7 kB
    Iñaki Arenaza
  4. ad-user-creation-18-incr.diff
    01/May/07 3:31 PM
    7 kB
    Iñaki Arenaza
  5. ad-user-creation-18-incr.diff
    01/May/07 5:28 AM
    6 kB
    Iñaki Arenaza
  6. ad-user-creation-19.diff
    26/May/07 6:41 AM
    7 kB
    Iñaki Arenaza

Issue Links

Activity

Hide
Iñaki Arenaza added a comment -

As part of this patch is in fact what MDL-9575 tries to fix, I'm attaching a new patch with just the AD user creation part.

Show
Iñaki Arenaza added a comment - As part of this patch is in fact what MDL-9575 tries to fix, I'm attaching a new patch with just the AD user creation part.
Hide
Iñaki Arenaza added a comment -

Ups! I forgot to mention that the new patch is called ad-user-creation-18-incr.diff (to show that it's incremental with respecto to MDL-9575)

Saludos. Iñaki.

Show
Iñaki Arenaza added a comment - Ups! I forgot to mention that the new patch is called ad-user-creation-18-incr.diff (to show that it's incremental with respecto to MDL-9575) Saludos. Iñaki.
Hide
Iñaki Arenaza added a comment -

I've updated the patch to use translatable error strings, instead of hard-coded ones.

Saludos. Iñaki.

Show
Iñaki Arenaza added a comment - I've updated the patch to use translatable error strings, instead of hard-coded ones. Saludos. Iñaki.
Hide
Iñaki Arenaza added a comment -

I'm really sorry, but there is a small typo in the translatable strings that throws a nice syntax error. Here is the updated patch. Sorry for the inconveniences.

Saludos. Iñaki.

Show
Iñaki Arenaza added a comment - I'm really sorry, but there is a small typo in the translatable strings that throws a nice syntax error. Here is the updated patch. Sorry for the inconveniences. Saludos. Iñaki.
Hide
Petr Škoda (skodak) added a comment -

hello, I would like to get this into HEAD, but it does not apply cleanly after my recent commits, could you please prepare a new patch? thanks!

Show
Petr Škoda (skodak) added a comment - hello, I would like to get this into HEAD, but it does not apply cleanly after my recent commits, could you please prepare a new patch? thanks!
Hide
Iñaki Arenaza added a comment -

OK, here is the updated version for HEAD.

Saludos. Iñaki.

Show
Iñaki Arenaza added a comment - OK, here is the updated version for HEAD. Saludos. Iñaki.
Hide
Petr Škoda (skodak) added a comment -

committed into cvs, thanks a lot!

Show
Petr Škoda (skodak) added a comment - committed into cvs, thanks a lot!

People

Vote (1)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: