860144 - Disallow sending messages with a single comma as reply-to header (check for correct syntax)

Status: NEW

(MailNews Core :: Composition, defect)

Description

[rsx11m]

Steps to reproduce:

1. Open a new message
2. Set "To:" to yourself
3. Select "Reply-to:" in the next line
4. Enter a single comma (,)
5. Click "Send"
6. Watch the message arrive with "Reply-to: ," header
7. This triggers bug 860103 when reading the message

Per RFC 5322, "Reply-to:" has to specify an address-list, which in turn is
   address-list    =   (address *("," address)) / obs-addr-list
   address         =   mailbox / group

Specifically, neither of those should be empty, thus "," doesn't appear to be a valid address-list. Even if obs-addr-list covers that part (I'm not sure if I read the syntax there correctly), using it should be discouraged. Thus, entering a single comma in the "Reply-to:" field should trigger a warning similar to a wrong address entered into the "From:" or "To:" fields.

Comment 1

[:aceman]

we could add some stricter checking to MsgComposeCommands.js::CheckValidEmailAddress , however we must get the valid format right, so that we do not disable valid stuff.

Or in mailnews/compose/src/nsMsgCompUtils.cpp::mime_sanity_check_fields but that one currently has less strict checking than MsgComposeCommands.js::CheckValidEmailAddress .