User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7a) Gecko/20040216 Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7a) Gecko/20040216 If you specify in a message's 'To' field a mailing list from the address book (i.e. not a mail address of a server-side mailing list, but a list defined in your address book), in which some entries have empty e-mail address (yes, this can happen and it is logical), MailNews will ask the server to also send copies of the message to the address "<>" , possibly repeated as many times as there are entries with empty e-mail addresses (although I haven't checked this last possibility). The result is failure messages from your SMTP server telling you that sending the message to "<>" has failed. This should not happen - the entries with empty e-mail addresses should be silently ignored, not sent to the mail server. The bug is rather simple and if noone takes the time to fix it, I might have to do it myself - but please don't make me! It's a 1-minute-long chore for someone who knows the code... it'll take me a lot longer, not to mention the time spent chasing after reviewers and sreviewers. Reproducible: Always Steps to Reproduce:
Here is the response from my server, in case this helps somehow (some personal details altered): The original message was received at Sat, 20 Mar 2004 19:32:35 +0200 (IST) from [XXX.XXX.XXX.XXX] ----- The following addresses had permanent delivery errors ----- <> Reporting-MTA: dns; legolas.inter.net.il Arrival-Date: Sat, 20 Mar 2004 19:32:35 +0200 (IST) Final-Recipient: rfc822; @legolas.inter.net.il Action: failed Status: 5.1.1 Remote-MTA: X-Unix; mirapoint Diagnostic-Code: smtp; 550 5.1.1 User unknown Last-Attempt-Date: Sat, 20 Mar 2004 19:33:10 +0200 (IST) Subject: Interesting Idea From: Eyal Rozenberg <firstname.lastname@example.org> Date: Sat, 20 Mar 2004 19:33:59 +0200 To: Xxx Xxxx <email@example.com>, "Zzzz Zzzz (?)" <>, Yyy Yyyyy <firstname.lastname@example.org>
Created attachment 145885 [details] [diff] [review] naive fix I am guessing this has only remained unconfirmed since nobody has had the time to have a look at this bug. Anyway, here's a fix. I hope there aren't any other places that need modification except nsMsgCompose.cpp...
Eyal, if you specify the exact steps needed to reproduce this bug, and a link to a test mailing list (I don't want to spam an actual list), then I would be happy to verify this bug. Prog.
Steps to reproduce: 1. Open the address book 2. Create a new address book card with the name "bug 238098 recipient" and no e-mail address 3. Create a new address book card with your own name and e-mail (since you don't want to bother other people with your "this is a test" e-mails) 4. Create a new address book list called "238098 test list" 5. Drag the cards created in steps 2 and 3 into this new list 6. Close the address book 7. Open a new message composition window 8. In the "To:" field, type "238098 test list" 9. In the "Subject" field and in the body type "this is my 238098 test" 10. Send the message Expected Results: 1. You will receive a mail message from yourself named "this is my 238098 test" 2. You will receive a mail message from your SMTP server complaining about its inability to send your e-mail message to "<>" BTW, I need to check my own naive fix for what happens when _no_ e-mail addresses exist on the list.
Regarding my previous comment: I just checked, and the effect is that you get a "no recipients specified" error message. Pretty decent.
Confirmed with 1.7b and 1.6. In my case, no mail at all is being sent. I just get an error from the SMTP server saying "Bad address syntax" and the whole process stops. Prog.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Created attachment 146507 [details] [diff] [review] Better way? With attachment 145885 [details] [diff] [review] you can actually bcc an address card without an email address but with the "prefer text" option to force a plain text mail to what would normally be an html recipient, but I think that's being silly. This way just avoids processing address cards with no email address.
This was is fine. I didn't consider this question consciously, I just saw there were some side-effects to having someone on a mailing list and I followed the 'principle of minimum change'. I'm not even sure if it is silly, for example if the person on the list is someone you are printing the e-mail for and he/she prefers doesn't want you to print fancy layout & images maybe it does make sense to force the mail into plain text. That's kind of stretching it, but if the user doesn't want this to happen, why we would he/she select 'prefers text' for someone without an e-mail? Now this begs the question of whether the 'prefers text' GUI should not be grayed-out when an e-mail address is not defined.
Well, we currently allow completely blank address cards...
It's not like I'm arguing... I think we should add a comment into the code to explain the dilemma, and then just make the choice, it can be a coin toss AFAIAC, it really doesn't matter very much.
Created attachment 150362 [details] [diff] [review] 'better way' patch against 1.8a That line of code changed. I think this change will still work as planned.
Attachment #145885 - Attachment is obsolete: true
Created attachment 150363 [details] [diff] [review] 'better way' patch against 1.8 whoops. I wish you could remove patches rather than invalidate them.
Attachment #150362 - Attachment is obsolete: true
Attachment #150363 - Attachment is obsolete: true
> Anyway, that's a different bug. And now it really is a different bug: http://bugzilla.mozilla.org/show_bug.cgi?id=246089
sorry for the spam. making bugzilla reflect reality as I'm not working on these bugs. filter on FOOBARCHEESE to remove these in bulk.
Assignee: sspitzer → nobody
Filter on "Nobody_NScomTLD_20080620"
QA Contact: esther → composition
still see this (while researching bug 147738)
OS: Windows XP → All
Hardware: PC → All
Version: Trunk → unspecified
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.