stable
Clone or download
Read-only
request #7239 - LDAP/Tuleap user synchronization no longer retrieve extra attributes
Regression introduced by request #7151 (commit feb42fe) Before that modification, Tuleap retrieved all LDAP default attributed. After this patch, if a site use a custom LDAP_UserSync (in /etc/tuleap/plugins/ldap/site-content/synchronize_user.txt) and rely on extra paramters for custom synchronization, the process will be broken. This commit rewrite (with test) the LDAP authentication and ensure that extra params are fetched if they are defined. Change-Id: I28ff80b102de9fe1e6be1f79f1583c7f580cfa6b
Modified Files
| Name | ||||
|---|---|---|---|---|
| M | plugins/ldap/bin/convertSvnToLdap.php | +1 | −1 | Go to diff View file |
| M | plugins/ldap/bin/registerUser.php | +1 | −1 | Go to diff View file |
| M | plugins/ldap/include/LDAP.class.php | +9 | −4 | Go to diff View file |
| A | plugins/ldap/include/LDAP_AuthenticationFailedException.class.php | +25 | −0 | Go to diff View file |
| M | plugins/ldap/include/LDAP_BackendSVN.class.php | +1 | −1 | Go to diff View file |
| M | plugins/ldap/include/LDAP_DirectorySynchronization.class.php | +1 | −1 | Go to diff View file |
| M | plugins/ldap/include/LDAP_GroupManager.class.php | +1 | −1 | Go to diff View file |
| M | plugins/ldap/include/LDAP_UserGroupManager.class.php | +1 | −1 | Go to diff View file |
| M | plugins/ldap/include/LDAP_UserManager.class.php | +55 | −2 | Go to diff View file |
| A | plugins/ldap/include/LDAP_UserNotFoundException.class.php | +25 | −0 | Go to diff View file |
| M | plugins/ldap/include/autoload.php | +4 | −2 | Go to diff View file |
| M | plugins/ldap/include/ldapPlugin.class.php | +23 | −44 | Go to diff View file |
| M | plugins/ldap/tests/LDAP_UserManagerTest.php | +210 | −3 | Go to diff View file |
| M | plugins/ldap/tests/bootstrap.php | +65 | −0 | Go to diff View file |
| M | plugins/ldap/www/welcome.php | +1 | −1 | Go to diff View file |
| M | plugins/requesthelp/include/RequestHelpActions.class.php | +1 | −1 | Go to diff View file |