nus-cs2103-AY2324S1 / pe-dev-response

0 stars 0 forks source link

"\" not allowed in Email Format #396

Open nus-se-script opened 1 year ago

nus-se-script commented 1 year ago

"\" is an acceptable email character but isn't allowed.

Screenshot 2023-11-18 at 5.03.13 PM.png


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

TehOPanas commented 1 year ago

Team's Response

Given that we can only do so much with just validation regex and no DNS lookup to validate an email address, we can not 100% guarantee to cover all possible correct email.

There is also not an standardized validation format that every mail service provider follows, we can only try to use the stricter and more popular options provided by reputable parties.

Our current constraint on email is based of OWASP validation regex.

I think that trying to accommodate all possible cases of email will lead to more ambiguity and the effort required to do so definitely exceeds the value it can provide, thus we will only consider it for future increments.

(Changes to FeatureFlaw: Tester does not provide evidence that \ is an acceptable character and after searching online, I see no concrete evidence supporting it. Thus, I believe that this is an invalid email, meaning that it is not a functionality bug)

Duplicate status (if any):

--