search

sugiyem / pe

0 stars 0 forks source link

Email validation bug #5

Open sugiyem opened 1 year ago

sugiyem commented 1 year ago

The system allow invalid email, such as roy@com to be inserted.

image.png

soc-pe-bot commented 1 year ago

Team's Response

This is a valid email according to our input validation. Attached below is an excerpt from our User Guide.

image.png

This is intentional as we do not want to have too strict input validation for users. If we were to increase the restriction on the type of emails provided, it will also lead to other bugs since some emails might not be accepted. It is hard to find a middle ground of emails to accept. As such we decided to follow this scheme.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]