Merge pull request #3401 from safe-global/GH-3400/fix-hardware-wallet…

…-signatures GH-3400 transaction creation with ledger and keystone
safe-global · Apr 8, 2024 · b7bdc71 · b7bdc71
2 parents e7458a9 + 1eaf6a5
commit b7bdc71
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/Multisig/Logic/Ledger/LedgerController.swift b/Multisig/Logic/Ledger/LedgerController.swift
@@ -103,7 +103,7 @@ class LedgerController {
 
                 let gnosisSafeSignature = r + s + Data([v + 4])
 
-                completion(gnosisSafeSignature.toHexString(), nil)
+                completion(gnosisSafeSignature.toHexStringWithPrefix(), nil)
 
             case .failure(_):
                 completion(nil, "Please check that Ethereum App is running on the Ledger device.")

diff --git a/Multisig/UI/Settings/OwnerKeyManagement/KeystoneOwnerKey/KeystoneSignFlow.swift b/Multisig/UI/Settings/OwnerKeyManagement/KeystoneOwnerKey/KeystoneSignFlow.swift
@@ -92,7 +92,7 @@ extension KeystoneSignFlow: QRCodeScannerViewControllerDelegate {
 extension SECP256K1.UnmarshaledSignature {
     var safeSignature: String {
         let signature = r + s + Data([v + 4])
-        return signature.toHexString()
+        return signature.toHexStringWithPrefix()
     }
 
     init?(keystoneSignature signature: String, isLegacyTx: Bool, chainId: String) {