Fixed code example for Querying LDAP section
Merge pull request #142 from matheustardivo/master

Fixed code example for Querying LDAP section
Change authenticate_with_ldap to find_for_ldap_authentication. This both better matches the structure of Devise's database_authenticatable and allows ldap to work alongside lockable. Refactor ldap strategy to work with these changes.
Fix the specs broken by changing the authenticate_with_ldap method name.a
Authenticate users even on find, so local records aren't created when they shouldn't be.
Merge pull request #152 from aaronklaassen/lockable-compatibility

Devise Lockable compatibility
new optional advanced parameter for creating custom password encription
devise 3.2.2, secret key compatiblity
updated readme
update version
Merge pull request #154 from activars/custom_password_generator

Custom password encryption builder
adding strip_whitespace_keys
Merge pull request #160 from mathieumahe/strip_whitespace_keys

adding strip_whitespace_keys
Check User status in LDAP Group w/out admin auth

Should allow a user to check group status without requiring admin authentication.
Not including AD at this point.

Signed-off-by: "Jake Champlin" <jchamplin@jake.champlin.27@gmail.com>

Testing Group List w/out admin auth

Should allow LDAP user to search group membership without authenticating as admin LDAP
user.

Signed-off-by: "Jake Champlin" <jake.champlin.27@gmail.com>

Check for group status w/out admin auth

Need to dev better way of testing changes. Laziness ensues.

Signed-off-by: "Jake Champlin" <jake.champlin.27@gmail.com>

Forgot return statement. Silly

Not too bright.

Signed-off-by: "Jake Champlin" <jake.champlin.27@gmail.com>

Testing still. Returning all known groups.

Hopefully will return all LDAP groups with filter in place.

Signed-off-by: "Jake Champlin" <jake.champlin.27@gmail.com>

Basic Commits

Commit 6.

Signed-off-by: "Jake Champlin" <jake.champlin.27@gmail.com>

Commit.

Unsure of how to test gems locally without pulling into a ruby app

Signed-off-by: "Jake Champlin" <jake.champlin.27@gmail.com>

Commit.

gah.

Signed-off-by: "Jake Champlin" <jake.champlin.27@gmail.com>

Debugging statements

Just work

Signed-off-by: "Jake Champlin" <jake.champlin.27@gmail.com>

Complete shit commit

Horrible coding practices.

Signed-off-by: "Jake Champlin" <jake.champlin.27@gmail.com>

Cleaning up commit.

Commit Cleanup.

Signed-off-by: "Jake Champlin" <jake.champlin.27@gmail.com>

Cleanup Commit

Debugging Still.

Signed-off-by: "Jake Champlin" <jake.champlin.27@gmail.com>

It works

Signed-off-by: "Jake Champlin" <jake.champlin.27@gmail.com>

Working Gem so far

Gem works to find groups to authenticate by without neededing admin auth

Signed-off-by: "Jake Champlin" <jake.champlin.27@gmail.com>

Works Still.

Still works as a passable gem.

Signed-off-by: "Jake Champlin" <jake.champlin.27@gmail.com>

Debugging statements

Debugs

Signed-off-by: "Jake Champlin" <jake.champlin.27@gmail.com>

Adding Check Membership for Auth

Adding membership check without admin authentication as a config option.

Signed-off-by: "Jake Champlin" <jake.champlin.27@gmail.com>

Fixed a spelling error

Took 10 minutes. Wow

Signed-off-by: "Jake Champlin" <jake.champlin.27@gmail.com>

Added logging statement for Auth

Added logging statement for authentication tests

Signed-off-by: "Jake Champlin" <jake.champlin.27@gmail.com>
Fixed Formatting Issues for Patch.

Fixed formatting issues to comply with repo's coding style.

Signed-off-by: "Jake Champlin" <jake.champlin.27@gmail.com>
RSpec Tests for non-admin binded auth

Quick RSPEC acceptance test for authorizing a user group without binding to admin user
account.

Signed-off-by: "Jake Champlin" <jake.champlin.27@gmail.com>
Updates for Rails 4.1 and more recent RSpec interface and usage changes.
Updating readme with more test instructions.
Merge branch 'admin_auth' of github.com:jchampy/devise_ldap_authenticatable into jchampy-admin_auth
Adding docs/tests around new ldap_check_group_membership_without_admin

Removed a couple extraneous internal functions to slim down the logic
from the initial PR. Also, removed the user_in_ldap_group? Devise model
method that was implied but not fully added in the PR. If we have a
static option, it seems redundant. The only thing the extra model method
would accomplish would be to allow model users to specify whether to use
the admin LDAP binding to check group membership in the explicit lookup
function (the static config is used for the auto group checking during
the required groups login flow). This seems like a pretty extraordinary
case of an app; namely, it seems like the only case you'd ever want to
check group membership with non-admin credentials would be if you can't
get or don't want to use admin credentials. I can't think of a plausible
use case where you'd have the credentials and would benefit from an
admin check later on where you wouldn't just want to use those admin
credentials globally (and thus leave the new option at its default
false).