summaryrefslogtreecommitdiffhomepage
path: root/packages/browser/src
diff options
context:
space:
mode:
authorMatthew Miller <matthew@millerti.me>2022-12-10 22:40:45 -0800
committerGitHub <noreply@github.com>2022-12-10 22:40:45 -0800
commit461b67c0c6fadfb55c1da6c9c8ab2693ba4c9a03 (patch)
treeae713d608966ac977390e7bbfbeab5ea79a191bc /packages/browser/src
parent33528afe001d4aca62052dce204c0398c3127ffd (diff)
parent90dd7f247182329987a8e23f476d9280d8d5c265 (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.ts6
-rw-r--r--packages/browser/src/methods/startAuthentication.test.ts3
-rw-r--r--packages/browser/src/methods/startRegistration.test.ts7
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 = {