Discuss CancelationToken on FormsApplicationListener

[lucas-zimerman]

To not lose the comment from Matt

#108 (review)

Yikes. I see this was in the old code and you are just refactoring, but we really shouldn't completely lose this task. It should be saved somewhere. We should also register it for cancellation so it can be shut down gracefully. I'm OK if this is done in a future PR, but wanted to point it out.

On

sentry-xamarin/Src/Sentry.Xamarin.Forms/Internals/FormsApplicationListener.cs

Lines 30 to 45 in 6c15473

	public void Invoke() =>
	//Don't lock the main Thread while waiting for the current application to be created.
	Task.Run(async () =>
	{
	var application = await GetCurrentApplication().ConfigureAwait(false);
	if (application is null)
	{
	_options.DiagnosticLogger?.Log(SentryLevel.Warning, "Sentry.Xamarin.Forms timeout for tracking Application.Current. Navigation tracking is going to be disabled");
	return;
	}
	foreach(var hook in _listeners)
	{
	hook.Invoke(application);
	}
	});

[bruno-garcia]

Task.Run() should have the value assigned somewhere and the result/exception inspected, or at least a whole try/catch in the body handling any error (we should do this anyway). Also we could go with ContinuesWith(...)