ruishanteo / pe

0 stars 0 forks source link

Email format error message #5

Open ruishanteo opened 9 months ago

ruishanteo commented 9 months ago

Testing with the command: add n/Joh p/98765432 e/johnd@bb.b-b r/Doctor

This is the email format error message:

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.

Point 2 states that the domain name is made of domain labels separated by periods, which means bb.b-b are two domain labels of bb and b-b. By this line, the domain name should be accepted. But the third bullet point states that each domain label is separated only by hyphens, making the domain name not accepted.

nus-pe-bot commented 9 months ago

Team's Response

Hi tester, thank you for pointing this out. You are absolutely right that your input should have been accepted by the application. We believe that since having hyphens in emails is not a very common occurrence in domain labels, this problem will unlikely affect normal operations of the product, and will only appear in very rare situations, causing a minor inconvenience.

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]