-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Passkey Support #12733
base: main
Are you sure you want to change the base?
Passkey Support #12733
Conversation
[Passkey #1 cloned] start passkey enrollment rpc (#11872) 1. implemented rpc call to startPasskeyEnrollment 2. defined startPasskeyEnrollment request and response 3. added unit tests [Passkey #2] finalize passkey enrollment rpc (#11885) Implemented finalizePasskeyEnrollment rpc call Defined finalizePasskeyEnrollment Request and Response object Added unit tests [passkey #3] startPasskeySignIn RPC call (#11889) Implemented start passkey sign in rpc call Defined startPasskeySignInRequest & Response Added Unit tests [Passkey #4] finalizePasskeySignIn rpc (#11904) 1. Implemented passkey finalize sign in rpc call 2. Defined finalizePasskeySignIn request & response 3. Added unit tests. [Passkey #5] Auth startPasskeySignInWithCompletion: and finalizePasskeySignInWithPlatformCredential:completion (#11952) [Passkey #6] Update passkey related proto naming change (#11953) [Passkey #7] FIRUser startPasskeyEnrollmentWithName:completion: & finalizePasskeyEnrollmentWithPlatformCredential:completion: implementation (#11991) [Passkey #8] Added enrolledPasskeys support on getAccountInfo rpc (#12002) [Passkey #9] Passkey withdraw (#12127)
Generated by 🚫 Danger |
Apple API Diff ReportCommit: 1949880 FirebaseAuthClassesFIRUser[ADDED] -finalizePasskeyEnrollmentWithPlatformCredential:completion:Swift:
+ func finalizePasskeyEnrollment ( with platformCredential : ASAuthorizationPlatformPublicKeyCredentialRegistration ) async throws -> AuthDataResult
Objective-C:
+ - ( void ) finalizePasskeyEnrollmentWithPlatformCredential : ( nonnull ASAuthorizationPlatformPublicKeyCredentialRegistration * ) platformCredential completion : ( nullable void ( ^ )( FIRAuthDataResult * _Nullable , NSError * _Nullable )) completion ; [ADDED] -startPasskeyEnrollmentWithName:completion:Swift:
+ func startPasskeyEnrollment ( with name : String ?) async throws -> ASAuthorizationPlatformPublicKeyCredentialRegistrationRequest
Objective-C:
+ - ( void ) startPasskeyEnrollmentWithName :( nullable NSString * ) name completion : ( nullable void ( ^ )( ASAuthorizationPlatformPublicKeyCredentialRegistrationRequest * _Nullable , NSError * _Nullable )) completion ; [ADDED] enrolledPasskeysSwift:
+ var enrolledPasskeys : [ PasskeyInfo ] { get }
Objective-C:
+ @property ( nonatomic , readonly ) API_UNAVAILABLE ( watchos ) NSArray < FIRPasskeyInfo *> * enrolledPasskeys ; [ADDED] -unenrollPasskeyWithCredentialID:completion:Swift:
+ func unenrollPasskey ( with credentialID : String ) async throws
Objective-C:
+ - ( void ) unenrollPasskeyWithCredentialID :( nonnull NSString * ) credentialID completion :( nullable void ( ^ )( NSError * _Nullable )) completion ; FIRAuth[ADDED] -startPasskeySignInWithCompletion:Swift:
+ func startPasskeySignIn () async throws -> ASAuthorizationPlatformPublicKeyCredentialAssertionRequest
Objective-C:
+ - ( void ) startPasskeySignInWithCompletion : ( nullable void ( ^ )( ASAuthorizationPlatformPublicKeyCredentialAssertionRequest * _Nullable , NSError * _Nullable )) completion ; [ADDED] -finalizePasskeySignInWithPlatformCredential:completion:Swift:
+ func finalizePasskeySignIn ( with platformCredential : ASAuthorizationPlatformPublicKeyCredentialAssertion ) async throws -> FIRAuthDataResult
Objective-C:
+ - ( void ) finalizePasskeySignInWithPlatformCredential : ( nonnull ASAuthorizationPlatformPublicKeyCredentialAssertion * ) platformCredential completion : ( nullable void ( ^ )( FIRAuthDataResult * _Nullable , NSError * _Nullable )) completion ; [ADDED] FIRPasskeyInfo[ADDED] FIRPasskeyInfoSwift:
+ class PasskeyInfo : NSObject
+ var name : String { get }
+ var credentialID : String { get }
Objective-C:
+ @interface FIRPasskeyInfo : NSObject
+ @property ( nonatomic , readonly ) NSString * _Nonnull name ;
+ @property ( nonatomic , readonly ) NSString * _Nonnull credentialID ; EnumerationsFIRAuthErrorCode[ADDED] FIRAuthErrorCodePasskeyEnrollmentNotFoundSwift:
+ case passkeyEnrollmentNotFound = 17209
Objective-C:
+ FIRAuthErrorCodePasskeyEnrollmentNotFound = 17209 |
Coverage Report 1Affected Products
Test Logs |
@brief start sign in with passkey retrieving challenge from GCIP and create an assertion request. | ||
@param completion Optionally; a block which creates a assertation request. | ||
|
||
@remarks // TODO @liubinj add possible error codes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is a fast follow up after private preview. Let me know if you think it's better to remove it for now @kevinthecheung
This PR is for TW review for private preview. We will not merge it to main branch.