search

cydrobolt / polr

:aerial_tramway: A modern, powerful, and robust URL shortener
https://polrproject.org
GNU General Public License v2.0
4.98k stars 890 forks source link

LDAP authentication #144

Open MarekPikula opened 8 years ago

MarekPikula commented 8 years ago

Hi, are there plans to support LDAP as authentication backend?

cydrobolt commented 8 years ago

This isn't currently planned, but I will look into this and see if there is a viable solution. In the meantime, you can sync LDAP users into Polr using intermediary scripts.

MarekPikula commented 8 years ago

I'm not really in a hurry. I see that YOURLS have this implemented via PHP's LDAP module.

cydrobolt commented 8 years ago

Oh, that is interesting. I'll take a look at their code. Thanks!

opm22 commented 7 years ago

That would be great for some business deployments. Any news on this subject?

cydrobolt commented 7 years ago

This (https://github.com/Adldap2/Adldap2-Laravel/blob/master/readme.md) seems to be a viable way to implement LDAP authentication in Lumen. However, the specific implementation will be more complicated.

Do we create a different login form for users with LDAP? Is it an optional module?

Adding LDAP authentication would require enabling certain modules and facades in the app. Do we include these service providers and modules by default, or do we require enterprise users to in comment the modules in order to use the functionality?

opm22 commented 7 years ago

I think it should be optional although any medium deployment probably will choose the ldap integration.

spencerryan commented 7 years ago

Another +1 for LDAP support.

HannanAkbar commented 6 years ago

+1

jacek-jonca commented 6 years ago

+1