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

group-user association attribute changed after click on "count groups" #12682

Closed
blizzz opened this issue Nov 27, 2018 · 1 comment · Fixed by #15604
Closed

group-user association attribute changed after click on "count groups" #12682

blizzz opened this issue Nov 27, 2018 · 1 comment · Fixed by #15604

Comments

@blizzz
Copy link
Member

blizzz commented Nov 27, 2018

In the LDAP configuration, the user-group-association is set on uniqueMember (the default). Also available is "memberUid", which has downsides. Groups are checked and available.

After a while revisit the configuration. When you click on “Count Groups”, it kicks off the detection of the association attribute (when it is set to default, which happens to be uniqueMember). If the more or the same amount of users (→ sorting) are returned for both attributes, memberUid will be used instead of uniqueMember.

It should not change in this case.

From https://help.univention.com/t/nextcloud-13-group-sync-issues/10337

@nextcloud-bot

This comment has been minimized.

blizzz added a commit that referenced this issue May 17, 2019
* uniqueMember was the default so we did not know whether this setting is
  desired or the initial value
* autodetection of the user-group association attribute runs only when it
  was not set (as far as we knew)
* the default is now empty
* thus LDAPProvider might return this value as well (in exceptional cases)
* if a group base is given (edge case), use this instead of general base
* resolves #12682

Signed-off-by: Arthur Schiwon <[email protected]>
@MorrisJobke MorrisJobke added this to the Nextcloud 17 milestone May 22, 2019
nachoparker pushed a commit to nachoparker/server that referenced this issue Jun 28, 2019
* uniqueMember was the default so we did not know whether this setting is
  desired or the initial value
* autodetection of the user-group association attribute runs only when it
  was not set (as far as we knew)
* the default is now empty
* thus LDAPProvider might return this value as well (in exceptional cases)
* if a group base is given (edge case), use this instead of general base
* resolves nextcloud#12682

Signed-off-by: Arthur Schiwon <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants