Skip to content

Conversation

pjf-dev
Copy link
Contributor

@pjf-dev pjf-dev commented Apr 14, 2025

Description

Fix #2632: This fixes the issue described in #2632 where authentication would fail without prompting for fallback credentials when biometry wasn't available or enrolled on iOS platforms.

Root cause

The authenticate function checked the stored biometric status and failed immediately if unavailable, ignoring fallback logic when allowDeviceCredential = true.

Proposed solution

Ensure fallback credential is also disabled before failing early due to biometric availability in the iOS module of the plugin.

Additional Notes

I've tested the fix on a physical iPhone 15 Pro Max running iOS 18.3 and confirmed that a fallback prompt now appears if biometry is disabled and fallback is allowed.

@pjf-dev pjf-dev requested a review from a team as a code owner April 14, 2025 23:29
@pjf-dev pjf-dev changed the title fix(biometric) prompt for fallback credentials when enabled and biometry unavailable on iOS (fixes #2632) fix(biometric) prompt for fallback credentials when enabled and biometry unavailable on iOS (fix #2632) Apr 14, 2025
@pjf-dev
Copy link
Contributor Author

pjf-dev commented Apr 14, 2025

I believe I covered mostly everything, but let me know if anything else is needed for this PR

@pjf-dev pjf-dev force-pushed the fix/biometric/ios-fallback-authentication branch from cab0eff to b615f47 Compare April 15, 2025 13:55
@FabianLars FabianLars requested a review from lucasfernog April 15, 2025 14:39
Copy link
Contributor

Package Changes Through b615f47

There are 4 changes which include log with minor, log-js with minor, biometric with patch, biometric-js with patch

Planned Package Versions

The following package releases are the planned based on the context of changes in this pull request.

package current next
api-example 2.0.24 2.0.25
api-example-js 2.0.20 2.0.21
biometric 2.2.0 2.2.1
biometric-js 2.2.0 2.2.1
log 2.3.1 2.4.0
log-js 2.3.1 2.4.0

Add another change file through the GitHub UI by following this link.


Read about change files or the docs at github.com/jbolda/covector

Copy link
Member

@lucasfernog lucasfernog left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice catch!

@lucasfernog lucasfernog merged commit 10f9e66 into tauri-apps:v2 Apr 16, 2025
8 checks passed
@pjf-dev pjf-dev deleted the fix/biometric/ios-fallback-authentication branch April 16, 2025 21:03
gezihuzi pushed a commit to Hypobenthos/plugins-workspace that referenced this pull request Jun 22, 2025
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] [biometric] Fallback logic ignored when biometry is unavailable on iOS
2 participants