Hello, when I try to edit the display username with something useful for us, the process does not complete with a message “Must be less than or equal to 256 characters" even though the size is much less than 256 characters. I use samba AD with more than 400 users.
Is it also possible to have a notes field in the user details that is only visible to the administrator ?
Thank you
Hi Stelios, just to have an idea: how many characters are you trying to enter?
It may be possible, only if both AD and OpenLDAP provide attributes that can store notes. About restricting access to them, this could probably be achieved with LDAP ACLs. It is something to reason about.
Hi Davide, i ‘m trying to update with less than 100 characters, now there are only 27.
For the extra field ( something like notes ) there is no need for restriction, just using for administrator without parsing anywhere.
Thank you
Ok, if LDAP provides a notes attribute and no secrets are stored in it, the implementation is feasible.
If 27 characters trigger the validation error, there is something wrong. I can’t find any limitation in the BE code. @stephdl do you have any idea where the limit comes from?
{
"error": "Validation errors: [display_name: String length must be less than or equal to 32]\n",
"exit_code": 10,
"file": "task/module/samba3/e6c036a6-502a-4630-a147-68a2ca029b92",
"output": [
{
"error": "display_name_string_lte",
"field": "display_name",
"parameter": "display_name",
"value": "stephane de labrusse stephane de labrusse stephane de labrusse"
}
]
}
openldap is not concerned
So, may it change to 256 chars or have to live with only 32 ?
I’ve to check, but I’m afraid that limit is enforced by samba-tool or pdbedit Samba alter-user action schema.
Sorry, my mistake: I looked at ns8-core code, not in ns8-samba.
isssue + pr available at display_name maxLength is too small, should be 256 characters · Issue #8003 · NethServer/dev · GitHub
Thank you Steph, could you file also a draft issue in Backlog for the notes field?
Edit: User/Group note field and LDAP attribute · Issue #8005 · NethServer/dev · GitHub
Thank you Davide and Steph, i can wait for the next core update
hello @Stelios would you like to test the fix, you should use a test machine if possible


