nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Email Format #1289

Open nus-pe-bot opened 6 days ago

nus-pe-bot commented 6 days ago

Expected: DOMAIN must be at least 2 characters long and start and end with alphanumeric characters.

image.png

Actual: DOMAIN does not accept a.a, which is 3 characters long and starts and end with alphanumeric characters in accordance with the expectation.

Steps to reproduce:

  1. List (provided the list is not empty)
  2. edit 1 e/test@a.a

image.png

Low severity since the email domain is usually not a.a or similar.


[original: nus-cs2103-AY2425S1/pe-interim#543] [original labels: severity.Low type.FunctionalityBug]

jarrellc1 commented 5 days ago

Team's Response

The email test@a.a fails validation because the DOMAIN rule in the UG requires it to be at least 2 characters long and start and end with alphanumeric characters. While a.a is 3 characters overall, the top level domain(TLDs), the "a" after "." is only 1 character, violating the intended interpretation of the rule. This behaviour aligns with the UG and practical email validation standards, which assume TLDs are at least 2 characters long. In summary "a.a" is by default an invalid email structure to begin with, and will not be accepted in the system.

Duplicate status (if any):

--