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 inheritance and Indirect group membership #13

Open
nitnelave opened this issue Jun 28, 2021 · 7 comments
Open

Group inheritance and Indirect group membership #13

nitnelave opened this issue Jun 28, 2021 · 7 comments
Labels
backend Issues that require a backend change enhancement New feature or request

Comments

@nitnelave
Copy link
Member

More of a nice-to-have, I don't expect many users to have a very complex group membership structure that requires that.

We can make a group inherit from another one, or be a subgroup of another one: if group A has subgroup B, then users in B are indirectly part of group A.

To avoid making too many requests for reading (common case), we can keep a "resloved" membership table that contains both direct and indirect memberships. This can be updated when adding a user to a group, and can be reset when deleting a user or a group.

Another way to do it is to have just the resolved group inheritance, not the full user one; then a user is part of group A if:

  • They are explicitly a member of A.
  • They are explicitly a member of a group that resolves to inherit from A.
    That's achievable with a single query, with a join.
    This would only require updates/rebuilding when adding/deleting groups/group inheritance.
@nitnelave nitnelave added the enhancement New feature or request label Jun 28, 2021
@nitnelave
Copy link
Member Author

@nitnelave nitnelave added the backend Issues that require a backend change label Jul 15, 2022
@nitnelave
Copy link
Member Author

Design for the feature, feel free to comment on it: https://docs.google.com/document/d/1xkUFGdjfbTI5rC_sh1QIRET6njsK3Ci2ueGAWO-FJY4/edit?usp=drivesdk

@lordratner
Copy link
Contributor

1000% want this feature. It's really the only thing I think is truly "missing" from LLDAP.

@jacobw
Copy link
Contributor

jacobw commented Oct 12, 2023

@lordratner
Copy link
Contributor

Is this on the radar still? Just wondering for planning purposes.

@nitnelave
Copy link
Member Author

Yes, it is. However, expect development to be slow in the next few months (like it has been in the past few months) due to personal circumstances.

@lordratner
Copy link
Contributor

Yes, it is. However, expect development to be slow in the next few months (like it has been in the past few months) due to personal circumstances.

No worries. LLDAP is the best game in town, thank you for what you've done already.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Issues that require a backend change enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants