MSAL issue with Copilot Studio SDK

Offen
#527 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Anfängerfreundlichkeit
55/100
Issue-Typ
Bug
Klarheit
Größtenteils klar
Aktivitätsstatus
Ruhig
Tech-Stack
react, typescript

Rechercherichtung

Beginnen Sie damit, die MSAL-Initialisierung und die loginPopup-Aufrufe im webpart zu lokalisieren, und untersuchen Sie anschließend, wie sich das React-Rendering auf die MSAL-Instanz auswirkt. Überprüfen Sie, dass dieselbe Instanz wiederverwendet wird und handleRedirectPromise() aufgerufen wird. Als abgeschlossen gilt die Aufgabe, wenn die Authentifizierung das Token an das webpart zurückgibt und das weitergeleitete SharePoint-Popup automatisch geschlossen wird.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

Issue Description:
We developed a webpart to access copilot agent data. When configuring the webpart on a SharePoint site, it opens a popup for authentication. After authentication succeeds, the token is not being returned to the webpart.

Microsoft Support team findings below,

The Auth pop-up opening the Sharepoint site is a direct symptom of new MSAL instance created on every call combined with missing handleRedirectPromise().
Here's what's happening step by step:

  1. loginPopup() opens a popup to login.microsoftonline.com.
  2. User picks their account → Azure AD authenticates successfully
  3. Azure AD redirects the popup to the redirect Uri, which is settings. redirectUri || window.location.origin — i.e., the SharePoint site URL
  4. The popup now loads the full SharePoint site instead of closing
    The popup should close automatically at this point. MSAL's popup flow works by having the parent window monitor the popup's URL via polling. When the popup redirects back to the same origin with the auth code in the URL hash, the parent MSAL instance reads the code, processes it, and closes the popup.
    But because a new MSAL instance is created on every call and handleRedirectPromise() is never called:
    • The MSAL instance that opened the popup may have been garbage collected (React re-render)
    • The polling interval that monitors the popup URL is gone
    • Nobody reads the auth code from the popup's URL
    • Nobody closes the popup
    • The popup just sits there showing the SharePoint site it was redirected to
    So the SharePoint site loading in the popup is the redirect working correctly from Azure AD's side — but MSAL on the parent side failing to intercept and close it.

To fix this, the MSAL instance must be a singleton, and handleRedirectPromise() must be called.

Vorherrschende Sprache
TypeScript
Sterne
807
Forks
493
Ø Merge
17 Std. 8 Min.
Gemergte PRs (30 T.)
3

Entwicklungsumgebung

Dieses Projekt bietet weder Dev-Container noch Dockerfile noch Beitragsleitfaden – die Einrichtung liegt bei Ihnen. Beginnen Sie mit der README; die allgemeinen Schritte stehen in unserem Leitfaden für den ersten Beitrag.

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus microsoft/CopilotStudioSamples

Alle Issues in microsoft/CopilotStudioSamples

Ähnliche Issues

Weitere Issues zu TypeScript

Neue Issues direkt in Ihr Postfach

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