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

[Bug] SystemWebViewOptions.BrowserRedirectSuccess not working on iOS #4844

Closed
weases opened this issue Jul 17, 2024 · 3 comments
Closed

[Bug] SystemWebViewOptions.BrowserRedirectSuccess not working on iOS #4844

weases opened this issue Jul 17, 2024 · 3 comments
Labels
answered needs attention Delete label after triage public-client question untriaged Do not delete. Needed for Automation

Comments

@weases
Copy link

weases commented Jul 17, 2024

Library version used

4.60.4

.NET version

.NET Standard 2.1

Scenario

PublicClient - mobile app

Is this a new or an existing app?

The app is in production, I haven't upgraded MSAL, but started seeing this issue

Issue description and reproduction steps

Trying to implement a workaround for the bug of iOS just showing the save password dialog briefly - see here for more information.

As stated in this comment one way to solve this would be to extend the flow and show a login success page.

Unfortunately this does not seem to work properly - the new page is never shown.

SystemWebViewOptions options = new()
{
	//HtmlMessageSuccess = "<p> You have successfully signed in. </p>"
	BrowserRedirectSuccess = new Uri("https://www.microsoft.com")
};

AuthenticationResult result = await authBuilder
.WithSystemWebViewOptions(options)
.ExecuteAsync().ConfigureAwait(true);

I am using this in a Xamarin.Forms application using iOS 17.5.1.

Any idea?

Relevant code snippets

No response

Expected behavior

No response

Identity provider

Azure B2C Basic Policy

Regression

No response

Solution and workarounds

No response

@weases weases added needs attention Delete label after triage untriaged Do not delete. Needed for Automation labels Jul 17, 2024
@bgavrilMS
Copy link
Member

This is by design. On ios the sdk is able to fully control the browser. There is no need to display a success message, the app will just close the browser.

On windows, mac and Linux, the sdk cannot close the browser. So we display a message for the user to do it.

@weases
Copy link
Author

weases commented Jul 21, 2024

I see
Could you consider this issue as a feature request then please?
As mentioned above I would need this as a workaround for this issue. I guess there will be no fix for this in the foreseeable future...

@iulico-1
Copy link
Contributor

Closing the issue as by design.

@iulico-1 iulico-1 closed this as not planned Won't fix, can't repro, duplicate, stale Jul 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
answered needs attention Delete label after triage public-client question untriaged Do not delete. Needed for Automation
Projects
None yet
Development

No branches or pull requests

4 participants