[MINOR BUG] Trailing whitespace in email fields #450

Closed
opened 2025-11-13 12:00:59 -06:00 by GiteaMirror · 3 comments
Owner

Originally created by @yellowsnow3 on GitHub (Jun 19, 2025).

Originally assigned to: @miloschwartz on GitHub.

What happened: I invited a user and put in their email address, on the invite page the user autofilled the email with their clipboard, but with a space behind it. This caused Pangolin to reject the signup, they were only allowed to sign up when the field contained only the email without the space.

A fix would be just cutting out any whitespace before and after email addresses or other fields.

This issue would be eliminated by implementing the Email autofill in #923, but the issue may still be present in other fields across the whole project that I did not check.

Originally created by @yellowsnow3 on GitHub (Jun 19, 2025). Originally assigned to: @miloschwartz on GitHub. What happened: I invited a user and put in their email address, on the invite page the user autofilled the email with their clipboard, but with a space behind it. This caused Pangolin to reject the signup, they were only allowed to sign up when the field contained only the email without the space. A fix would be just cutting out any whitespace before and after email addresses or other fields. This issue would be eliminated by implementing the Email autofill in #923, but the issue may still be present in other fields across the whole project that I did not check.
GiteaMirror added the needs investigatingnon-critical bug labels 2025-11-13 12:00:59 -06:00
Author
Owner

@github-actions[bot] commented on GitHub (Jul 4, 2025):

This issue has been automatically marked as stale due to 14 days of inactivity. It will be closed in 14 days if no further activity occurs.

@github-actions[bot] commented on GitHub (Jul 4, 2025): This issue has been automatically marked as stale due to 14 days of inactivity. It will be closed in 14 days if no further activity occurs.
Author
Owner

@yellowsnow3 commented on GitHub (Aug 24, 2025):

Issue fixed in #1129 (merged in 1.9.0).

@yellowsnow3 commented on GitHub (Aug 24, 2025): Issue fixed in #1129 (merged in 1.9.0).
Author
Owner

@oschwartz10612 commented on GitHub (Aug 24, 2025):

Thanks :}

@oschwartz10612 commented on GitHub (Aug 24, 2025): Thanks :}
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/pangolin#450