Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for authentication through federated identity providers #47

Closed
NicolasCARPi opened this issue Jul 9, 2014 · 11 comments
Closed

Comments

@NicolasCARPi
Copy link
Contributor

Add LDAP support.

@NicolasCARPi NicolasCARPi changed the title ROADMAP Add LDAP support Nov 1, 2014
@sulian
Copy link

sulian commented May 24, 2016

👍

@gecgooden
Copy link

gecgooden commented Oct 31, 2016

Has there been any progress on this feature?
If not, could you suggest where I should look to so I can start working on it? I assume it'd be best to add the logic to app/classes/Auth.php?

@NicolasCARPi
Copy link
Contributor Author

Hello,

There hasn't been any progress yet. Indeed Auth would be a good place to start. The checkCredentials() method should probably be the one to modify to access an LDAP server.

@gecgooden
Copy link

Cool, I'll have a look at that soon. Thanks!

@NicolasCARPi
Copy link
Contributor Author

NicolasCARPi commented Dec 18, 2016

This can be a good inspiration: https://wordpress.org/plugins/simple-ldap-login/

And this to know how it works: http://www.brennan.id.au/20-Shared_Address_Book_LDAP.html

@kirkog86
Copy link

kirkog86 commented Feb 7, 2017

Hi all, any update regarding LDAP support?

@gecgooden
Copy link

Unfortunately, I've not had the opportunity to work on it, so no progress has been made.

@kirkog86
Copy link

kirkog86 commented Feb 7, 2017

OK. I'll start to work on it. Will update on progress.

@NicolasCARPi
Copy link
Contributor Author

Leaving this here: https://github.com/Jasig/phpCAS

@NicolasCARPi
Copy link
Contributor Author

So here is what will happen: there will not be a LDAP integration, but something better: an saml2 mechanism for identity, that can work with identity federations (like edugain, or like the one in your institute). Behind the IDP, it can be LDAP or anything else (probably LDAP anyway), but that doesn't concern elabftw, because of this layer of abstraction :)

I'll begin working on it after releasing the (hopefully) last patch for 1.5 branch. I'm leaving this issue open, but renaming it.

@NicolasCARPi NicolasCARPi changed the title Add LDAP support Add support for authentication through federated identity providers Mar 10, 2017
@NicolasCARPi
Copy link
Contributor Author

The code is written. Now I need to test it in real conditions and also allow flexibility to work with the different attributes used by each organization.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

4 participants