@onekeyfe/react-native-keychain-module 1.1.19 → 1.1.20

package/KeychainModule.podspec

@@ -21,6 +21,7 @@ Pod::Spec.new do |s|
 
   s.dependency 'React-jsi'
   s.dependency 'React-callinvoker'
+  s.dependency 'ReactNativeNativeLogger'
 
   load 'nitrogen/generated/ios/KeychainModule+autolinking.rb'
   add_nitrogen_files(s)

package/ios/KeychainModule.swift

@@ -1,13 +1,12 @@
 import NitroModules
 
 class KeychainModule: HybridKeychainModuleSpec {
-    
+
     private let moduleCore = KeychainModuleCore()
-    
+
     public func setItem(params: SetItemParams) throws -> Promise<Void> {
-          let typedParams = params
           do {
-            try moduleCore.setItem(params: typedParams)
+            try moduleCore.setItem(params: params)
             return Promise.resolved(withResult: Void())
           } catch let error as KeychainModuleError {
             switch error {
@@ -22,11 +21,10 @@ class KeychainModule: HybridKeychainModuleSpec {
             return Promise.rejected(withError: NSError(domain: "keychain_set_error", code: -1000, userInfo: [NSLocalizedDescriptionKey: "Failed to set keychain item", NSUnderlyingErrorKey: error]))
           }
     }
-    
+
   public func getItem(params : GetItemParams) throws -> Promise<Variant_NullType_GetItemResult> {
-        let typedParams = params
         do {
-            if let result = try moduleCore.getItem(params: typedParams) {
+            if let result = try moduleCore.getItem(params: params) {
                 return Promise.resolved(withResult: Variant_NullType_GetItemResult.second(result))
             } else {
                 return Promise.resolved(withResult: Variant_NullType_GetItemResult.first(NullType.null))
@@ -42,11 +40,10 @@ class KeychainModule: HybridKeychainModuleSpec {
             return Promise.rejected(withError: NSError(domain: "keychain_get_error", code: -1000, userInfo: [NSLocalizedDescriptionKey: "Failed to get keychain item", NSUnderlyingErrorKey: error]))
         }
     }
-    
+
     public func removeItem(params: RemoveItemParams) throws -> Promise<Void> {
-        let typedParams = params
         do {
-            try moduleCore.removeItem(params: typedParams)
+            try moduleCore.removeItem(params: params)
             return Promise.resolved(withResult: Void())
         } catch let error as KeychainModuleError {
         switch error {

package/ios/KeychainModuleCore.swift

@@ -8,6 +8,7 @@
 
 import Foundation
 import Security
+import ReactNativeNativeLogger
 
 // MARK: - Constants
 
@@ -33,6 +34,7 @@ class KeychainModuleCore {
 
   func setItem(params: SetItemParams) throws {
     guard let valueData = params.value.data(using: .utf8) else {
+      OneKeyLog.error("Keychain", "setItem: failed to encode value")
       throw KeychainModuleError.encodingFailed
     }
 
@@ -64,8 +66,10 @@ class KeychainModuleCore {
     let status = SecItemAdd(query as CFDictionary, nil)
 
     guard status == errSecSuccess else {
+      OneKeyLog.error("Keychain", "setItem: failed, OSStatus: \(status)")
       throw KeychainModuleError.operationFailed(status)
     }
+    OneKeyLog.info("Keychain", "setItem: success")
   }
 
   // MARK: - Get Item
@@ -86,12 +90,15 @@ class KeychainModuleCore {
     if status == errSecSuccess {
       if let valueData = result as? Data,
          let value = String(data: valueData, encoding: .utf8) {
+        OneKeyLog.debug("Keychain", "getItem: found")
         return GetItemResult(key: params.key, value: value)
       }
       return nil
     } else if status == errSecItemNotFound {
+      OneKeyLog.debug("Keychain", "getItem: not found")
       return nil
     } else {
+      OneKeyLog.error("Keychain", "getItem: failed, OSStatus: \(status)")
       throw KeychainModuleError.operationFailed(status)
     }
   }
@@ -110,8 +117,10 @@ class KeychainModuleCore {
 
     // Both success and item not found are acceptable for delete
     guard status == errSecSuccess || status == errSecItemNotFound else {
+      OneKeyLog.error("Keychain", "removeItem: failed, OSStatus: \(status)")
       throw KeychainModuleError.operationFailed(status)
     }
+    OneKeyLog.info("Keychain", "removeItem: success")
   }
 
   // MARK: - Check Item Existence
@@ -126,6 +135,7 @@ class KeychainModuleCore {
     ]
 
     let status = SecItemCopyMatching(query as CFDictionary, nil)
+    OneKeyLog.debug("Keychain", "hasItem: \(status == errSecSuccess)")
     return status == errSecSuccess
   }
 
@@ -169,6 +179,8 @@ class KeychainModuleCore {
     // Common error codes:
     // errSecMissingEntitlement (-34018): Missing iCloud Keychain entitlement
     // errSecNotAvailable (-25291): iCloud Keychain not available/signed out
-    return addStatus == errSecSuccess
+    let enabled = addStatus == errSecSuccess
+    OneKeyLog.info("Keychain", "iCloud sync check result: \(enabled)")
+    return enabled
   }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@onekeyfe/react-native-keychain-module",
-  "version": "1.1.19",
+  "version": "1.1.20",
   "description": "react-native-keychain-module",
   "main": "./lib/module/index.js",
   "types": "./lib/typescript/src/index.d.ts",