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

NAS-129889 / 24.10 / Fix schema for directory services users #13972

Merged
merged 2 commits into from
Jul 3, 2024

Conversation

anodos325
Copy link
Contributor

We impose more restrictive validation on our local usernames than NSS on Linux imposes. nss_winbind for instance uses \ as a separator for domain name and username to prevent collisions with local users.

This commit fixes our validator for UserEntry but maintains for UserCreate and fixes various minor issues with schema returned by directory services cache.

We impose more restrictive validation on our local usernames than
NSS on Linux imposes. nss_winbind for instance uses \ as a separator
for domain name and username to prevent collisions with local users.

This commit fixes our validator for UserEntry but maintains for
UserCreate and fixes various minor issues with schema returned by
directory services cache.
@anodos325 anodos325 requested a review from themylogin July 3, 2024 15:45
@anodos325 anodos325 added the jira label Jul 3, 2024
@bugclerk
Copy link
Contributor

bugclerk commented Jul 3, 2024

@bugclerk bugclerk changed the title Fix schema for directory services users NAS-129889 / 24.10 / Fix schema for directory services users Jul 3, 2024
@anodos325 anodos325 requested a review from yocalebo July 3, 2024 16:55
Copy link
Contributor

@yocalebo yocalebo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Really nice

@anodos325 anodos325 merged commit bef6b6f into master Jul 3, 2024
3 checks passed
@anodos325 anodos325 deleted the fix-ds-users-schema branch July 3, 2024 17:14
@bugclerk
Copy link
Contributor

bugclerk commented Jul 3, 2024

This PR has been merged and conversations have been locked.
If you would like to discuss more about this issue please use our forums or raise a Jira ticket.

@truenas truenas locked as resolved and limited conversation to collaborators Jul 3, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants