polonel / trudesk

:coffee: :seedling: Trudesk is an open-source help desk/ticketing solution.
http://trudesk.io
Other
1.31k stars 442 forks source link

Username truncated to 25 characters causing inconvenience for accounts created by email #555

Closed mrt-prodz closed 1 year ago

mrt-prodz commented 2 years ago

Is this a BUG REPORT or FEATURE REQUEST?:

What happened: Username and fullname is truncated to 25 characters and it is causing a small inconvenience with accounts created from email resulting in a truncated email as login.

What did you expect to happen: Login for an account created through email would be better if it was the full email.

How to reproduce it (as minimally and precisely as possible): Using an email address above 25 characters length and using Trudesk with mailcheck activated :

Anything else we need to know?: This might be silly to many users, but for us it's really the very last issue before deploying Trudesk internally, I know people will fail to read the account creation email properly and ask us why they can't login to see their tickets.

All our emails are in the [firstname].[lastname]@[domain].[ext] format, so going above 25 characters is quite easy.

On our environment I fear that changing :

https://github.com/polonel/trudesk/blob/5bbc42395de7d2589daa97d6837500826448ac11/src/models/user.js#L100

to

user.username = utils.applyMaxTextLength(utils.sanitizeFieldPlainText(user.username.toLowerCase().trim()))

could cause issue somewhere else, however it would successfully change username length from 25 to 255 characters.

Using grep I checked where applyMaxShortTextLength was called and models/user.js is the only place, but I'm not sure about potential UI issue. I will test it and see if username (and fullname) above this limit could break something.

I could also create a new constant / method in utils/index.js with a 64 characters limit to keep login length reasonable.

What is your opinion about this ?

Environment:

github-actions[bot] commented 1 year ago

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

github-actions[bot] commented 1 year ago

This issue was closed because it has been stalled for 5 days with no activity.