E0735389 / pe

0 stars 0 forks source link

Correct email address rejected #12

Open E0735389 opened 5 months ago

E0735389 commented 5 months ago
edit 2 /email a..b@nus.edu.sg

Emails should be of the format local-part@domain and adhere to the following constraints:
1. The local-part should only contain alphanumeric characters and these special characters, excluding the parentheses, (+_.-). The local-part may not start or end with any special characters.
2. This is followed by a '@' and then a domain name. The domain name is made up of domain labels separated by periods.
The domain name must:
    - end with a domain label at least 2 characters long
    - have each domain label start and end with alphanumeric characters
    - have each domain label consist of alphanumeric characters, separated only by hyphens, if any.

The email address follows the instruction exactly.

nus-se-bot commented 5 months ago

Team's Response

Thank you for bringing this up! While we understand your observation, we want to emphasize that this issue only occurs when two or more special characters are in a row, which is quite uncommon since most people don't use multiple special characters consecutively. Thus, cases like 'edit 2 /email a.b@gmail.com' or 'edit 2 /email a.b.c@gmail.com' should work without any problems. Given that this impacts a small fraction of users who use two or more consecutive special characters, we don't consider this a medium-severity issue.

Additionally, our research shows that most email standards, including Gmail (the largest email provider with over 1.5 billion users), do not allow two or more consecutive special characters in an email address.

Screenshot 2024-04-21 at 9.15.42 PM.png

Regardless, we will update the instructions to be clearer. Thanks

Items for the Tester to Verify

:question: Issue severity

Team chose [severity.VeryLow] Originally [severity.Medium]

Reason for disagreement: The bug is not just cosmetic, it is a factual error in the description of how the program validates email address, so severity.VeryLow would be unsuitable.