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 LDAP as authentication backend #202

Closed
nitnelave opened this issue Apr 8, 2021 · 4 comments
Closed

Add LDAP as authentication backend #202

nitnelave opened this issue Apr 8, 2021 · 4 comments

Comments

@nitnelave
Copy link

Hello,

It would be great to have a possibility to re-use an existing LDAP backend for users. It shouldn't be too hard with django, hopefully.

Thanks,

@cdubz
Copy link
Member

cdubz commented Apr 8, 2021

There is a django-auth-ldap/django-auth-ldap plugin that would support this -- it looks well maintained but requires quite a bit of config. I don't have an LDAP server to test with but I'd be happy to consider a PR for this. Theoretically one could also just install the package on an existing BabyBuddy deployment and then configure it per their documentation.

@cdubz cdubz added enhancement Feature requests or improvements to existing functionality help wanted Issues where maintainers could use assistance from others labels Apr 8, 2021
@nitnelave
Copy link
Author

I could have a look, but I'm not familiar with Django. From the plugin's page, I need to add some configuration (in python), but I'm not sure where to add it. Babybuddy doesn't seem to have a settings.py or equivalent.

@nitnelave
Copy link
Author

Ah, I found it in babybuddy/settings/base.py. I'll have a look, and might also add authentication via REMOTE_USER header as well.

@cdubz
Copy link
Member

cdubz commented Nov 5, 2022

We have support for REMOTE_USER now via REVERSE_PROXY_AUTH so I think this use case is covered well enough.

@cdubz cdubz closed this as completed Nov 5, 2022
@cdubz cdubz removed enhancement Feature requests or improvements to existing functionality help wanted Issues where maintainers could use assistance from others labels Nov 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants