ChiliProject is not maintained anymore. Please be advised that there will be no more updates.
We do not recommend that you setup new ChiliProject instances and we urge all existing users to migrate their data to a maintained system, e.g. Redmine. We will provide a migration script later. In the meantime, you can use the instructions by Christian Daehn.
Add LDAP filter to ldap authentication (Feature #388)
An LDAP filter will provide a simple way to restrict the access to ChiliProject to a certain group of users. A filter can be used in many different ways and as it ist entirely definable by the admin, it can suit many different setups.
This patch will requiere an update of Net::LDAP (Bug #258). To update this manually you only need to delete to folder vendor/plugins/ruby-net-ldap-0.0.4/ and download the new version and put it in /ver/plugins/. The current (0.2.2) version can be found here: http://rubyforge.org/projects/net-ldap/
The patch was created for Redmine but sadly gets no attention there. Maybe you can give me some feedback.
Wow, this is almost identical to an LDAP filter I wrote for a client (part of code). I think we can add it but I might use my version because I think I have some tests for it too. I also didn't have to upgrade Net::LDAP at all.
Do you have a working LDAP installation? I have a bunch of other LDAP code I'd like to get a second opinion on too (failover, basic ldap groups, pulling LDAP attributes into ChiliProject custom fields).
- Assignee set to Eric Davis
yes, I have an working LDAP setup. I can test your changes if you like... but the I have to install ChiliProject first :)
The update of Net::LDAP was not required for this patch but for the things I wrote into the filter. As far as I remember, the filter did not work if it contained a space character.
I just had a quick look at your pull request and it looks good so far, though I have some comments, I've added them to the pull request on github. Also, net-ldap has been moved from vendor/plugins to the gemfile.
Could you update your pull request to the current chiliproject master and include the comments from github? Thanks!
BTW: The pull request is at https://github.com/chiliproject/chiliproject/pull/49
New pull request is here: https://github.com/chiliproject/chiliproject/pull/142
This is a rewrite of the patch with code from Eric Davis including updated tests. I tested the code and it works fine here.