Stop using async void in C#! Do this instead.

…re though. 2\. You are calling an async function (sendEmail) from a synchronous function (the event handler). You should always handle the returned Task from an async function or weird things happen with Exceptions you might not expect (though they make sense with how async is implemented). This video covers in a bit more detail why this is bad: link to this video When you can do is declare the event handler async void. (Yes that video says not to do that; it is ok in scenarios where it's unavoidable if you keep in mind the consequences) so you can then await on sendEmail. You can't declare event handlers async Task since the standard signature has a return value of void. So for compatibility reason…

…ored in the returned task. The exception is rethrown when the task is awaited. Make sure that any async method that can produce an exception has a return type of Task or Task<TResult> and that calls to the method are awaited. It kill the software if an exception is thrown in a method that is Task<void>. If you prefer a video that talk about it: link to this video…