projecthydra/hydra-ldap — pull request #1 by DanCoughlin (2012-07-09 16:49:43+00:00)
Title: Verify users exist

Hey Justin,

We (Mike Giarlo and I) did some work on the hydra-ldap module to try and gain some flexibility for the different LDAP configurations between Northwestern and Penn State. I'm providing a couple links that hope to clarify the work we did and help you decide if it's merge worthy.

1) The diffs
Here is a link on the differences between the master version and what I added in my branch "verify-users-exist" .
Most of our changes were related to abstracting the differences in calls to get group names between Northwestern and Penn State. We did add a few functions to make sure users exist, users are unique, and a group exists. Implementing these changes would require Northwestern to change how they call hydra-ldap, and I've added comments in the code to how they would do that.

2) ScholarSphere updates
Here is a link to our commit that checked in these updates in Scholarsphere .

I am hoping there is some use in this and your able to follow my babbling. Let me know if you have questions.

Thanks!
Dan