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

Strange behavior on MAUI WebAuthenticator for android #22296

Closed
noorez opened this issue May 8, 2024 · 3 comments
Closed

Strange behavior on MAUI WebAuthenticator for android #22296

noorez opened this issue May 8, 2024 · 3 comments
Labels
area-essentials Essentials: Device, Display, Connectivity, Secure Storage, Sensors, App Info platform/android 🤖 s/needs-repro Attach a solution or code which reproduces the issue s/no-recent-activity Issue has had no recent activity t/bug Something isn't working

Comments

@noorez
Copy link

noorez commented May 8, 2024

Description

I am trying to use the WebAuthenticator on an Android platform to authenticate via OAuth authorization code flow with SAML/SSO.

  1. we initiate to the authorize endpoint like: the /oauth2/authorizePoint
  2. redirect to SAML/SSO provider to authenticate
  3. SAML Assertion returns back to app to /oauth2/authorizePoint
  4. /oauth2/authorize has a redirect back to the application scheme myapp://callback

the app appears to receive the response via the custom scheme, however, while having the remote chrome debugger attached, I can see the chrome tab webview strangely attempt to 'repeat' the authentication from start.
--> call /oauth2/authorize --> notice we are already authenticated --> call back to app myapp://callback.

We are definitely not invoking the authentication twice.

The double callback is causing an error since one of them has already tried to complete the PKCE workflow and the second attempt fails as the code has already been redeemed.

Steps to Reproduce

No response

Link to public reproduction project repository

No response

Version with bug

8.0.21 SR4.1

Is this a regression from previous behavior?

Not sure, did not test other versions

Last version that worked well

Unknown/Other

Affected platforms

Android

Affected platform versions

No response

Did you find any workaround?

No

Relevant log output

No response

@noorez noorez added the t/bug Something isn't working label May 8, 2024
Copy link
Contributor

github-actions bot commented May 8, 2024

Hi I'm an AI powered bot that finds similar issues based off the issue title.

Please view the issues below to see if they solve your problem, and if the issue describes your problem please consider closing this one and thumbs upping the other issue to help us prioritize it. Thank you!

Open similar issues:

Closed similar issues:

Note: You can give me feedback by thumbs upping or thumbs downing this comment.

@PureWeen PureWeen added platform/android 🤖 s/needs-repro Attach a solution or code which reproduces the issue labels May 8, 2024
Copy link
Contributor

Hi @noorez. We have added the "s/needs-repro" label to this issue, which indicates that we require steps and sample code to reproduce the issue before we can take further action. Please try to create a minimal sample project/solution or code samples which reproduce the issue, ideally as a GitHub repo that we can clone. See more details about creating repros here: https://github.com/dotnet/maui/blob/main/.github/repro.md

This issue will be closed automatically in 7 days if we do not hear back from you by then - please feel free to re-open it if you come back to this issue after that time.

@Eilon Eilon added the area-essentials Essentials: Device, Display, Connectivity, Secure Storage, Sensors, App Info label May 8, 2024
@dotnet-policy-service dotnet-policy-service bot added the s/no-recent-activity Issue has had no recent activity label May 13, 2024
Copy link
Contributor

This issue has been automatically marked as stale because it has been marked as requiring author feedback to reproduce the issue but has not had any activity for 4 days. It will be closed if no further activity occurs within 3 days of this comment. If it is closed, feel free to comment when you are able to provide the additional information and we will re-investigate.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-essentials Essentials: Device, Display, Connectivity, Secure Storage, Sensors, App Info platform/android 🤖 s/needs-repro Attach a solution or code which reproduces the issue s/no-recent-activity Issue has had no recent activity t/bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants