Skip to content
This repository has been archived by the owner on Dec 16, 2022. It is now read-only.

RFC: Adding LDAP authentication support #653

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

doanac
Copy link

@doanac doanac commented Feb 20, 2016

This is an RFC to get your thoughts on adding an LDAP authentication backend to Shout. I've tried to break this up into small changes to make it easy to follow.

I'd like to add more functionality around LDAP (populating more config fields), but would like to get a feel for the general thoughts around this before proceeding.

Andy Doan added 3 commits February 19, 2016 22:38
By adding an optional "token" parameter to the findClient method, we can
simplify the logic of the auth function.

Signed-off-by: Andy Doan <[email protected]>
This uses a callback mechanism to fit the needs of the upcoming LDAP
authentication function.

Signed-off-by: Andy Doan <[email protected]>
This allows logins to authenticate against an LDAP server. If the user
doesn't exist before hand they will be created dynamically.

Signed-off-by: Andy Doan <[email protected]>
@doanac doanac mentioned this pull request Feb 20, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant