Mail doesn't work with multiple to addresses

Question

Possible Duplicate:
Unable to send an email to multiple addresses/recipients using C#

I have used below code to send mail in script task

 string MailFromName = "Admin";
            System.Net.Mail.SmtpClient mailobj = new System.Net.Mail.SmtpClient();
            System.Net.Mail.MailAddress MailFrom = new System.Net.Mail.MailAddress(MailFromEmail, MailFromName);
            System.Net.Mail.MailAddress MailTo = new System.Net.Mail.MailAddress(MailToEmail, MailToEmail);
            System.Net.Mail.MailMessage mailmsg = new System.Net.Mail.MailMessage(MailFrom, MailTo);
            mailmsg.IsBodyHtml = true;
            mailmsg.Subject = strMessageSubject;
            mailmsg.Body = strMessageBody;
            mailobj.Host = strSMTPServerName;
            mailobj.Send(mailmsg);

It is working fine when I am using MailToEmail as "myaddress@myMail.com" i.e. for one email address

but this doesn't send any mail(also it dosen't fail) when I have multiple adress in to list

ex: "MyAdress@MyMail.com; MySecondAddress@MyMail.com"

How to resolve this?

EDIT New Code

 string MailFromName = "Admin";
            System.Net.Mail.SmtpClient mailobj = new System.Net.Mail.SmtpClient();
            System.Net.Mail.MailAddress MailFrom = new System.Net.Mail.MailAddress(MailFromEmail, MailFromName);
            System.Net.Mail.MailAddress MailTo = new System.Net.Mail.MailAddress(MailToEmail, MailToEmail);
            System.Net.Mail.MailMessage mailmsg = new System.Net.Mail.MailMessage(MailFrom, MailTo);
            mailmsg.IsBodyHtml = true;
            mailmsg.Subject = strMessageSubject;
            mailmsg.Body = strMessageBody;
            foreach (string str in multipleToMsg)
            {
                mailmsg.To.Add(str);
            }


            mailobj.Host = strSMTPServerName;
            mailobj.Send(mailmsg);

_How_ do you add multiple addresses? That seems to be the key code and it's missing. — H H, Sep 11 '12 at 13:50
MailToEmail is a string variable which has multiple address seperated with **;** — Zerotoinfinity, Sep 11 '12 at 13:50
`using System.Net.Mail` would have spared you a lot of clutter. — Adam, Sep 11 '12 at 13:51

score 4 · Accepted Answer · answered Sep 11 '12 at 13:51

4

You've not shown how exactly you are adding the recipients. However to add multiple recipients you add to the "To" collection:

MailMessage message = new MailMessage();
message.To.Add("sillyjoe@stackoverflow.com");

"To" is a collection of MailAddresses. Make sure you are adding it to that collection and not attempting to concatenate email addresses all into one MailAddress object.

answered Sep 11 '12 at 13:51

Arran

24,648
6
68
78

I believe I am doing same here System.Net.Mail.MailMessage mailmsg = new System.Net.Mail.MailMessage(MailFrom, MailTo); – Zerotoinfinity Sep 11 '12 at 13:54
Not quite, the constructor allows you to only pass in one recipient. If you want to give it multiple recipients you cannot use the constructor, you must add to the collection afterwards. – Arran Sep 11 '12 at 13:57
Then How can I modify my Current code :-( I am failing to do that. When I am creating message object I am not able to add from address – Zerotoinfinity Sep 11 '12 at 13:58
Please check my edit part, I am still not able to get the mail – Zerotoinfinity Sep 11 '12 at 14:04
From can be specified by doing mailmsg.From. Show us the string of emails. – Arran Sep 11 '12 at 14:05
**Cannot implicitly convert type 'string' to 'System.Net.Mail.MailAddress'** – Zerotoinfinity Sep 11 '12 at 14:11
mail1@company.com; mail2.company.com; – Zerotoinfinity Sep 11 '12 at 14:11
Oops ! my mistake ! now it is fine thanks :-) +1 – Zerotoinfinity Sep 11 '12 at 14:13

Mauro · Answer 2 · 2012-09-11T14:07:33.643

2

Accoring to MSDN: MailMessage Class the "To" property is a collection of MailAddresses

so you just need to do something like

mailmsg.To.Add(new System.Net.Mail.MailAddress(MailToEmail, MailToEmail)); 
mailmsg.To.Add(new System.Net.Mail.MailAddress(MailToEmail2, MailToEmail2))

or in a foreach loop

//get email addresses into a collection called emailAdds
foreach (var emailAdd in emailAdds)
{
    mailmsg.To.Add(new System.Net.Mail.MailAddress(emailAdd, emailAdd )); 
}

edited Sep 11 '12 at 14:07

answered Sep 11 '12 at 13:52

Mauro

4,531
3
30
56

As my Mail adress are coming from the database I may not know how many to address are there. :-( – Zerotoinfinity Sep 11 '12 at 13:56
yeah you can use a for/foreach loop there. I just used it to show you how to add multiple items – Mauro Sep 11 '12 at 14:06

score 0 · Answer 3 · answered Sep 11 '12 at 13:51

0

To specify multiple addresses you need to use the To property which is a MailAddressCollection

message.To.Add("one@example.com, one@example.com"));
message.To.Add("two@example.com, two@example.com"));

answered Sep 11 '12 at 13:51

Michael G

6,695
2
41
59

Mail doesn't work with multiple to addresses

3 Answers3