Skip to content

feat: add CredentialAlreadyExists error for matched excludeCredentials#98

Open
LaustStream wants to merge 1 commit intof-23:stablefrom
LaustStream:fix/ios-1006-matched-excluded-credential
Open

feat: add CredentialAlreadyExists error for matched excludeCredentials#98
LaustStream wants to merge 1 commit intof-23:stablefrom
LaustStream:fix/ios-1006-matched-excluded-credential

Conversation

@LaustStream
Copy link
Copy Markdown

Fixes #97

iOS: map ASAuthorizationError 1006 (matchedExcludedCredential, iOS 18+). Android: detect CreatePublicKeyCredentialDomException with InvalidStateError + 'excluded credential' message. JS: add CredentialAlreadyExistsError + handleNativeError case.

Follows the precedent of #86 (UserCancelled for code 1001).

iOS: map ASAuthorizationError 1006 (matchedExcludedCredential, iOS 18+).
Android: detect CreatePublicKeyCredentialDomException with
InvalidStateError + 'excluded credential' message.
JS: add CredentialAlreadyExistsError + handleNativeError case.

Follows the precedent of f-23#86 (UserCancelled for code 1001).
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.

Add CredentialAlreadyExists error type for matched excludeCredentials (iOS 1006 / Android InvalidStateError)

1 participant