chore: server code formatting

packages/server/lib/new-admin/graphql/modules/authentication/FIDOPasswordlessStrategy.js

@@ -11,35 +11,41 @@ const devMode = require('minimist')(process.argv.slice(2)).dev
 const REMEMBER_ME_AGE = 90 * T.day
 
 const generateAttestationOptions = (session, options) => {
-  return users.getUserById(options.userId).then(user => {
-    return Promise.all([credentials.getHardwareCredentialsByUserId(user.id), user])
-  }).then(([userDevices, user]) => {
-    const opts = simpleWebauthn.generateAttestationOptions({
-      rpName: 'Lamassu',
-      rpID: options.domain,
-      userName: user.username,
-      userID: user.id,
-      timeout: 60000,
-      attestationType: 'indirect',
-      excludeCredentials: userDevices.map(dev => ({
-        id: dev.data.credentialID,
-        type: 'public-key',
-        transports: ['usb', 'ble', 'nfc', 'internal']
-      })),
-      authenticatorSelection: {
-        userVerification: 'discouraged',
-        requireResidentKey: false
-      }
+  return users
+    .getUserById(options.userId)
+    .then(user => {
+      return Promise.all([
+        credentials.getHardwareCredentialsByUserId(user.id),
+        user,
+      ])
     })
+    .then(([userDevices, user]) => {
+      const opts = simpleWebauthn.generateAttestationOptions({
+        rpName: 'Lamassu',
+        rpID: options.domain,
+        userName: user.username,
+        userID: user.id,
+        timeout: 60000,
+        attestationType: 'indirect',
+        excludeCredentials: userDevices.map(dev => ({
+          id: dev.data.credentialID,
+          type: 'public-key',
+          transports: ['usb', 'ble', 'nfc', 'internal'],
+        })),
+        authenticatorSelection: {
+          userVerification: 'discouraged',
+          requireResidentKey: false,
+        },
+      })
 
-    session.webauthn = {
-      attestation: {
-        challenge: opts.challenge
+      session.webauthn = {
+        attestation: {
+          challenge: opts.challenge,
+        },
       }
-    }
 
-    return opts
-  })
+      return opts
+    })
 }
 
 const generateAssertionOptions = (session, options) => {
@@ -50,16 +56,16 @@ const generateAssertionOptions = (session, options) => {
         allowCredentials: devices.map(dev => ({
           id: dev.data.credentialID,
           type: 'public-key',
-          transports: ['usb', 'ble', 'nfc', 'internal']
+          transports: ['usb', 'ble', 'nfc', 'internal'],
         })),
         userVerification: 'discouraged',
-        rpID: options.domain
+        rpID: options.domain,
       })
 
       session.webauthn = {
         assertion: {
-          challenge: opts.challenge
-        }
+          challenge: opts.challenge,
+        },
       }
 
       return opts
@@ -77,40 +83,38 @@ const validateAttestation = (session, options) => {
       credential: options.attestationResponse,
       expectedChallenge: `${expectedChallenge}`,
       expectedOrigin: `https://${options.domain}${devMode ? `:3001` : ``}`,
-      expectedRPID: options.domain
-    })
-  ])
-    .then(([user, verification]) => {
-      const { verified, attestationInfo } = verification
+      expectedRPID: options.domain,
+    }),
+  ]).then(([user, verification]) => {
+    const { verified, attestationInfo } = verification
 
-      if (!(verified || attestationInfo)) {
-        session.webauthn = null
-        return false
-      }
+    if (!(verified || attestationInfo)) {
+      session.webauthn = null
+      return false
+    }
 
-      const {
-        counter,
-        credentialPublicKey,
-        credentialID
-      } = attestationInfo
+    const { counter, credentialPublicKey, credentialID } = attestationInfo
 
-      return credentials.getHardwareCredentialsByUserId(user.id)
-        .then(userDevices => {
-          const existingDevice = userDevices.find(device => device.data.credentialID === credentialID)
+    return credentials
+      .getHardwareCredentialsByUserId(user.id)
+      .then(userDevices => {
+        const existingDevice = userDevices.find(
+          device => device.data.credentialID === credentialID,
+        )
 
-          if (!existingDevice) {
-            const newDevice = {
-              counter,
-              credentialPublicKey,
-              credentialID
-            }
-            credentials.createHardwareCredential(user.id, newDevice)
+        if (!existingDevice) {
+          const newDevice = {
+            counter,
+            credentialPublicKey,
+            credentialID,
           }
+          credentials.createHardwareCredential(user.id, newDevice)
+        }
 
-          session.webauthn = null
-          return verified
-        })
-    })
+        session.webauthn = null
+        return verified
+      })
+  })
 }
 
 const validateAssertion = (session, options) => {
@@ -119,17 +123,24 @@ const validateAssertion = (session, options) => {
 
     return credentials.getHardwareCredentialsByUserId(user.id).then(devices => {
       const dbAuthenticator = _.find(dev => {
-        return Buffer.from(dev.data.credentialID).compare(base64url.toBuffer(options.assertionResponse.rawId)) === 0
+        return (
+          Buffer.from(dev.data.credentialID).compare(
+            base64url.toBuffer(options.assertionResponse.rawId),
+          ) === 0
+        )
       }, devices)
 
       if (!dbAuthenticator.data) {
-        throw new Error(`Could not find authenticator matching ${options.assertionResponse.id}`)
+        throw new Error(
+          `Could not find authenticator matching ${options.assertionResponse.id}`,
+        )
       }
 
-      const convertedAuthenticator = _.merge(
-        dbAuthenticator.data,
-        { credentialPublicKey: Buffer.from(dbAuthenticator.data.credentialPublicKey) }
-      )
+      const convertedAuthenticator = _.merge(dbAuthenticator.data, {
+        credentialPublicKey: Buffer.from(
+          dbAuthenticator.data.credentialPublicKey,
+        ),
+      })
 
       let verification
       try {
@@ -138,7 +149,7 @@ const validateAssertion = (session, options) => {
           expectedChallenge: `${expectedChallenge}`,
           expectedOrigin: `https://${options.domain}${devMode ? `:3001` : ``}`,
           expectedRPID: options.domain,
-          authenticator: convertedAuthenticator
+          authenticator: convertedAuthenticator,
         })
       } catch (err) {
         console.error(err)
@@ -148,20 +159,22 @@ const validateAssertion = (session, options) => {
       const { verified, assertionInfo } = verification
 
       if (!verified) {
-        context.req.session.webauthn = null
         return false
       }
 
       dbAuthenticator.data.counter = assertionInfo.newCounter
-      return credentials.updateHardwareCredential(dbAuthenticator)
-        .then(() => {
-          const finalUser = { id: user.id, username: user.username, role: user.role }
-          session.user = finalUser
-          if (options.rememberMe) session.cookie.maxAge = REMEMBER_ME_AGE
+      return credentials.updateHardwareCredential(dbAuthenticator).then(() => {
+        const finalUser = {
+          id: user.id,
+          username: user.username,
+          role: user.role,
+        }
+        session.user = finalUser
+        if (options.rememberMe) session.cookie.maxAge = REMEMBER_ME_AGE
 
-          session.webauthn = null
-          return verified
-        })
+        session.webauthn = null
+        return verified
+      })
     })
   })
 }
@@ -170,5 +183,5 @@ module.exports = {
   generateAttestationOptions,
   generateAssertionOptions,
   validateAttestation,
-  validateAssertion
+  validateAssertion,
 }