Kick off async method in constructor in c#

Question

I'm wondering is it safe to call async method in a constructor in the following way:

Let's say we have an async method Refresh that is fetching data from the internet. We are also using Reactive Extensions to notify everyone that is interested that new data was fetched.

I'm wondering is it safe to call Refresh first time in a class constructor? Can I use such construction?

Task.Run(Refresh);

or

Refresh().ConfigureAwait(false)

I'm not really interested here if the method has finished or not, since I will get notified through Reactive Extensions when data is fetched.

Is it ok to do something like this?

public class MyClass
{
    BehvaiorSubject<Data> _dataObservable = new BehvaiorSubject(Data.Default);
    IObservable DataObservable => _dataObservable;

    public MyClass()
    {
        Refresh().ConfigureAwait(false);     
    }

    public async Task Refresh()
    {
        try
        {
           var data = await FetchDataFromNetwork();

           _dataObservable.OnNext(data);

        }
        catch (VariousExceptions e)
        {
           //do some appropriate stuff
        }
        catch(Exception)
        {
           //do some appropriate stuff
        }
    }
}

Answer 1

Though people are against the idea, we have similar things in our project :)

The thing is you have to properly handle any exceptions thrown from that Task in case they go unobserved. Also you might need to expose the task via either a method or a property, just so that it is possible to await (when necessary) the async part is finished.

class MyClass
{
    public MyClass()
    {
        InitTask = Task.Delay(3000);

        // Handle task exception.
        InitTask.ContinueWith(task => task.Exception, TaskContinuationOptions.OnlyOnFaulted);
    }

    public Task InitTask { get; }
}