CommunityToolkit / CommunityToolkit/Graph-Controls

MsalProvider constructor crashes unless you pass scopes

Offen
#195 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Area: Providers
Vorherrschende Sprache
C#
Sterne
156
Forks
38
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

https://github.com/CommunityToolkit/Graph-Controls/blob/d985141a317c96ad571bd9dd8097e38969b649d6/CommunityToolkit.Authentication.Msal/MsalProvider.cs#L63

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

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.