Sign-up form password field char limit inconsistency #22

New Issue

simonxciv · 2022-11-25T06:05:46Z

simonxciv commented

2022-11-25 06:05:46 +00:00

When signing up via /auth/sign_up, the password field has maxlength="72" set, however the password confirmation field does not. When filled with a password manager defaulting to a > 72 char password, the password gets truncated in the first field and not the second, resulting in a mismatch that's not evident until the submission fails.

Some client-side validation + adding maxlength attribute on the confirmation field would fix this 😃

When signing up via /auth/sign_up, the password field has `maxlength="72"` set, however the password confirmation field does not. When filled with a password manager defaulting to a > 72 char password, the password gets truncated in the first field and not the second, resulting in a mismatch that's not evident until the submission fails. Some client-side validation + adding maxlength attribute on the confirmation field would fix this 😃

Rin added the

type/bug

area/ux

labels 2022-11-25 06:06:24 +00:00

kouhai was assigned by Rin

2022-11-25 06:06:44 +00:00

Rin self-assigned this 2022-11-25 06:06:44 +00:00

Rin commented

2022-11-25 06:26:35 +00:00

File at fault is app/views/auth/passwords/edit.html.haml. Working on a fix.

File at fault is `app/views/auth/passwords/edit.html.haml`. Working on a fix.

Rin commented