diff options
author | Matthew Miller <matthew@millerti.me> | 2022-12-10 22:40:45 -0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-12-10 22:40:45 -0800 |
commit | 461b67c0c6fadfb55c1da6c9c8ab2693ba4c9a03 (patch) | |
tree | ae713d608966ac977390e7bbfbeab5ea79a191bc /packages/browser/src | |
parent | 33528afe001d4aca62052dce204c0398c3127ffd (diff) | |
parent | 90dd7f247182329987a8e23f476d9280d8d5c265 (diff) |
Merge pull request #299 from MasterKale/feat/isomorphic
feat/isomorphic
Diffstat (limited to 'packages/browser/src')
-rw-r--r-- | packages/browser/src/helpers/webAuthnAbortService.ts | 6 | ||||
-rw-r--r-- | packages/browser/src/methods/startAuthentication.test.ts | 3 | ||||
-rw-r--r-- | packages/browser/src/methods/startRegistration.test.ts | 7 |
3 files changed, 14 insertions, 2 deletions
diff --git a/packages/browser/src/helpers/webAuthnAbortService.ts b/packages/browser/src/helpers/webAuthnAbortService.ts index 479df55..8955526 100644 --- a/packages/browser/src/helpers/webAuthnAbortService.ts +++ b/packages/browser/src/helpers/webAuthnAbortService.ts @@ -15,8 +15,10 @@ class WebAuthnAbortService { this.controller.abort('Cancelling existing WebAuthn API call for new one'); } - this.controller = new AbortController(); - return this.controller.signal; + const newController = new AbortController(); + + this.controller = newController; + return newController.signal; } } diff --git a/packages/browser/src/methods/startAuthentication.test.ts b/packages/browser/src/methods/startAuthentication.test.ts index e112124..9ebdd77 100644 --- a/packages/browser/src/methods/startAuthentication.test.ts +++ b/packages/browser/src/methods/startAuthentication.test.ts @@ -116,6 +116,7 @@ test('should return base64url-encoded response values', async () => { }, getClientExtensionResults: () => ({}), type: 'webauthn.get', + authenticatorAttachment: '', }); }); }); @@ -163,7 +164,9 @@ test('should send extensions to authenticator if present in options', async () = const extensions: AuthenticationExtensionsClientInputs = { credProps: true, appid: 'appidHere', + // @ts-ignore uvm: true, + // @ts-ignore appidExclude: 'appidExcludeHere', }; const optsWithExts: PublicKeyCredentialRequestOptionsJSON = { diff --git a/packages/browser/src/methods/startRegistration.test.ts b/packages/browser/src/methods/startRegistration.test.ts index 2dc11be..38ea52e 100644 --- a/packages/browser/src/methods/startRegistration.test.ts +++ b/packages/browser/src/methods/startRegistration.test.ts @@ -96,9 +96,14 @@ test('should return base64url-encoded response values', async () => { response: { attestationObject: Buffer.from(mockAttestationObject, 'ascii'), clientDataJSON: Buffer.from(mockClientDataJSON, 'ascii'), + getTransports: () => [], + getAuthenticatorData: () => new Uint8Array(), + getPublicKey: () => null, + getPublicKeyAlgorithm: () => -999, }, getClientExtensionResults: () => ({}), type: 'webauthn.create', + authenticatorAttachment: '', }); }); }); @@ -132,7 +137,9 @@ test('should send extensions to authenticator if present in options', async () = const extensions: AuthenticationExtensionsClientInputs = { credProps: true, appid: 'appidHere', + // @ts-ignore uvm: true, + // @ts-ignore appidExclude: 'appidExcludeHere', }; const optsWithExts: PublicKeyCredentialCreationOptionsJSON = { |