-
Notifications
You must be signed in to change notification settings - Fork 119
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
Amplify.Auth.fetchAuthSession() returns isSignedIn value false even though refresh token hasn't expired in Samsung Galaxy A53 #2725
Comments
It would be helpful to try and log the exception which is provided in each of the result types inside the auth session. Without more information into the issue (especially if you are only seeing this on a single device type), its difficult to determine what may be happening. |
Thanks @tylerjroach for the reply. For now, I've updated the library from 2.13.0 to 2.14.10 and pushed a release. I've yet to see how that affects this issue on the said device. This issue does seem very odd as it has only happened on a single device. Will update when I can. |
Update: |
@tylerjroach I'm wondering if there is some other external factor that could be at play, eg. VPN, Disk Cleaner App etc, or perhaps some setting we need to update from the AWS console. If there is a direction you could point me to, then let me know please. |
Hi @bijaykumarpun , Thanks for posting the issue, we will take a look and provide updates |
Marking this as a bug pending further investigation. @bijaykumarpun would you be able to take @tylerjroach's suggestion and add log messages indicating what errors, if any, are returned in the various results in |
@mattcreaser Yes, will get back with the exception result. |
Error logs from the user.
cc: @tylerjroach It seems the Furthermore, the user who faces this issue may have DST (Daylight Saving Time), and I want to know if the DST can cause any issue as such? |
DST should not cause an issue unless the user is in a region that recently added or removed DST and their Operating System has not updated to be aware that the region is participating (or not) in DST). It's possible that a library has added the internet permission for you and merged that permission into the manifest but it is best to add permissions on your own app manifest. However, if the users phone does not have internet, this would be the cause reason for the failures as seen. |
Hi @bijaykumarpun, I just happened to run into this issue and noticed its similarity to a different issue (#2783). Essentially if the user is offline and the access token has expired, |
this case facing me also (and im using the v2.16.1) this workaround works for me . |
Yes @vincetran, that was most likely the case with the user being affected. |
I believe this issue was fixed in the referenced by #2830 |
This issue is now closed. Comments on closed issues are hard for our team to see. |
Before opening, please confirm:
Language and Async Model
Kotlin - Coroutines
Amplify Categories
Authentication
Gradle script dependencies
Environment information
Please include any relevant guides or documentation you're referencing
No response
Describe the bug
This is a very device-specific issue and reported on Samsung Galaxy A53 SM-A536E only.
AWSCognitoAuthSession.isSignedIn
value istrue
AWSCognitoAuthSession.isSignedIn
value isfalse
and access token is not refreshedBecause the access token is not refreshed, they are navigated to the sign in screen (this is our own logic).
I want to know if there is anything that could potentially prevent the session token from being refreshed. As this issue has occurred in only one device, and debugging isn't going to be easy/accessible, I'm trying to see if there is anything that I missed on my end and how I can solve this issue.
I want to emphasize again that this only occurs on one device only (as far as the reports go), and I haven't been able to reproduce on any other deivces.
Code Snippet
Log output
amplifyconfiguration.json
GraphQL Schema
Additional information and screenshots
No response
The text was updated successfully, but these errors were encountered: