LDAP Account support - Push allowed/wanted?

agstrike / silverstrike

Finance Management Made Easy

https://silverstrike.org

MIT License

420 stars 98 forks source link

LDAP Account support - Push allowed/wanted? #136

Open Alsterdetektive1 opened 2 years ago

Alsterdetektive1 commented 2 years ago

Hey, I searched a bit for a good finance Software with ldap Support. So at the end i came to this software which is made by django. LDAP Support Implementation shall be very easy, so i want to ask if it is desired, that i implement it and make a push request to the git.

simhnna commented 2 years ago

sorry I didn't get back to this. I guess this can be handled using an extra dependency and modifying the authentication config?

If something needs to be changed in the code, I'm happy to merge it

Alsterdetektive1 commented 2 years ago

Hey, yeah just need to Import django_auth_ldap as authentication backends:

AUTHENTICATION_BACKENDS = [ 'django_auth_ldap.backend.LDAPBackend', 'django.contrib.auth.backends.ModelBackend', ]

Then there is a need to Set ldap settings in settings.py (e.g. with .env file or within docker-compose) and it is working (https://pypi.org/project/django-auth-ldap/)