272165 - Invalid mailto address silently dropped: href="mailto:([e-mail]" - should either fire warning or just take it over into addressing area for later correction or error handling when sending

Status: NEW

(Thunderbird :: Message Compose Window, defect)

Description

[Glen Gibb]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7.5) Gecko/20041107 Firefox/1.0
Build Identifier: version 0.9+ (20041128)

A mailto: link on a page contained an invalid e-mail address - it accidentally
contained a left-parenthesis. eg: mailto:([e-mail]

When clicking on this link from Firefox the Compose Message window opens,
however the To field is empty (it doesn't contain the e-mail address).
Additionally, it only contains one line to enter an e-mail address, rather than
multiple lines which you'd typically be presented with when composing a message.

Although this isn't a major problem it was a little confusing when the To field
appeared empty, especially when there was only one line for entering an address.

The URL listed above contains an example of the problem at the bottom of the
page under the "How To Apply" section.

Reproducible: Always
Steps to Reproduce:
1. Click on a mailto: link containing a parenthesis. eg: mailto:([e-mail]
Actual Results:  
The Compose New Message window that appears exhibits the problem of no To e-mail
address and only one line for entering an address

Expected Results:  
Either the invalid e-mail address should appear in the To: field, or a warning
message explaining the address is invalid should be displayed. Additional lines
for entering other e-mail addresses should be present. 

May be related to Bug 231032 which appears to be caused by problems with the
command line arguments passed to thunderbird.exe

Comment 1

[Mike Cowperthwaite]

Yes, this happens.  Reproduced with TB 1.0, Win2K.

Mozilla suite also does not fill in the To: field, but it does not exhibit the 
problem with the address fields seen in TB.

Comment 2

[Wayne Mery (:wsmwk)]

*** Bug 348825 has been marked as a duplicate of this bug. ***

Comment 3

[Mike Cowperthwaite]

I think that dupe is misplaced.

I retested this with current builds, and I'm not seeing the "only one address field" symptom, with 1.5.0.7 nor 2a1/3a1 nightlies.

Comment 4

[Wayne Mery (:wsmwk)]

similar to suite bug 193791

Comment 5

[Thomas D. (:thomas8)]

Attachment: Testcase1.html with invalid mailto link having href="([e-mail]"

This still fails on current Trunk, 25.0a1 (2013-07-24) / WinXP

As per comment 0, we should either fire an immediate warning about the invalid address, or just accept it into the addressing area for manual correction; if it's still invalid when message gets sent, I assume we already fire warnings.

Might be simple to fix after finding the right spot in code.

Comment 6

[Thomas D. (:thomas8)]

FTR: *Valid* cases having "(" in mailto, as seen in testcase1 (attachment 780843 [details]) seem to work ok, so it's really only about the invalid case.