-
Notifications
You must be signed in to change notification settings - Fork 112
WebView errors treated as cancel #726
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
Comments
Yup, you're right, we shouldn't treat this as a cancel error. We'll put this in backlog, and fix it with prioritization. |
What is the progress on this please? |
Hey Wei, Any update on this ? Thanks, |
Also keen to know what the latest is on this... |
Sorry we haven't got time to work on it. We'll update once this is prioritized. |
@weijjia - any updates on this? I am getting similar error on calling Following is the error:
|
I'll leave the prioritization to @nazukj |
Does anybody have a workaround for this? I am sure I am not the first one trying to do this. All I am trying to do is have the user enter UserName (email ID in my case) and Password which I want to pass onto Azure AD using ADAL to get them authenticated and return a result (a token or whatever that I can use in further interaction). Or, is ADAL not for this? If not what other library can I use to work with my AzureAD? |
@paragw09ckp ADAL is definitely for this purpose. Can you get us the verbose ADAL logs? There should be something going wrong after you enters creds in the webview, verbose logs will help us know more. |
@weijjia, @nazukj - thanks for the quick reply, appreciated. For verbose logs, not sure how to get more detailed logs. Will try to do it now. BTW, I tried the sample app given on this link and it seems to work fine. But here also, it opens up the Microsoft Azure AD login page which I what I am trying to avoid. Any idea on:
|
ADAL should work in different versions, the only way we can answer why it's not working is that we get diagnostic logs. https://github.com/AzureAD/azure-activedirectory-library-for-android#logs |
I may have some insights on this. The
It is called whenever the web view attempts to navigate to a URL starting with the redirect URL. The actual string looks like this, with minor changes to protect the guilty:
This clearly describes a legitimate error situation. That method, however, considers the mere presence of an "error" parameter in the URL to signal a cancellation, so the Passing the |
Same goes for: and |
I submitted a PR to fix this a year and a half ago. Not a great deal has happened since, which is disappointing. |
10-18 12:52:27.241 28419-28419/com.microsoft.skydrive W/BasicWebViewClient: 2016-10-18 19:52:27-e4a8b1f1-40a0-4651-b02a-6d892fe16241-Cancel error:invalid_resource ver:1.10.0 AADSTS50001: Resource 'https://officeapps.live.com' is disabled.
Trace ID: aed66cce-a470-4595-aedb-14e63c1d3e88
Correlation ID: e4a8b1f1-40a0-4651-b02a-6d892fe16241
Timestamp: 2016-10-18 19:52:26Z
This causes com.microsoft.aad.adal.AuthenticationCancelError: User cancelled the flow RequestId:216639036 CorrelationId: e4a8b1f1-40a0-4651-b02a-6d892fe16241
at com.microsoft.aad.adal.AcquireTokenRequest.onActivityResult(AcquireTokenRequest.java:563)
at com.microsoft.aad.adal.AuthenticationContext.onActivityResult(AuthenticationContext.java:731)
While expected would be AuthenticationException with some error code
The text was updated successfully, but these errors were encountered: