MsalProvider constructor crashes unless you pass scopes

[asklar]

Graph-Controls/CommunityToolkit.Authentication.Msal/MsalProvider.cs

Line 63 in d985141

public MsalProvider(IPublicClientApplication client, string[] scopes = null, bool autoSignIn = true)

the default value for scopes is null, but we unconditionally run the Select(...) on it. Fix would be to either not provide a default argument for scopes (since it's arguably needed), or at least use scopes?.Select(...) so that we don't end up with a null deref.
note there's 2 ctors with this issue

[ghost]

Hello asklar, thank you for opening an issue with us!

I have automatically added a "needs triage" label to help get things started. Our team will analyze and investigate the issue, and escalate it to the relevant team if possible. Other community members may also look into the issue and provide feedback 🙌