search

TehOPanas / pe

0 stars 0 forks source link

Email validation constraint too loose #4

Open TehOPanas opened 12 months ago

TehOPanas commented 12 months ago

After trying out the product, I discovered that the constraint for the email field is a bit loose given that emails like a@bc are allowed to be entered even though this is clearly not a valid email.

image.png

I suggest the development team can maybe reference some standards from RFC or OWASP

nus-se-script commented 11 months ago

Team's Response

We agree that this is a valid concern. However, when our team was discussing restrictions for email, we wanted to implement a restriction that was flexible enough to prevent overzealous input validation.

Validation of every possible email is also not a trivial task and hence it is not as important as the current features as explained in nus-cs2103-AY2324S1/forum#336. Hence, we have categorised this bug report as NotInScope.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]