Fixed mistakes constructing a valid JWK from public key data

m-barthelemy · Nov 18, 2024 · 2efa730 · 2efa730
1 parent b93d4df
commit 2efa730
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 10 deletions.
diff --git a/Sources/AcmeSwift/APIs/AcmeSwift+Orders.swift b/Sources/AcmeSwift/APIs/AcmeSwift+Orders.swift
@@ -334,9 +334,9 @@ extension AcmeSwift {
             let jwk = JWK.ecdsa(
                 nil,
                 identifier: nil,
-                x: publicKey.prefix(upTo: publicKey.count/2).base64EncodedString(),
-                y: publicKey.suffix(from: publicKey.count/2).base64EncodedString(),
-                curve: nil
+                x: publicKey.prefix(publicKey.count/2).toBase64UrlString(),
+                y: publicKey.suffix(publicKey.count/2).toBase64UrlString(),
+                curve: .p256
             )
             let encoder = JSONEncoder()
             encoder.outputFormatting = .sortedKeys
@@ -347,6 +347,6 @@ extension AcmeSwift {
 
 extension SHA256Digest {
     var base64URLString: String {
-        Data(self).base64EncodedString().base64ToBase64Url()
+        Data(self).toBase64UrlString()
     }
 }
diff --git a/Sources/AcmeSwift/Helpers/AcmeRequestBody.swift b/Sources/AcmeSwift/Helpers/AcmeRequestBody.swift
@@ -65,12 +65,12 @@ struct AcmeRequestBody<T: EndpointProtocol>: Encodable {
 
         self.protected = .init(
             alg: .es256,
-            jwk: accountURL == nil ? JWTKit.JWK.ecdsa(
+            jwk: accountURL == nil ? JWK.ecdsa(
                 nil,
                 identifier: nil,
-                x: publicKey.prefix(upTo: publicKey.count/2).base64EncodedString(),
-                y: publicKey.suffix(from: publicKey.count/2).base64EncodedString(),
-                curve: nil
+                x: publicKey.prefix(publicKey.count/2).toBase64UrlString(),
+                y: publicKey.suffix(publicKey.count/2).toBase64UrlString(),
+                curve: .p256
             ) : nil,
             kid: accountURL,
             nonce: nonce,

diff --git a/Sources/AcmeSwift/Models/AcmeError.swift b/Sources/AcmeSwift/Models/AcmeError.swift
@@ -17,8 +17,6 @@ public enum AcmeError: Error, Sendable {
     /// No nonce (anti-replay) value was returned by the endpoint
     case noNonceReturned
 
-    case jwsEncodeError(String)
-
     case dataCorrupted(String)
     case errorCode(UInt, String)