search

nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Feature Flaw: Lack email validation for valid email #180

Open nus-se-script opened 1 week ago

nus-se-script commented 1 week ago

Reproduce: run add n/Betsy Crowe t/friend e/betsycrowe@example c/CS1231S p/1234567 t/struggling

Expected: Warning/Alert users of incorrect email

Actual: Screenshot 2024-11-15 at 5.36.57 PM.png

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

juancarlovieri commented 1 week ago

Team's Response

Hi, thanks for your feedback.

However, the email you specified is actually a valid email according to the standard for email address format. The domain part of an email address can also be a local domain name with no TLD.

Therefore, we are rejecting this bug as it is invalid.

Reference: https://tools.ietf.org/html/rfc5322#section-3.4.1

Duplicate status (if any):

--