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

Updating Name only updates some names #74

Open
mxmeinhold opened this issue Aug 30, 2020 · 4 comments
Open

Updating Name only updates some names #74

mxmeinhold opened this issue Aug 30, 2020 · 4 comments
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@mxmeinhold
Copy link
Contributor

When a user sets name, profiles only updates cn, however cn is not the only reference to a user's name, so this leaves potentially outdated or dead names in the givenName, initals, sn, and gecos fields, or otherwise allow a user to see and update those fields.

@mxmeinhold mxmeinhold added bug Something isn't working good first issue Good for newcomers labels Aug 30, 2020
@rbuzzell
Copy link

I was just about to report this. This still exists as an issue.

@JayHors JayHors mentioned this issue Aug 17, 2021
@Mstrodl
Copy link
Member

Mstrodl commented Sep 8, 2022

FWIW: The purpose of cn isn't quite the same as those other fields

For example: cn=Lonnie,givenName=Lon,sn=Gerol

I think in a typical ldap setup, the way we have it is pretty normal. @mbillow could probably tell me I'm full of shit though

@mbillow
Copy link
Member

mbillow commented Sep 9, 2022

The right course of action here, imo, is to break first and last name (possibly also add a nickname option) into different fields and then compute the correct values for each of the LDAP attributes on submission.

That saves people from having to manually edit each field, but also gives them the flexibility to set their name to whatever they want.

@Mstrodl
Copy link
Member

Mstrodl commented Sep 13, 2022

The right course of action here, imo, is to break first and last name (possibly also add a nickname option) into different fields and then compute the correct values for each of the LDAP attributes on submission.

That saves people from having to manually edit each field, but also gives them the flexibility to set their name to whatever they want.

For some people, given name follows family name, so constructing a name in this way doesn't work for everyone

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants