This should be in the to ldap mapping pulldowns:

Options should include
-- password entered on user create form, defaulting to good random password
-- good random password

password should be encoded, as all values passed to ldap.

Comments

johnbarclay’s picture

johnbarclay commented

I added the password pulldown for provisioning to ldap entries. I committed it but haven't run the simpletests or documented the options. Here are the options:

  • Pwd: Random -- uses a random drupal generated password
  • Pwd: User -- uses password supplied on user creation form. if none available should not provision account (*not fullly implemented yet).
  • Pwd: User or Random -- uses password supplied on user generated form. if none available uses random password
  • Pwd: User or None -- uses password supplied on user generated form. if none available does not provide a password, but still provisions account (* not fully implemented)

at admin/config/people/ldap/user/test drill down in the array to:
provisionLdapEntry method results > context that is enabled > proposed
to see the way the mappings will behave.

johnbarclay’s picture

johnbarclay commented
Status: Active » Needs review

This is finished except for unit testing. And is committed to 7.x-2.x-dev.

The way the password options work are explained in the UI beneath the mapping to ldap table.

Needs review, testing and simpletest coverage.

Here's the commit: http://drupalcode.org/project/ldap.git/commitdiff/b4be558a40445e3e7d2fbe...

johnbarclay’s picture

johnbarclay commented
Component: Code » SimpleTests
nonsie’s picture

nonsie
she/her
Primary language Estonian
commented

#1 is not entirely correct. Pwd: User and Pwd: User or None are not currently available options.

johnbarclay’s picture

johnbarclay commented
Status: Needs review » Needs work

I committed a related patch #1884922-13: LDAP User: Password field disabled Makes use case of Provisioning Passwords from Drupal to LDAP unusable related to password behavior and provisioning.

http://drupalcode.org/project/ldap.git/commitdiff/cb3dede7d354e4a6c0bd9f...

larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
commented
Issue summary: View changes
Issue tags: +Needs tests

  • johnbarclay committed b4be558 on 8.x-3.x 
    Issue #1718200: options for provisioning passwords from drupal to ldap...
grahl’s picture

grahl
he/him
Location Zürich
commented
Component: SimpleTests » Code
Status: Needs work » Closed (outdated)
Issue tags: -Needs tests

Tests won't be written for 7.