Add coverage and support for Bcrypt v2b

Author: ptoffy

.github/workflows/test.yml

@@ -1,4 +1,4 @@
-name: Lint, Build & Test
+name: Lint, Build & Test with Coverage
 on: 
   push:
     branches: [ main ]
@@ -18,5 +18,9 @@ jobs:
         run: swift build
       - name: Test
         run: swift test
+      - name: Generate Coverage Report
+        uses: vapor/[email protected]
+        with:
+          codecov_token: ${{ secrets.CODECOV_TOKEN }}
 
 

Sources/Bcrypt/BcryptError.swift

@@ -3,4 +3,5 @@ public enum BcryptError: Error {
     case invalidSalt
     case emptyPassword
     case invalidCost
+    case passwordTooLong
 }

Sources/Bcrypt/EksBlowfish.swift

@@ -144,28 +144,14 @@
 
             return (a &+ b) ^ c &+ d
         }
-
-        func BLFRND(s: [[UInt32]], p: [UInt32], i: inout UInt32, j: UInt32, n: Int) {
-            i ^= F(s: s, x: j) ^ p[n]
+        
+        var i = 1
+        while i <= 16 {
+            Xr ^= F(s: s, x: Xl) ^ p[i]
+            Xl ^= F(s: s, x: Xr) ^ p[i + 1]
+            i &+= 2
         }
 
-        BLFRND(s: s, p: p, i: &Xr, j: Xl, n: 1)
-        BLFRND(s: s, p: p, i: &Xl, j: Xr, n: 2)
-        BLFRND(s: s, p: p, i: &Xr, j: Xl, n: 3)
-        BLFRND(s: s, p: p, i: &Xl, j: Xr, n: 4)
-        BLFRND(s: s, p: p, i: &Xr, j: Xl, n: 5)
-        BLFRND(s: s, p: p, i: &Xl, j: Xr, n: 6)
-        BLFRND(s: s, p: p, i: &Xr, j: Xl, n: 7)
-        BLFRND(s: s, p: p, i: &Xl, j: Xr, n: 8)
-        BLFRND(s: s, p: p, i: &Xr, j: Xl, n: 9)
-        BLFRND(s: s, p: p, i: &Xl, j: Xr, n: 10)
-        BLFRND(s: s, p: p, i: &Xr, j: Xl, n: 11)
-        BLFRND(s: s, p: p, i: &Xl, j: Xr, n: 12)
-        BLFRND(s: s, p: p, i: &Xr, j: Xl, n: 13)
-        BLFRND(s: s, p: p, i: &Xl, j: Xr, n: 14)
-        BLFRND(s: s, p: p, i: &Xr, j: Xl, n: 15)
-        BLFRND(s: s, p: p, i: &Xl, j: Xr, n: 16)
-
         xl = Xr ^ p[17]
         xr = Xl
     }

Sources/Bcrypt/Hasher.swift

@@ -58,6 +58,14 @@ extension Bcrypt {
             } else {
                 password + [0]
             }
+        
+        switch version {
+        case .v2a: break
+        case .v2b:
+            guard password.count <= 72 else {
+                throw BcryptError.passwordTooLong
+            }
+        }
 
         if cost < 4 || cost > 31 {
             throw BcryptError.invalidCost