search

bmanara / pe

0 stars 0 forks source link

Valid email given, but app rejects it and throws an error #2

Open bmanara opened 2 weeks ago

bmanara commented 2 weeks ago

Bug Details

When given a valid email such as a__a@example.com (which follows the rules given by the user guide and error message), the app rejects it and throws an error, stating 

Emails should be of the format local-part@domain and adhere to the following constraints:
1. The local-part should only contain alphanumeric characters and these special characters, excluding the parentheses, (+_.-). The local-part may not start or end with any special characters.
2. This is followed by a '@' and then a domain name. The domain name is made up of domain labels separated by periods.
The domain name must:
    - end with a domain label at least 2 characters long
    - have each domain label start and end with alphanumeric characters
    - have each domain label consist of alphanumeric characters, separated only by hyphens, if any.

Steps to reproduce:

  1. Launch app
  2. Type add n/Waluigi p/92345678 e/a__a@example.com a/Blk123 Bukit Gombak
  3. Observe error message thrown.

Note: This also occurs for any other command that deals with emails.

Screenshots

image.png

This bug was inherited by AB3.

soc-se-bot commented 1 week ago

Team's Response

image.png As this email is not a valid email in real world, we deemed this as very unlikely to happen. However, we do acknowledge we need a better email validation in UG and error message.

Items for the Tester to Verify

:question: Issue severity

Team chose [severity.Low] Originally [severity.Medium]

Reason for disagreement: [replace this with your explanation]