-
Notifications
You must be signed in to change notification settings - Fork 209
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
Android: OTP auth browser closes when minimising app #921
Comments
Has anyone else experienced this or is it just me? Seems to happen in basic example project so would assume its universal, or is there some additional intent configuration needed to fix this maybe? |
We ran into something like this and changing the |
We are experiencing the same. Changing the launch mode to |
We ended up stopping the use of web browser for OTP login and rather using the flow through Auth0 api:
This also fixed some other Android issues when using the Auth0 api hooks and authenticate() flow through web browserwhere Credentials fail be set on first login attempt due to an error related to "Lock screen code has changed" (although it has not...) |
@mateoguzmana Do you recall how to reproduce the buggy deep link behavior? We're still testing the change but haven't seen anything like that yet. We're using EDIT: We did end up running into a problem where the app wouldn't load from a deep/universal link when the app was totally closed. It worked fine when tapping a deep/universal link while the app was in the background, but would just show a blank screen with our primary app color for a moment then shut down when the app's previous state was "closed". |
@freethejazz good to know if works good for you! our reproduction path was quite simple:
We are still investigating the culprit of this as well though. Perhaps it is something else related to our configuration. We are using |
I may have found a better lead to this issue. It seems like other libraries have faced the same, and there is a good workaround for this. I am not an Android expert but what I understood after all this research: By default if you use To address the above, you can wrap your main activity with another activity that checks whether the main activity has been launched or not and then prevent re-launching it. Which helps keeping any other activity open and any other library depending on See:
I implemented the fix on our side, and everything seems to work as expected now, including deep links. We will continue testing for any possible side effects of this solution, but so far it looks very promising. I wanted to share this update here in case it helps anyone. |
@mateoguzmana Thank you for sharing, very helpful! We looked into the links you shared and have implemented the fix ourselves and it's seemed to resolve all of our issues as well. Given the in-app browser/universal login approach seems to be the recommended path by Auth0, and that handling deep links is such a common use case, I would propose that the Auth0 team consider adding this workaround as part of the android setup steps in this library's docs. @poovamraj: I've seen you've been an active contributor on the android side of this project and other android Auth0 projects. Would you be the right person to review this? |
Checklist
Description
When authorising with email OTP on Android the browser here the user enters their authentication details will be closed if the user minimises the app and returns.
This is a typical usage scenario when the user needs to check their email app for OTP password.
I have seen a variety of issues related to this from some years back, all of them closed. But the issue seems to be present still.
Reproduction
Reproduced in the samples app
Additional context
The issue is reproducable in the minimal samples project
Note the samples app was modified to use latest package versions:
react-native-auth0 version
3.2.1
React Native version
0.74.2
Expo version
51.0.11
Platform
Android
Platform version(s)
14
The text was updated successfully, but these errors were encountered: