c# - Is it okay to attach async event handler to System.Timers.Timer?

Question

I have already read the SO posts here and article here. I have a timer event that fires every once in a while and I want to do some asynchronous processing inside the handler, so something along the lines of:

Timer timer = new Timer();
timer.Interval = 1000;
timer.Elapsed += timer_Elapsed; // Please ignore this line. But some answers already given based on this line so I will leave it as it is.
timer.Elapsed += async (sender, arguments) => await timer_Elapsed(sender, arguments);

private async Task timer_Elapsed(object sender, ElapsedEventArgs e)
{
    await Task.Delay(10);
}

The code above is compiling and working.

But I am not sure why the code is compiling. The ElapsedEventHandler expected signature is

void timer_Elapsed(object sender, ElapsedEventArgs e)

However my method returns Task instead of void since async void is not recommended. but that does not match with ElapsedEventHandler signature and yet it's still compiling and working?

Is it okay to call an async method on Timer.Elapsed? The code will be executed inside a Windows service.

Update 1

async void is "not recommended", with one very important exception: event handlers.

Does it matter if it's an asynchronous event handler or synchronous event hander?

An MSDN article here says:

Void-returning async methods have a specific purpose: to make asynchronous event handlers possible.

Timer.Elapsed is I think a synchronous event handler; can I still attach async void to it?

Answer 1

async void is "not recommended", with one very important exception: event handlers.

Your code compiles fine (well, the second event subscription…the first would generate a compile-time error, assuming the same timer_Elapsed() method in both statements), because the compiler can infer the delegate's return type should be void. The async anonymous method could also return Task, but in this case that would be the wrong method signature, so you get void instead.

It would also be fine to just declare your event handle as async void:

   private async void timer_Elapsed(object sender, ElapsedEventArgs e)
   {
      await Task.Delay(10);
   }

Used like:

timer.Elapsed += timer_Elapsed;

Returning void for async methods is not ideal, but in the case of an event handler, there is no code that is going to use the Task anyway (unless the event is specifically implemented to understand async methods, as in Asynchronous events in C#). There's no reason to bend over backwards to comply with what would otherwise be the correct coding practice, if you get zero benefit from doing so.

See also Should I avoid 'async void' event handlers?

Addendum:

From your edit to the question:

Timer.Elapsed is I think synchronous event handler can I still attach async void to it?

It's not the event that is asynchronous or synchronous, but the handler itself. And that's determined entirely by whether you use async and await for the handler method. You may, as described in your question and my answer, use an async void handler method with the Elapsed event, just as you may with any other event (assuming the event signature requires void as the handler return type, which is of course the standard for conventional .NET events).