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

SSSD should use UID/GID/SID/UUID for RDN attribute in the cache #2609

Closed
sssd-bot opened this issue May 2, 2020 · 0 comments
Closed

SSSD should use UID/GID/SID/UUID for RDN attribute in the cache #2609

sssd-bot opened this issue May 2, 2020 · 0 comments

Comments

@sssd-bot
Copy link

sssd-bot commented May 2, 2020

Cloned from Pagure issue: https://pagure.io/SSSD/sssd/issue/1567

  • Created at 2012-10-05 18:26:48 by sgallagh
  • Closed as Invalid
  • Assigned to nobody

Currently, much of the SSSD LDB cache is keyed on the name of the user, group, etc.

However, names are potentially changeable, whereas in realistic environments the ID never will be.

The benefit to switching would be to simplify the renaming code that we have throughout the SSSD as well as potentially simplifying the AD provider's SID->POSIX ID mapping (for those environments using assigned POSIX IDs instead of the automatic ID-mapping)

Obviously, upgrading to this new organization would need to be carefully thought-out.

Comments


Comment from simo at 2012-10-05 20:07:25

FWIW I do not think we really need to change the database layout (modify the RDN).
We care about anchoring an object to a uuid really only when modifications are made to the database (relatively rare).
What matters is how we match a local database user to its counterpart in the remote server, that can be done by a different attribute in the object, doesn't have to be cached object RDN.


Comment from jhrozek at 2012-10-25 12:48:06

After an IRC discussion with Simo and Stephen we decided not to fix this ticket.We would rather store a RID (and filter by subdomain to avoid matching a user with the same RID coming from two domains).

design: =>
design_review: => 0
fedora_test_page: =>
resolution: => wontfix
status: new => closed


Comment from sgallagh at 2017-02-24 14:54:32

Metadata Update from @sgallagh:

  • Issue set to the milestone: NEEDS_TRIAGE
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant