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

nested groups work when checkbox nested groups is unchecked #699

Open
jnweiger opened this issue Dec 16, 2021 · 2 comments
Open

nested groups work when checkbox nested groups is unchecked #699

jnweiger opened this issue Dec 16, 2021 · 2 comments

Comments

@jnweiger
Copy link
Contributor

Seen with user_ldap 0.16.0 RC2 and owncloud server 10.9.0 RC2

while testing #683 (comment)

  • Follow reproducer steps explained in 683, nested groups work as expected. OK
  • go to advanced tab and disable the nested groups checkbox:
    image
  • nested groups still work.

Expected behaviour: when disabled, nested groups stop working

@jvillafanez
Copy link
Member

I think that it won't work with the "groupScan" algorithm because that was the previous behaviour. For other algorithms, it might work partially without the nested groups checkbox, but there might be issues. Need to investigate.

@jvillafanez
Copy link
Member

countUsersInGroup, getUserGroups seem to be the functions that might cause problems returning "wrong" results.

The countUsersInGroups is expected to count the number of users in a group, including nested groups. If G1 contains G2, and G2 contains U1, U2 and U3, then G1 is expected to count 3 users (assuming nested groups is enabled).
The getUserGroups is expected to return all the groups the user belongs to. In the above example, U1 should return G1 and G2 (assuming nested groups is enabled)

In those 2 functions, there hasn't been any change, and they'll be affected by the nested groups checkbox. I'm not fully sure where they're used, but for the time being I think it's better to be consistent and follow the restrictions.

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

No branches or pull requests

2 participants