search

yiiilonggg / pe

0 stars 0 forks source link

[Email Format] Single Domain Emails #4

Open yiiilonggg opened 4 days ago

yiiilonggg commented 4 days ago

Background

Domain portion does not specify the min/max number of domain labels.

Test Case

add n/testemaildomainsingle p/92234567 e/email@security

Expected Behaviour

Reject email adding, single domain emails can cause security issues.

Encountered Behaviour

Successful addition

Remarks, if any

Very low, since unlikely in real life, unless malicious user. Feature flaw since it should be guarded against. Understand if team decides its NotInScope.

nus-pe-script commented 15 hours ago

Team's Response

Thank you! We decided that it is a valid flaw, but we wish to provide the user the flexibility to add emails as they wish. Users are expected to clarify such instances of emails and whilst it should be guarded against, we ultimately give the user a choice on whether to enter such emails under certain contacts.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]