How to get status of sent email using ASP.NET MVC

Question

0.00/5 (No votes)

See more:

I am sending email its working fine now i want to save status of each email which could be success or fail

how could i do this.?

What I have tried:

public async static Task SendEmailAsync(string email, string subject, string message)
{
try
{
var _email = "xyz@gmail.com";
var _epass = "pwd";
var _displayname = "test mail";
MailMessage mymessage = new MailMessage();
mymessage.To.Add(email);
// mymessage.ReplyTo=email;
mymessage.From = new MailAddress(_email, _displayname);
mymessage.Subject = subject;
mymessage.Body = message;
mymessage.DeliveryNotificationOptions = DeliveryNotificationOptions.OnSuccess;

mymessage.IsBodyHtml = true;
using (SmtpClient smtp = new SmtpClient()) {
smtp.EnableSsl = true;
smtp.Host = "smtp.gmail.com";
smtp.Port = 587;
smtp.UseDefaultCredentials = false;
smtp.Credentials = new NetworkCredential(_email,_epass);
smtp.DeliveryMethod = SmtpDeliveryMethod.Network;
smtp.SendCompleted += (s, e) => { smtp.Dispose(); };
await smtp.SendMailAsync(mymessage);

}
}
catch (SmtpFailedRecipientsException ex){
throw ex;
}
}

Posted 20-Oct-19 3:41am

Amar P Sutar

Updated 20-Oct-19 7:32am

Add a Solution

Comments

F-ES Sitecore 20-Oct-19 10:24am

How do you define what is success and what is failure?

Richard Deeming 22-Oct-19 12:15pm

catch (SmtpFailedRecipientsException ex){
    throw ex;
}

Don't do that. You've just destroyed the stack trace of the exception.

If you really need to re-throw an exception, just use throw; instead of throw ex;.

But in this case, since you're not doing anything with the exception you've caught, just remove the try..catch block altogether.

2 solutions

Solution 2

Please update the catch block with the code given below,

catch (SmtpFailedRecipientsException ex)
       {
           for (int i = 0; i < ex.InnerExceptions.Length; i++)
           {
               SmtpStatusCode status = ex.InnerExceptions[i].StatusCode;
               if (status == SmtpStatusCode.MailboxBusy ||
                   status == SmtpStatusCode.MailboxUnavailable)
               {
                   Console.WriteLine("Delivery failed - retrying in 5 seconds.");
                   System.Threading.Thread.Sleep(5000);
                   client.Send(message);
               }
               else
               {
                   Console.WriteLine("Failed to deliver message to {0}",
                       ex.InnerExceptions[i].FailedRecipient);
               }
           }
       }

Posted 20-Oct-19 7:32am

Hussain Mustafa1

Comments

Richard Deeming 22-Oct-19 12:13pm

Thread.Sleep is not a good choice within an async method. You should use await Task.Delay(...); instead.

Also, those Console.WriteLine calls won't do much good within an ASP.NET application.

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Dave Kreskowiak · Accepted Answer · 2019-10-20T07:02:00

That depends on what you mean by "checking the status of the email".

You can get a status on the email being posted to your local email server to so it can be sent. You should really be using the synchronous version of SendMail since it appears you're trying to do this in a Task. That Task can make the operation Async separate from the call to SendMailAsync, and then you would be able to get the status back without using the Event handler.

After the message is successfully posted to the mail server specified in smtp.Host, it's impossible for you to know if the intended receiver actually got the email.