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.

Sign up for GitHub

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

Jump to bottom

Biometrics not working on multiple instances #620

Closed
kober32 opened this issue Sep 13, 2024 · 2 comments
Closed

Biometrics not working on multiple instances #620

kober32 opened this issue Sep 13, 2024 · 2 comments
Assignees
@hvge
Labels
android bug critical
Milestone
1.9.0

Comments

@kober32
Copy link
Member

kober32 commented Sep 13, 2024 •
edited
Loading

On Android, when multiple instances (with different instanceId) of PowerAuthSDK are created, signing with biometrics only works on the latest instance.

This is due to the shared biometric key, which is force-recreated when the biometrics factor is turned on (or activation is removed) making any biometrics already added on other instances invalid.

We should also consider adding documentation and tests that are working with multiple instances because many nuances when working in such a setup are undocumented and unexpected by default.

Note: this is not an issue on iOS where this works correctly.
@hvge hvge added the critical label Sep 13, 2024
@hvge
Copy link
Member

hvge commented Sep 13, 2024 •
edited
Loading

Raised bug to "critical", because it's already problematic on client's production.

@hvge
Copy link
Member

hvge commented Sep 13, 2024

The issue has to be fixed in the following release branches:

  • release/1.7.x - the latest SDK supported in React Native wrapper
  • release/1.8.x
  • release/1.9x - upcoming release still in the development

@hvge hvge added this to the 1.9.0 milestone Sep 13, 2024
hvge added a commit that referenced this issue Sep 16, 2024
@hvge
Android: #620: Added missing documentation
b4cce80
hvge added a commit that referenced this issue Sep 17, 2024
@hvge
Android: #620: Mapping logic moved to separate class
9c4ebb7
hvge added a commit that referenced this issue Sep 19, 2024
@hvge
Android: Fix #620: Store biometry key for each PowerAuthSDK instance …
14933b1 
…(1.7.x branch)
hvge added a commit that referenced this issue Sep 20, 2024
@hvge
Android: #620: Deprecate removeActivationLocal(Context, boolean) (1.7.x)
423ed0a
hvge added a commit that referenced this issue Sep 20, 2024
@hvge
Android: Fix #620: Store biometry key for each PowerAuthSDK instance …
ed652a5 
…(1.8.x branch)
hvge added a commit that referenced this issue Sep 20, 2024
@hvge
Android: #620: Deprecate removeActivationLocal(Context, boolean)
08f7281
hvge added a commit that referenced this issue Sep 20, 2024
@hvge
Android: #620: Updated documentation
859dcff
hvge added a commit that referenced this issue Sep 20, 2024
@hvge
Android: #620: Updated documentation (1.8.x)
7761695
hvge added a commit that referenced this issue Sep 20, 2024
@hvge
Android: #620: Updated documentation (1.7.x)
0073cc6
@hvge hvge mentioned this issue Sep 23, 2024
Closed
hvge added a commit that referenced this issue Sep 23, 2024
@hvge
Merge pull request #626 from /issues/620-android-biometry-1.7.x
9756636 
Store biometry key for each PowerAuthSDK instance (1.7.x)
hvge added a commit that referenced this issue Sep 23, 2024
@hvge
Merge pull request #627 from /issues/620-android-biometry-1.8.x
b82a808 
Store biometry key for each PowerAuthSDK instance (1.8.x)
@hvge hvge closed this as completed in ffd98b4 Sep 23, 2024
hvge added a commit that referenced this issue Sep 23, 2024
@hvge
Merge pull request #621 from /issues/620-android-biometry
e1c8460 
Store biometry key for each PowerAuthSDK instance
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hvge hvge

Labels
android bug critical
Projects
None yet
Milestone
1.9.0
Development

No branches or pull requests
2 participants
@hvge @kober32