Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Opens many new tabs for authentication on iOS - Edge/Safari #7298

Open
2 tasks
bensonjm opened this issue Sep 9, 2024 · 2 comments
Open
2 tasks

Opens many new tabs for authentication on iOS - Edge/Safari #7298

bensonjm opened this issue Sep 9, 2024 · 2 comments
Labels
bug-unconfirmed A reported bug that needs to be investigated and confirmed msal-browser Related to msal-browser package msal-react Related to @azure/msal-react Needs: Attention 👋 Awaiting response from the MSAL.js team public-client Issues regarding PublicClientApplications question Customer is asking for a clarification, use case or information.

Comments

@bensonjm
Copy link

bensonjm commented Sep 9, 2024

Core Library

MSAL.js (@azure/msal-browser)

Core Library Version

3.23.0

Wrapper Library

MSAL React (@azure/msal-react)

Wrapper Library Version

2.0.22

Public or Confidential Client?

Public

Description

I've configured msal-react for a react app but when I try to use it from my iPhone, whether it Edge or Safari, it keeps opens up multiple tabs.

I've created a sample repo.

Error Message

No response

MSAL Logs

No response

Network Trace (Preferrably Fiddler)

  • Sent
  • Pending

MSAL Configuration

auth: {
    clientId: <clientId>,
    authority: https://login.microsoftonline.com/<tenantId>,
    redirectUri: "/",
    postLogoutRedirectUri: "/",
  },
  cache: {
    cacheLocation: "localStorage",
    storeAuthStateInCookie: isIE || isEdge || isFirefox,
  },

Relevant Code Snippets

I've added the repo

Reproduction Steps

https://github.com/bensonjm/msal-react-test

Expected Behavior

It shouldn't be opening multiple tabs

Identity Provider

Entra ID (formerly Azure AD) / MSA

Browsers Affected (Select all that apply)

Edge, Safari

Regression

No response

Source

External (Customer)

@bensonjm bensonjm added bug-unconfirmed A reported bug that needs to be investigated and confirmed question Customer is asking for a clarification, use case or information. labels Sep 9, 2024
@github-actions github-actions bot added msal-browser Related to msal-browser package msal-react Related to @azure/msal-react public-client Issues regarding PublicClientApplications labels Sep 9, 2024
@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs: Attention 👋 Awaiting response from the MSAL.js team label Sep 9, 2024
@bensonjm bensonjm changed the title Too many new tabs for authentication on iOS - Edge/Safari Opens many new tabs for authentication on iOS - Edge/Safari Sep 9, 2024
@sameerag
Copy link
Member

sameerag commented Sep 9, 2024

@bensonjm This could be a property of your browser. Have you checked if the browser is set to open a new tab when you try to open a popup? Also can you confirm your sample is using acquireTokenPopup?

@microsoft-github-policy-service microsoft-github-policy-service bot added Needs: Author Feedback Awaiting response from issue author and removed Needs: Attention 👋 Awaiting response from the MSAL.js team labels Sep 9, 2024
@bensonjm
Copy link
Author

@sameerag , it's using InteractionType.Redirect

@microsoft-github-policy-service microsoft-github-policy-service bot added Needs: Attention 👋 Awaiting response from the MSAL.js team and removed Needs: Author Feedback Awaiting response from issue author labels Sep 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug-unconfirmed A reported bug that needs to be investigated and confirmed msal-browser Related to msal-browser package msal-react Related to @azure/msal-react Needs: Attention 👋 Awaiting response from the MSAL.js team public-client Issues regarding PublicClientApplications question Customer is asking for a clarification, use case or information.
Projects
None yet
Development

No branches or pull requests

2 participants