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

feat: Add ldap authentication #268

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

astenmco
Copy link

@astenmco astenmco commented Jun 29, 2022

Here is how the ldap authentication feature works:

• To activate LDAP authentication, you must enter the URL of the ldap server in the /server/.env file with the LDAP_SERVER variable.
example: LDAP_SERVER=db.debian.org:389
If this value is not provided in this file, Planka will work with normal authentication.

• If this authentication option is activated, it is impossible to modify your password and email from Planka. Indeed, this information must be that of the active directory.

• In the event of an access problem with the ldap server (server is down), only the admin account can connect without going through this ldap server. (This account must be created manually, with email=admin@admin.admin or username=admin.)

@astenmco astenmco mentioned this pull request Jun 29, 2022
@CLAassistant
Copy link

CLAassistant commented Jan 16, 2023

CLA assistant check
All committers have signed the CLA.

@mlopezgva
Copy link

Hi!
Any chance to get this in main branch?
We are about to switch (again!) project / task manager suite, and the team would like something in-house and not remotely similar to Jira (management's proposal...). But, for that, AD / LDAP is a must.

@meltyshev
Copy link
Member

Hi! Any chance to get this in main branch? We are about to switch (again!) project / task manager suite, and the team would like something in-house and not remotely similar to Jira (management's proposal...). But, for that, AD / LDAP is a must.

After adding OIDC and default admin environment variables, it will be easier to accept this PR, but some tweaking and testing needs to be done here. I'll try to get on it when I have time.

@ygranger-oa
Copy link

Hi,
this feature would be really appreciated. Thank you.

@RphCos
Copy link

RphCos commented May 17, 2024

Agreed ! It would be very nice.

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 this pull request may close these issues.

6 participants