jamz903 / pe

0 stars 0 forks source link

Slashes should be allowed in email #1

Open jamz903 opened 11 months ago

jamz903 commented 11 months ago

How to replicate: add n/john Wick p/12345678 e/test/test@gmail.com c/Google j/Software Engineert/NUS t/metInHackathon

test/test@gmail.com is a valid email (since slashes are a printable character, and allowed) and should be accepted (valid according to RFC2822)

nus-se-script commented 11 months ago

Team's Response

The team recognizes this as a legitimate feature flaw.

However, slashes are discouraged in email addresses for the local part, and are a rare use case.

For example, https://www.jochentopf.com/email/chars.html "The forward slash is often used in email addresses used by X.400 gateways. Example: 'PN=Joe/OU=X400/@gateway.com'. Because the slash is used as a directory separator in UNIX and other operating systems, it is probably not a good idea to use it in local parts. If you ever want to save the mail in a mailbox named after the local part, you will have problems."

As this appears in only rare situations and can be worked around by using the note command in the rare event that it does occur, we downgrade the severity of the bug to severity.Low image.png

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]