Skip to content

Commit

Permalink
Merge pull request #2967 from 18F/stevegsa-fix-webauthn-supported-pro…
Browse files Browse the repository at this point in the history
…tocols

LG-1337 Fix supported webauthn protocols
  • Loading branch information
stevegsa authored May 28, 2019
2 parents 5e6e9f6 + 93f4314 commit 52716af
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 7 deletions.
6 changes: 1 addition & 5 deletions app/javascript/app/webauthn.js
Original file line number Diff line number Diff line change
Expand Up @@ -44,11 +44,7 @@ const enrollWebauthnDevice = ({ userId, userEmail, userChallenge, excludeCredent
},
{
type: 'public-key',
alg: -8, // EdDSA
},
{
type: 'public-key',
alg: -35, // ECDSA w/SHA-384
alg: -257,
},
],
timeout: 800000,
Expand Down
3 changes: 1 addition & 2 deletions spec/javascripts/app/webauthn_spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -49,8 +49,7 @@ describe('WebAuthn', () => {
},
pubKeyCredParams: [
{ type: 'public-key', alg: -7 },
{ type: 'public-key', alg: -8 },
{ type: 'public-key', alg: -35 },
{ type: 'public-key', alg: -257 },
],
timeout: 800000,
attestation: 'none',
Expand Down

0 comments on commit 52716af

Please sign in to comment.