System.Exception when I try to send email

Question

I am trying to send an email with c#. Everything seems to be ok no errors but when the programms tries to send the mail I get an exception : "System.Exception - Exception was not treated"

here is the code:

public void sendMail()
    {
        SmtpClient client = new SmtpClient();
        client.Port = 587;
        client.Host = "smtp.gmail.com";
        client.EnableSsl = true;
        client.Timeout = 100;
        client.DeliveryMethod = SmtpDeliveryMethod.Network;
        client.UseDefaultCredentials = false;
        client.Credentials = new System.Net.NetworkCredential("[email protected]", "mypassword");

        MailMessage mm = new MailMessage("[email protected]", "receiveremailaddress", "test", "test");
        mm.BodyEncoding = UTF8Encoding.UTF8;
        mm.DeliveryNotificationOptions = DeliveryNotificationOptions.OnFailure;


         client.Send(mm); // HERE I GET THE EXCEPTION !!!!!





    }

Bad practice. Any code above that throw will have no knowledge of what actually happened. If you don't mean to deal with the exception properly, then don't catch it. — DonBoitnott
– DonBoitnott, Commented May 30, 2014 at 13:53

haddow64 · Accepted Answer · 2014-05-30 14:19:01Z

2

The issue is your timeout is far too short try:

client.Timeout = 5000;

The default value is 100,000 (100 seconds), but smtp.gmail.com is fast enough that if you want to use 5000 it should be fine.

Also as per the above comment to your post; handle your exception (this is a very basic way of doing so).

try
{
...
}
catch(Exception ex)
{
Console.Writeline(ex);
}

edited May 30, 2014 at 14:19

answered May 30, 2014 at 14:13

haddow64

6741 gold badge7 silver badges25 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

CMS Over a year ago

:) thank you! yes the timeout was to short ! I changed it to 5000 and now everything wokrs perfectly

Kirk Woll Over a year ago

@haddow64, your comment about handling the exception by printing it to the console is bad advice. You should always let the exception bubble up unless you intend to actually handle the error or log it in a top-level exception handler.

Collectives™ on Stack Overflow

System.Exception when I try to send email

1 Answer 1

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related