thorin / redmine_ldap_sync

[UNMAINTAINED] A redmine plugin to synchronize both users and groups with an ldap server
http://www.redmine.org/plugins/redmine_ldap_sync
GNU General Public License v3.0
140 stars 129 forks source link

Synchronise locked users #222

Closed marius-balteanu closed 5 years ago

marius-balteanu commented 6 years ago

I'm working again on the integration with Redmine People and I've observed that the user fields are not synchronised for locked users (https://github.com/thorin/redmine_ldap_sync/blob/master/lib/ldap_sync/infectors/auth_source_ldap.rb#L120).

Is there any reason for this restriction? There are certain cases where this sync is useful. If you are interested to change the behaviour, I can provide a PR with this change or even better, with a setting to allow admin users to configure it.