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

LDAP - more flexibility in group-member-association #7301

Open
Peter2121 opened this issue Nov 27, 2017 · 5 comments
Open

LDAP - more flexibility in group-member-association #7301

Peter2121 opened this issue Nov 27, 2017 · 5 comments
Labels
1. to develop Accepted and waiting to be taken care of enhancement feature: ldap

Comments

@Peter2121
Copy link

Peter2121 commented Nov 27, 2017

Actually, user_ldap application does not permit to precise the exact configuration of the directory in terms of groups membership. The only parameter we can choose is 'Group-Member association', where one cannot mention a particular attribute's name, only selection from drop-down list is possible. Later, the application use this choice to select an attribute name AND select the supposed type of content of this attribute (DN or UID).
Unfortunately, there is no standard of groups implementation in LDAP. There are several types of implementation that cannot be used with NextCloud. So, NextCloud should be a little bit more flexible:

  • Ask the name of attribute, used for group-member association at the level of groups (membreuid, members,...), the name of the attribute could be done as a text, not only as a selection from a list;
  • Ask the type of data, used in this attribute (DN or an attribute's content);
  • If the type of data is NOT DN - ask the name of user's attribute, used to create the association (uid, mail, samaccoutname,...).

I think that such improvement (with good documentation) could decrease the number of questions about LDAP configuration and bug-reports about LDAP groups not working.

@MorrisJobke
Copy link
Member

cc @nextcloud/ldap

@MorrisJobke MorrisJobke added 0. Needs triage Pending check for reproducibility or if it fits our roadmap enhancement feature: ldap labels Nov 27, 2017
@blizzz
Copy link
Member

blizzz commented Nov 27, 2017

Good idea. Implementation requires some refactoring of this handling, but would improve the code in fact. Personally, I won't have resources for this however. As always, PRs are welcome and supported :)

@blizzz
Copy link
Member

blizzz commented Nov 27, 2017

Ah, that said, I would leave the most common options as they are and only offer the switching to a self-defined attribute as another option. Otherwise it would be just too complicated for people with little knowledge of LDAP. And too inconvenient for others.

@Peter2121
Copy link
Author

Sure, all this staff should be available in 'expert' tab, the default options are not so bad ;)

@blizzz blizzz changed the title LDAP - more flexibility LDAP - more flexibility in group-member-association Nov 27, 2017
@nextcloud-bot nextcloud-bot added the stale Ticket or PR with no recent activity label Jun 20, 2018
@skjnldsv skjnldsv removed stale Ticket or PR with no recent activity labels Jun 12, 2019
@Rincewind34
Copy link

Any progress on this feature? What exactly does "Dynamic Group Member URL" in advanced->directory settings? Seems that's not documented at all.

@skjnldsv skjnldsv added 1. to develop Accepted and waiting to be taken care of and removed 0. Needs triage Pending check for reproducibility or if it fits our roadmap labels Aug 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop Accepted and waiting to be taken care of enhancement feature: ldap
Projects
None yet
Development

No branches or pull requests

6 participants