Make Umbraco.Cms.Infrastructure.HostedServices.RecurringHostedServiceBase methods virtual

[PeterKvayt]

I faced with the following issue: RecurringHostedServiceBase class has has abstract method PerformExecuteAsync. This method called as callback in Timer.

Let's imagine that we set Thread.CurrentThread.CurrentUICulture in Main in Program.cs file as "en-GB", but by default Thread.CurrentThread.CurrentUICulture has "en-US" (it takes culture from machine region settings, but it doe not matter).
During Timer creating Thread.CurrentThread.CurrentUICulture has "en-GB" culture, but when PerformExecuteAsync start executing, new thread is created and this new thread has default culture = "en-US".
So if i want to have "en-GB" during PerformExecuteAsync, I need in my background task implementation (which inherited from RecurringHostedServiceBase) explicitly one more time setup Thread.CurrentThread.CurrentUICulture as "en-GB".
I think it is not good. It could be easily solved if methods were virtual.

What I am suggesting: it is not to bad to make virtual StartAsync, StopAsync, ExecuteAsync.

As I can see in release-13.3.2 methods StartAsync, StopAsync are already virtual. I am using umbraco 10.8.1, and as I can see for now umbraco 10 latest does not has this methods as virtual. Also it is important to make ExecuteAsync method as virtual too.

[PeterKvayt]

Here is pull request