Skip to content

[pull] main from expo:main#36

Merged
pull[bot] merged 9 commits into
code:mainfrom
expo:main
Jul 24, 2025
Merged

[pull] main from expo:main#36
pull[bot] merged 9 commits into
code:mainfrom
expo:main

Conversation

@pull

@pull pull Bot commented Jul 24, 2025

Copy link
Copy Markdown

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.3)

Can you help keep this open source service alive? 💖 Please sponsor : )

SYoder1 and others added 9 commits July 24, 2025 18:01
…38132)

# Why
I have not been able to reproduce this, but I have seen logs with our
users where an error is thrown for one reason or another, such as:
`Call to function 'ExpoSecureStore.getValueWithKeyAsync' has been
rejected. → Caused by: Could not Authenticate the user: Could not
authenticate the user`

Then until the app is killed, any biometrics actions (get or set) will
then throw a new error
`Call to function 'ExpoSecureStore.getValueWithKeyAsync' has been
rejected. → Caused by: Could not Authenticate the user: Authentication
is already in progress`.

# How
Looking at the code, it seems that if `openAuthenticationPrompt` throws
an error either from `assertBiometricsSupport()` or
`getCurrentActivity()`, the `isAuthenticating` flag is never reset back
to false, and "disables" biometrics until the app is killed.

# Test Plan
I haven't been able to reproduce this for my self.

# Checklist

<!--
Please check the appropriate items below if they apply to your diff.
-->

- [ ] I added a `changelog.md` entry and rebuilt the package sources
according to [this short
guide](https://github.com/expo/expo/blob/main/CONTRIBUTING.md#-before-submitting)
- [ ] This diff will work correctly for `npx expo prebuild` & EAS Build
(eg: updated a module plugin).
- [ ] Conforms with the [Documentation Writing Style
Guide](https://github.com/expo/expo/blob/main/guides/Expo%20Documentation%20Writing%20Style%20Guide.md)
# Why

- Update React Compiler to align with React 19.1. It's now in RC and
mostly stable.

<!--
Please describe the motivation for this PR, and link to relevant GitHub
issues, forums posts, or feature requests.
-->

# Test Plan

The E2E test should ensure it continues working.
@pull pull Bot locked and limited conversation to collaborators Jul 24, 2025
@pull pull Bot added the ⤵️ pull label Jul 24, 2025
@pull pull Bot merged commit 1db52f2 into code:main Jul 24, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants