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

fix: secure storage unable to read data from keychain #231

Merged
merged 1 commit into from
Apr 12, 2022
Merged

fix: secure storage unable to read data from keychain #231

merged 1 commit into from
Apr 12, 2022

Conversation

JoniVR
Copy link
Contributor

@JoniVR JoniVR commented Apr 12, 2022

In some cases, the app could be initialised before secure storage was ready to be read (isProtectedDataAvailable was false, see docs). This could lead to data seemingly being "lost", while in fact it was just the keychain not being ready to read the data yet.

This introduces a workaround that will wait for the keychain to be ready before fully initialising the app.

fixes #195

In some cases, the app could be initialised before secure storage was ready to be read (`isProtectedDataAvailable` was false). This could lead to data seemingly being "lost", while in fact it was just the keychain not being ready to read the data yet.

This introduces a workaround that will wait for the keychain to be ready before fully initialising the app.
@vercel
Copy link

vercel bot commented Apr 12, 2022

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/ammarahm-ed/rnmmkv/GWfGcSEzvQZJMAUQNxNtV2MycEBd
✅ Preview: https://rnmmkv-git-fork-jonivr-fix-secure-storage-data-bug-ammarahm-ed.vercel.app

@JoniVR
Copy link
Contributor Author

JoniVR commented Apr 12, 2022

@ammarahm-ed If possible, please release this as a patch version if 0.7.0 isn't ready yet, I'd love to deploy this asap :)

@ammarahm-ed ammarahm-ed merged commit c10a624 into ammarahm-ed:master Apr 12, 2022
@ammarahm-ed
Copy link
Owner

I will make a new release.

@JoniVR JoniVR deleted the fix/secure-storage-data-bug branch April 12, 2022 20:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] Data missing from storage after app restart
2 participants