@onekeyfe/react-native-bundle-update 3.0.28 → 3.0.29

package/android/src/main/java/com/margelo/nitro/reactnativebundleupdate/ReactNativeBundleUpdate.kt

@@ -404,7 +404,31 @@ object BundleUpdateStoreAndroid {
         }
     }
 
+    private val lastSHA256Failure = ThreadLocal<String?>()
+
+    /**
+     * Subtype of the most recent calculateSHA256 failure on this thread, or
+     * null if the last call succeeded. Surfaces the specific reason —
+     * FILE_NOT_FOUND / FILE_EMPTY / FILE_TRUNCATED / OOM / IO_<class> /
+     * UNEXPECTED_<class> — so analytics can split the previously opaque
+     * "Failed to calculate SHA256" bucket (mixpanel: 91.3 percent of
+     * verifyPackage failures) into actionable categories.
+     */
+    fun lastSHA256FailureReason(): String? = lastSHA256Failure.get()
+
     fun calculateSHA256(filePath: String): String? {
+        lastSHA256Failure.set(null)
+        val file = File(filePath)
+        if (!file.exists()) {
+            lastSHA256Failure.set("FILE_NOT_FOUND")
+            OneKeyLog.error("BundleUpdate", "calculateSHA256: file not found: $filePath")
+            return null
+        }
+        if (file.length() == 0L) {
+            lastSHA256Failure.set("FILE_EMPTY")
+            OneKeyLog.error("BundleUpdate", "calculateSHA256: file empty: $filePath")
+            return null
+        }
         return try {
             val digest = MessageDigest.getInstance("SHA-256")
             BufferedInputStream(FileInputStream(filePath)).use { bis ->
@@ -415,8 +439,29 @@ object BundleUpdateStoreAndroid {
                 }
             }
             bytesToHex(digest.digest())
+        } catch (e: java.io.FileNotFoundException) {
+            lastSHA256Failure.set("FILE_DISAPPEARED")
+            OneKeyLog.error("BundleUpdate", "calculateSHA256: file disappeared during read: ${e.message}")
+            null
+        } catch (e: java.io.EOFException) {
+            lastSHA256Failure.set("FILE_TRUNCATED")
+            OneKeyLog.error("BundleUpdate", "calculateSHA256: truncated file: ${e.message}")
+            null
+        } catch (e: SecurityException) {
+            lastSHA256Failure.set("PERMISSION_DENIED")
+            OneKeyLog.error("BundleUpdate", "calculateSHA256: permission denied: ${e.message}")
+            null
+        } catch (e: OutOfMemoryError) {
+            lastSHA256Failure.set("OOM")
+            OneKeyLog.error("BundleUpdate", "calculateSHA256: OutOfMemoryError on ${file.length()} bytes")
+            null
+        } catch (e: java.io.IOException) {
+            lastSHA256Failure.set("IO_${e.javaClass.simpleName}")
+            OneKeyLog.error("BundleUpdate", "calculateSHA256: ${e.javaClass.simpleName}: ${e.message}")
+            null
         } catch (e: Exception) {
-            OneKeyLog.error("BundleUpdate", "Error calculating SHA256: ${e.message}")
+            lastSHA256Failure.set("UNEXPECTED_${e.javaClass.simpleName}")
+            OneKeyLog.error("BundleUpdate", "calculateSHA256: ${e.javaClass.simpleName}: ${e.message}")
             null
         }
     }
@@ -455,8 +500,13 @@ object BundleUpdateStoreAndroid {
                 }
             }
         } catch (e: Exception) {
-            OneKeyLog.error("BundleUpdate", "Error parsing metadata JSON: ${e.message}")
-            throw Exception("Failed to parse metadata.json: ${e.message}")
+            // org.json's exception messages occasionally embed file paths or
+            // partial JSON content. Keep the rich detail in OneKeyLog (local
+            // only), but throw a class-tag-only message so the JS analytics
+            // layer cannot reflect arbitrary inner content. Mirrors the
+            // SHA256_<reason>/IO_<class> convention used elsewhere.
+            OneKeyLog.error("BundleUpdate", "Error parsing metadata JSON: ${e.javaClass.simpleName}: ${e.message}")
+            throw Exception("Failed to parse metadata.json: IO_${e.javaClass.simpleName}")
         }
         if (metadata.isEmpty()) {
             throw Exception("metadata.json is empty or contains no file entries")
@@ -1201,6 +1251,35 @@ class ReactNativeBundleUpdate : HybridReactNativeBundleUpdateSpec() {
         }
     }
 
+    /**
+     * Returns a low-cardinality, path-free tag describing why a download
+     * failed. Used in the JS-facing `update/error` event so listeners
+     * never observe Android's `/data/data/<pkg>/...` or `/data/user/<u>/<pkg>/...`
+     * paths that FileNotFoundException / IOException embed in `e.message`.
+     *
+     * Recognized payloads (same shape extractUpdateErrorCode parses):
+     *   - "Bundle SHA256 verification failed: <REASON>" → preserved verbatim;
+     *      JS extractor maps to SHA256_<REASON>.
+     *   - "HTTP <code>" / "HTTP 416 ..." → preserved verbatim; maps to HTTP_<code>.
+     *   - "Already downloading" / "Invalid version string format" /
+     *     "Bundle download URL must use HTTPS" → preserved verbatim; the
+     *     hooks unrecoverable-list matches them by exact substring.
+     *   - Anything else → "IO_<exceptionClassName>" so the JS extractor
+     *     splits the bucket on exception class without leaking the message.
+     */
+    private fun sanitizeErrorMessageForEvent(e: Exception): String {
+        val msg = e.message ?: return "IO_${e.javaClass.simpleName}"
+        if (msg.startsWith("Bundle SHA256 verification failed:")) return msg
+        if (msg.startsWith("HTTP ")) return msg
+        if (msg == "Already downloading" ||
+            msg == "Invalid version string format" ||
+            msg == "Bundle download URL must use HTTPS" ||
+            msg == "Empty response body" ||
+            msg == "Failed to finalize download"
+        ) return msg
+        return "IO_${e.javaClass.simpleName}"
+    }
+
     override fun addDownloadListener(callback: (BundleDownloadEvent) -> Unit): Double {
         val id = nextListenerId.getAndIncrement().toDouble()
         listeners.add(BundleListener(id, callback))
@@ -1274,6 +1353,11 @@ class ReactNativeBundleUpdate : HybridReactNativeBundleUpdateSpec() {
 
             val fileName = "$appVersion-$bundleVersion.zip"
             val filePath = File(BundleUpdateStoreAndroid.getDownloadBundleDir(context), fileName).absolutePath
+            // Resume support: download to <filename>.partial; rename to <filename>
+            // only after the full transfer + SHA256 verify pass. Mirrors the
+            // Desktop convention so a corrupt completion can never poison the
+            // "exists at filePath -> already valid" cache check above.
+            val partialFilePath = "$filePath.partial"
 
             val result = BundleDownloadResult(
                 downloadedFile = filePath,
@@ -1286,6 +1370,7 @@ class ReactNativeBundleUpdate : HybridReactNativeBundleUpdateSpec() {
             OneKeyLog.info("BundleUpdate", "downloadBundle: filePath=$filePath")
 
             val downloadedFile = File(filePath)
+            val partialFile = File(partialFilePath)
             if (downloadedFile.exists()) {
                 OneKeyLog.info("BundleUpdate", "downloadBundle: file already exists, verifying SHA256...")
                 if (verifyBundleSHA256(filePath, sha256)) {
@@ -1297,47 +1382,147 @@ class ReactNativeBundleUpdate : HybridReactNativeBundleUpdateSpec() {
                 } else {
                     OneKeyLog.warn("BundleUpdate", "downloadBundle: existing file SHA256 mismatch, re-downloading")
                     downloadedFile.delete()
+                    // Stale completed file invalidates any partial too.
+                    if (partialFile.exists()) partialFile.delete()
+                }
+            }
+
+            // Resume: if a partial from a previous run exists and is smaller
+            // than the expected size, send `Range: bytes=<offset>-` so the
+            // server fills in the rest. If the partial is exactly the
+            // expected size it's a process-killed-just-before-rename case
+            // (full body on disk but never promoted): try SHA verify
+            // before discarding so we save a full re-download.
+            val expectedSize = if (params.fileSize > 0) params.fileSize.toLong() else 0L
+            var partialBytes = 0L
+            if (partialFile.exists()) {
+                val partialSize = partialFile.length()
+                when {
+                    expectedSize > 0 && partialSize == expectedSize -> {
+                        OneKeyLog.info("BundleUpdate", "downloadBundle: partial matches expected size ($partialSize), trying promote+verify")
+                        if (downloadedFile.exists()) downloadedFile.delete()
+                        if (partialFile.renameTo(downloadedFile) && verifyBundleSHA256(filePath, sha256)) {
+                            OneKeyLog.info("BundleUpdate", "downloadBundle: recovered crashed-before-rename bundle, skipping download")
+                            isDownloading.set(false)
+                            Thread.sleep(1000)
+                            sendEvent("update/complete")
+                            return@async result
+                        } else {
+                            OneKeyLog.warn("BundleUpdate", "downloadBundle: promote+verify failed, discarding both files")
+                            if (downloadedFile.exists()) downloadedFile.delete()
+                            // partialFile is gone (renamed); nothing to delete
+                        }
+                    }
+                    expectedSize > 0 && partialSize > expectedSize -> {
+                        OneKeyLog.warn("BundleUpdate", "downloadBundle: stale partial (>expected), discarding: $partialSize/$expectedSize")
+                        partialFile.delete()
+                    }
+                    partialSize > 0 -> {
+                        partialBytes = partialSize
+                        OneKeyLog.info("BundleUpdate", "downloadBundle: resuming from $partialBytes bytes (expected=$expectedSize)")
+                    }
+                    else -> partialFile.delete()
                 }
             }
 
             sendEvent("update/start")
-            OneKeyLog.info("BundleUpdate", "downloadBundle: starting download...")
+            OneKeyLog.info("BundleUpdate", "downloadBundle: starting download (resume=${partialBytes > 0})...")
+
+            val requestBuilder = Request.Builder().url(downloadUrl)
+            if (partialBytes > 0) {
+                requestBuilder.addHeader("Range", "bytes=$partialBytes-")
+            }
+            val response = httpClient.newCall(requestBuilder.build()).execute()
+
+            // 416 Range Not Satisfiable: server says our offset is past the
+            // file length. Two sub-cases distinguishable from
+            // `Content-Range: bytes */<total>`:
+            //   (a) total == partialBytes → file is exactly complete on
+            //       server; our partial is the whole bundle and just
+            //       needs SHA verify + rename. Recover instead of wipe.
+            //   (b) anything else → partial is corrupt or bundle changed.
+            //       Wipe and bubble up.
+            if (response.code == 416) {
+                val contentRange = response.header("Content-Range")
+                response.close()
+                val totalFromHeader = contentRange
+                    ?.let { Regex("""bytes\s+\*\s*/\s*(\d+)""").find(it)?.groupValues?.getOrNull(1)?.toLongOrNull() }
+                if (totalFromHeader != null && totalFromHeader == partialBytes && partialFile.exists()) {
+                    OneKeyLog.info("BundleUpdate", "downloadBundle: HTTP 416 with total==$totalFromHeader matches partial, attempting promote+verify")
+                    if (downloadedFile.exists()) downloadedFile.delete()
+                    if (partialFile.renameTo(downloadedFile) && verifyBundleSHA256(filePath, sha256)) {
+                        OneKeyLog.info("BundleUpdate", "downloadBundle: 416 recovery succeeded, skipping download")
+                        sendEvent("update/complete")
+                        return@async result
+                    }
+                    OneKeyLog.warn("BundleUpdate", "downloadBundle: 416 recovery failed verify, discarding")
+                    if (downloadedFile.exists()) downloadedFile.delete()
+                }
+                OneKeyLog.warn("BundleUpdate", "downloadBundle: HTTP 416 (range not satisfiable), discarding partial and failing this attempt")
+                if (partialFile.exists()) partialFile.delete()
+                sendEvent("update/error", message = "HTTP 416 (range not satisfiable)")
+                throw Exception("HTTP 416")
+            }
 
-            val request = Request.Builder().url(downloadUrl).build()
-            val response = httpClient.newCall(request).execute()
+            val expectsResume = partialBytes > 0
+            val isPartialResponse = response.code == 206
 
-            if (!response.isSuccessful) {
+            if (!response.isSuccessful || (response.code != 200 && response.code != 206)) {
                 OneKeyLog.error("BundleUpdate", "downloadBundle: HTTP error, statusCode=${response.code}")
+                response.close()
                 sendEvent("update/error", message = "HTTP ${response.code}")
                 throw Exception("HTTP ${response.code}")
             }
 
+            // Server can ignore `Range` and return the full body with 200; in
+            // that case our partial is meaningless — restart fresh.
+            if (expectsResume && !isPartialResponse) {
+                OneKeyLog.warn("BundleUpdate", "downloadBundle: requested Range but server returned 200, restarting from scratch")
+                if (partialFile.exists()) partialFile.delete()
+                partialBytes = 0L
+            }
+
             val body = response.body ?: throw Exception("Empty response body")
-            val fileSize = if (params.fileSize > 0) params.fileSize.toLong() else body.contentLength()
-            OneKeyLog.info("BundleUpdate", "downloadBundle: HTTP 200, contentLength=$fileSize, downloading...")
+            val contentLength = body.contentLength()
+            // Total size of the whole resource (not the slice). On 206 prefer
+            // Content-Range's "/total" tail; fall back to partial+contentLength.
+            val totalSize: Long = if (isPartialResponse) {
+                val contentRange = response.header("Content-Range")
+                val parsedTotal = contentRange
+                    ?.let { Regex("""bytes \d+-\d+/(\d+)""").find(it)?.groupValues?.getOrNull(1)?.toLongOrNull() }
+                parsedTotal ?: (partialBytes + contentLength.coerceAtLeast(0L))
+            } else {
+                if (contentLength > 0) contentLength else expectedSize
+            }
+            OneKeyLog.info(
+                "BundleUpdate",
+                "downloadBundle: HTTP ${response.code}, contentLength=$contentLength, totalSize=$totalSize, partialBytes=$partialBytes, downloading..."
+            )
 
             // Ensure parent directory exists before writing
-            val parentDir = File(filePath).parentFile
+            val parentDir = File(partialFilePath).parentFile
             if (parentDir != null && !parentDir.exists()) {
                 parentDir.mkdirs()
                 OneKeyLog.info("BundleUpdate", "downloadBundle: created parent directory: ${parentDir.absolutePath}")
             }
 
-            var totalBytesRead = 0L
+            // Append iff server granted us a 206 partial; otherwise overwrite.
+            val appendMode = isPartialResponse
+            var totalBytesRead = if (isPartialResponse) partialBytes else 0L
             body.byteStream().use { inputStream ->
-                FileOutputStream(filePath).use { outputStream ->
+                FileOutputStream(partialFilePath, appendMode).use { outputStream ->
                     val buffer = ByteArray(8192)
                     var bytesRead: Int
 
-                    var prevProgress = 0
+                    var prevProgress = -1
                     while (inputStream.read(buffer).also { bytesRead = it } != -1) {
                         outputStream.write(buffer, 0, bytesRead)
                         totalBytesRead += bytesRead
-                        if (fileSize > 0) {
-                            val progress = ((totalBytesRead * 100) / fileSize).toInt()
+                        if (totalSize > 0) {
+                            val progress = ((totalBytesRead * 100) / totalSize).toInt().coerceIn(0, 100)
                             if (progress != prevProgress) {
                                 sendEvent("update/downloading", progress = progress)
-                                OneKeyLog.info("BundleUpdate", "download progress: $progress% ($totalBytesRead/$fileSize)")
+                                OneKeyLog.info("BundleUpdate", "download progress: $progress% ($totalBytesRead/$totalSize)")
                                 prevProgress = progress
                             }
                         }
@@ -1345,21 +1530,45 @@ class ReactNativeBundleUpdate : HybridReactNativeBundleUpdateSpec() {
                 }
             }
 
-            val downloadedFileAfter = File(filePath)
-            OneKeyLog.info("BundleUpdate", "downloadBundle: download finished, totalBytesRead=$totalBytesRead, fileExists=${downloadedFileAfter.exists()}, fileSize=${if (downloadedFileAfter.exists()) downloadedFileAfter.length() else -1}, verifying SHA256...")
+            val partialAfter = File(partialFilePath)
+            OneKeyLog.info(
+                "BundleUpdate",
+                "downloadBundle: download finished, totalBytesRead=$totalBytesRead, partialExists=${partialAfter.exists()}, partialSize=${if (partialAfter.exists()) partialAfter.length() else -1}, finalizing..."
+            )
+
+            // Promote .partial to final ONLY after the full transfer; renaming
+            // first means a SHA256 mismatch leaves no half-baked filePath that
+            // the next call would mistake for a cached good bundle.
+            if (downloadedFile.exists()) downloadedFile.delete()
+            if (!partialAfter.renameTo(downloadedFile)) {
+                OneKeyLog.error("BundleUpdate", "downloadBundle: rename .partial -> final failed")
+                sendEvent("update/error", message = "rename .partial failed")
+                throw Exception("Failed to finalize download")
+            }
+
+            OneKeyLog.info("BundleUpdate", "downloadBundle: verifying SHA256...")
             if (!verifyBundleSHA256(filePath, sha256)) {
+                val reason = BundleUpdateStoreAndroid.lastSHA256FailureReason() ?: "MISMATCH"
                 File(filePath).delete()
-                OneKeyLog.error("BundleUpdate", "downloadBundle: SHA256 verification failed after download")
-                sendEvent("update/error", message = "Bundle signature verification failed")
-                throw Exception("Bundle signature verification failed")
+                OneKeyLog.error("BundleUpdate", "downloadBundle: SHA256 verification failed after download, reason=$reason")
+                sendEvent("update/error", message = "SHA256_$reason")
+                throw Exception("Bundle SHA256 verification failed: $reason")
             }
 
             sendEvent("update/complete")
             OneKeyLog.info("BundleUpdate", "downloadBundle: completed successfully, appVersion=$appVersion, bundleVersion=$bundleVersion")
             result
             } catch (e: Exception) {
+                // Keep the rich detail in OneKeyLog (local-only). The JS
+                // event channel must NOT carry e.message verbatim — Android
+                // FileNotFoundException etc. embed the full /data/user/.../
+                // path including the package identifier, and downstream
+                // listeners would forward that into analytics. Emit only a
+                // domain+code shape; mirrors the iOS sendEvent payload at
+                // ReactNativeBundleUpdate.swift's "update/error" sites.
                 OneKeyLog.error("BundleUpdate", "downloadBundle: failed: ${e.javaClass.simpleName}: ${e.message}")
-                sendEvent("update/error", message = "${e.javaClass.simpleName}: ${e.message}")
+                val codeTag = sanitizeErrorMessageForEvent(e)
+                sendEvent("update/error", message = "${e.javaClass.simpleName}: $codeTag")
                 throw e
             } finally {
                 isDownloading.set(false)
@@ -1367,10 +1576,18 @@ class ReactNativeBundleUpdate : HybridReactNativeBundleUpdateSpec() {
         }
     }
 
+    /**
+     * Returns true on hash match. On false, callers may inspect
+     * BundleUpdateStoreAndroid.lastSHA256FailureReason() to distinguish a
+     * computation failure (FILE_TRUNCATED / OOM / IO_*) from a clean hash
+     * mismatch (reason == null).
+     */
     private fun verifyBundleSHA256(bundlePath: String, sha256: String): Boolean {
         val calculated = BundleUpdateStoreAndroid.calculateSHA256(bundlePath)
         if (calculated == null) {
-            OneKeyLog.error("BundleUpdate", "verifyBundleSHA256: failed to calculate SHA256 for: $bundlePath")
+            val reason = BundleUpdateStoreAndroid.lastSHA256FailureReason() ?: "UNKNOWN"
+            val fileSize = try { File(bundlePath).length() } catch (_: Exception) { -1L }
+            OneKeyLog.error("BundleUpdate", "verifyBundleSHA256: failed to calculate SHA256 for: $bundlePath, reason=$reason, fileSize=$fileSize")
             return false
         }
         val isValid = BundleUpdateStoreAndroid.secureCompare(calculated, sha256)
@@ -1412,8 +1629,13 @@ class ReactNativeBundleUpdate : HybridReactNativeBundleUpdateSpec() {
             if (!skipGPG) {
                 OneKeyLog.info("BundleUpdate", "verifyBundleASC: verifying SHA256 of downloaded file...")
                 if (!verifyBundleSHA256(filePath, sha256)) {
-                    OneKeyLog.error("BundleUpdate", "verifyBundleASC: SHA256 verification failed for file=$filePath")
-                    throw Exception("Bundle signature verification failed")
+                    // Promote the SHA256 subtype (FILE_TRUNCATED / OOM /
+                    // IO_<class> / MISMATCH) into the thrown message so
+                    // extractUpdateErrorCode in the JS layer can split
+                    // this bucket the same way the download stage does.
+                    val reason = BundleUpdateStoreAndroid.lastSHA256FailureReason() ?: "MISMATCH"
+                    OneKeyLog.error("BundleUpdate", "verifyBundleASC: SHA256 verification failed for file=$filePath, reason=$reason")
+                    throw Exception("Bundle SHA256 verification failed: $reason")
                 }
                 OneKeyLog.info("BundleUpdate", "verifyBundleASC: SHA256 verified OK")
             } else {

package/ios/ReactNativeBundleUpdate.swift

@@ -5,6 +5,7 @@ import CommonCrypto
 import Gopenpgp
 import SSZipArchive
 import MMKV
+import UIKit
 
 // OneKey GPG public key for signature verification
 private let GPG_PUBLIC_KEY = """
@@ -312,23 +313,91 @@ public class BundleUpdateStore: NSObject {
         return true
     }
 
+    /// Subtype of the most recent calculateSHA256 failure on this thread, or
+    /// nil if the last call succeeded. Surfaces FILE_NOT_FOUND / FILE_EMPTY /
+    /// FILE_DISAPPEARED / IO_<NSError code> / UNEXPECTED so analytics can
+    /// split the previously opaque "Failed to calculate SHA256" bucket
+    /// (mixpanel: 91.3% of verifyPackage failures are Android; iOS shares
+    /// the calculator and inherits the same blind spot for its 14 ASC +
+    /// 2 verifyPackage Promise-destroyed cases).
+    public static func lastSHA256FailureReason() -> String? {
+        return Thread.current.threadDictionary[kSHA256FailureKey] as? String
+    }
+    private static let kSHA256FailureKey = "so.onekey.bundleupdate.sha256.failure"
+    private static func setSHA256Failure(_ reason: String?) {
+        if let reason = reason {
+            Thread.current.threadDictionary[kSHA256FailureKey] = reason
+        } else {
+            Thread.current.threadDictionary.removeObject(forKey: kSHA256FailureKey)
+        }
+    }
+
     public static func calculateSHA256(_ filePath: String) -> String? {
-        guard let fileHandle = FileHandle(forReadingAtPath: filePath) else { return nil }
+        setSHA256Failure(nil)
+        let fm = FileManager.default
+        if !fm.fileExists(atPath: filePath) {
+            setSHA256Failure("FILE_NOT_FOUND")
+            OneKeyLog.error("BundleUpdate", "calculateSHA256: file not found: \(filePath)")
+            return nil
+        }
+        let attrs = (try? fm.attributesOfItem(atPath: filePath)) ?? [:]
+        let fileSize = (attrs[.size] as? NSNumber)?.int64Value ?? -1
+        if fileSize == 0 {
+            setSHA256Failure("FILE_EMPTY")
+            OneKeyLog.error("BundleUpdate", "calculateSHA256: file empty: \(filePath)")
+            return nil
+        }
+        guard let fileHandle = FileHandle(forReadingAtPath: filePath) else {
+            setSHA256Failure("FILE_DISAPPEARED")
+            OneKeyLog.error("BundleUpdate", "calculateSHA256: open failed (file disappeared between stat and open): \(filePath)")
+            return nil
+        }
         defer { fileHandle.closeFile() }
 
-        var context = CC_SHA256_CTX()
-        CC_SHA256_Init(&context)
-        while autoreleasepool(invoking: {
-            let data = fileHandle.readData(ofLength: 8192)
-            if data.count > 0 {
-                data.withUnsafeBytes { CC_SHA256_Update(&context, $0.baseAddress, CC_LONG(data.count)) }
-                return true
+        do {
+            var context = CC_SHA256_CTX()
+            CC_SHA256_Init(&context)
+            var threwError: Error?
+            // Wrap reads in try/catch via NSException bridge: FileHandle.readData
+            // can raise on read failure (NSFileHandleOperationException);
+            // ObjCRuntime catches those when bridged through NSObject methods.
+            while autoreleasepool(invoking: {
+                do {
+                    let data = try Self.safeRead(fileHandle: fileHandle, length: 8192)
+                    if data.count > 0 {
+                        data.withUnsafeBytes { CC_SHA256_Update(&context, $0.baseAddress, CC_LONG(data.count)) }
+                        return true
+                    }
+                    return false
+                } catch {
+                    threwError = error
+                    return false
+                }
+            }) {}
+            if let err = threwError {
+                let nsErr = err as NSError
+                setSHA256Failure("IO_\(nsErr.domain)_\(nsErr.code)")
+                OneKeyLog.error("BundleUpdate", "calculateSHA256: read failed: \(nsErr.domain) \(nsErr.code) \(nsErr.localizedDescription)")
+                return nil
             }
-            return false
-        }) {}
-        var hash = [UInt8](repeating: 0, count: Int(CC_SHA256_DIGEST_LENGTH))
-        CC_SHA256_Final(&hash, &context)
-        return hash.map { String(format: "%02x", $0) }.joined()
+            var hash = [UInt8](repeating: 0, count: Int(CC_SHA256_DIGEST_LENGTH))
+            CC_SHA256_Final(&hash, &context)
+            return hash.map { String(format: "%02x", $0) }.joined()
+        }
+    }
+
+    /// Raises throwing wrapper around FileHandle.read(upToCount:) so disk
+    /// I/O failures (truncated file, unmounted volume) become catchable Swift
+    /// errors rather than NSFileHandleOperationException.
+    private static func safeRead(fileHandle: FileHandle, length: Int) throws -> Data {
+        if #available(iOS 13.4, macOS 10.15.4, *) {
+            return try fileHandle.read(upToCount: length) ?? Data()
+        } else {
+            // Pre-iOS 13.4 fallback: classic readData(ofLength:) does not
+            // throw, but raises NSException; we cannot bridge that here so
+            // accept the legacy behavior on these old OS versions only.
+            return fileHandle.readData(ofLength: length)
+        }
     }
 
     public static func getNativeVersion() -> String? {
@@ -1036,6 +1105,10 @@ private class DownloadDelegate: NSObject, URLSessionDownloadDelegate {
     /// Continuation to bridge delegate callbacks → async/await
     private var continuation: CheckedContinuation<(URL, URLResponse), Error>?
     private var tempFileURL: URL?
+    /// Resume data captured from the most recent failure so the caller can
+    /// persist it for the next attempt. Populated in didCompleteWithError
+    /// when the system supplies NSURLSessionDownloadTaskResumeData.
+    private(set) var lastResumeData: Data?
     private let lock = NSLock()
 
     func setContinuation(_ cont: CheckedContinuation<(URL, URLResponse), Error>) {
@@ -1079,10 +1152,23 @@ private class DownloadDelegate: NSObject, URLSessionDownloadDelegate {
         lock.unlock()
 
         if let error = error {
+            // iOS attaches partial download bytes via userInfo so the next
+            // attempt can finish from the cut point instead of byte 0. Mixpanel
+            // shows ~5,940 of our failures (NSURL -1005 / -1001) carry ~11KB of
+            // resume data each — previously discarded.
+            let nsError = error as NSError
+            if let resumeData = nsError.userInfo[NSURLSessionDownloadTaskResumeData] as? Data, resumeData.count > 0 {
+                self.lastResumeData = resumeData
+                OneKeyLog.info("BundleUpdate", "download error captured resumeData: \(resumeData.count) bytes")
+            } else {
+                self.lastResumeData = nil
+            }
             cont?.resume(throwing: error)
         } else if let tempURL = tempFileURL, let response = task.response {
+            self.lastResumeData = nil
             cont?.resume(returning: (tempURL, response))
         } else {
+            self.lastResumeData = nil
             cont?.resume(throwing: NSError(domain: "BundleUpdate", code: -1,
                 userInfo: [NSLocalizedDescriptionKey: "Download completed without file"]))
         }
@@ -1105,6 +1191,7 @@ private class DownloadDelegate: NSObject, URLSessionDownloadDelegate {
         tempFileURL = nil
         onProgress = nil
         prevProgress = -1
+        lastResumeData = nil
     }
 }
 
@@ -1134,9 +1221,109 @@ class ReactNativeBundleUpdate: HybridReactNativeBundleUpdateSpec {
         return URLSession(configuration: config, delegate: delegate, delegateQueue: nil)
     }
 
+    /// Path of the bundle being downloaded right now (mirrors filePath in
+    /// downloadBundle). Set on entry, cleared on exit. Single value because
+    /// isDownloading enforces at most one in-flight download per process.
+    /// Read by the background-snapshot handler so it knows where to drop the
+    /// `.resume` sidecar.
+    private var activeDownloadFilePath: String?
+    private var didEnterBackgroundObserver: NSObjectProtocol?
+
     override init() {
         super.init()
         urlSession = createURLSession()
+        registerBackgroundSnapshotObserver()
+    }
+
+    deinit {
+        if let token = didEnterBackgroundObserver {
+            NotificationCenter.default.removeObserver(token)
+        }
+        // URLSession retains its delegate strongly until invalidated.
+        // Without this call the session (and its DownloadDelegate) would
+        // leak past module deallocation — relevant in dev hot-reload and
+        // any future test harness that spins up multiple module instances.
+        urlSession?.invalidateAndCancel()
+    }
+
+    /// On iOS, force-quit (user swipes the app off the App Switcher) cannot
+    /// fire `URLSession`'s `didCompleteWithError` — SIGKILL leaves no time
+    /// for callbacks. The kill is, however, *always* preceded by the app
+    /// transitioning to the background. We hook that transition and call
+    /// `cancel(byProducingResumeData:)` synchronously, so the resume blob
+    /// is on disk before the app can possibly be terminated. Memory-pressure
+    /// kills follow the same chain (the OS only reaps backgrounded apps
+    /// under memory pressure), so this also covers OOM termination.
+    ///
+    /// `beginBackgroundTask` extends the ~5s of guaranteed background runtime
+    /// to ~30s, which is plenty for a few-KB blob write — but we don't rely
+    /// on the extension to *complete* the snapshot; the `cancel` callback
+    /// returns the data synchronously, so even if iOS suspends us right
+    /// after `endBackgroundTask` the resume blob is already persisted.
+    private func registerBackgroundSnapshotObserver() {
+        didEnterBackgroundObserver = NotificationCenter.default.addObserver(
+            forName: UIApplication.didEnterBackgroundNotification,
+            object: nil,
+            queue: .main
+        ) { [weak self] _ in
+            self?.snapshotResumeDataForBackgrounding()
+        }
+    }
+
+    private func snapshotResumeDataForBackgrounding() {
+        guard let session = self.urlSession else { return }
+        // Read both isDownloading AND activeDownloadFilePath inside the
+        // same stateQueue.sync block so a concurrent downloadBundle entry
+        // (which writes filePath BEFORE flipping isDownloading) cannot
+        // produce a torn read where isDownloading=true but filePath=nil.
+        let snapshot: (Bool, String?) = self.stateQueue.sync {
+            (self.isDownloading, self.activeDownloadFilePath)
+        }
+        let (stillDownloading, snapshotPath) = snapshot
+        guard stillDownloading, let filePath = snapshotPath else { return }
+
+        let resumeDataPath = "\(filePath).resume"
+        let bgTaskName = "BundleUpdateResumeSnapshot"
+        var bgTaskId: UIBackgroundTaskIdentifier = .invalid
+        bgTaskId = UIApplication.shared.beginBackgroundTask(withName: bgTaskName) {
+            // Expiration handler — system is reclaiming us. Best-effort end.
+            if bgTaskId != .invalid {
+                UIApplication.shared.endBackgroundTask(bgTaskId)
+                bgTaskId = .invalid
+            }
+        }
+        OneKeyLog.info("BundleUpdate", "didEnterBackground: snapshotting resumeData for \(filePath)")
+
+        session.getAllTasks { tasks in
+            let group = DispatchGroup()
+            for task in tasks {
+                guard let dl = task as? URLSessionDownloadTask, dl.state == .running else { continue }
+                group.enter()
+                dl.cancel(byProducingResumeData: { data in
+                    if let data = data, data.count > 0 {
+                        do {
+                            let dir = (resumeDataPath as NSString).deletingLastPathComponent
+                            if !FileManager.default.fileExists(atPath: dir) {
+                                try FileManager.default.createDirectory(atPath: dir, withIntermediateDirectories: true)
+                            }
+                            try data.write(to: URL(fileURLWithPath: resumeDataPath), options: .atomic)
+                            OneKeyLog.info("BundleUpdate", "didEnterBackground: persisted resumeData (\(data.count) bytes) at \(resumeDataPath)")
+                        } catch {
+                            OneKeyLog.warn("BundleUpdate", "didEnterBackground: failed to persist resumeData: \(error)")
+                        }
+                    } else {
+                        OneKeyLog.info("BundleUpdate", "didEnterBackground: cancel produced no resumeData (task may have just completed)")
+                    }
+                    group.leave()
+                })
+            }
+            group.notify(queue: .main) {
+                if bgTaskId != .invalid {
+                    UIApplication.shared.endBackgroundTask(bgTaskId)
+                    bgTaskId = .invalid
+                }
+            }
+        }
     }
 
     private func sendEvent(type: String, progress: Int = 0, message: String = "") {
@@ -1181,7 +1368,16 @@ class ReactNativeBundleUpdate: HybridReactNativeBundleUpdateSpec {
                 OneKeyLog.warn("BundleUpdate", "downloadBundle: rejected, already downloading")
                 throw NSError(domain: "BundleUpdate", code: -1, userInfo: [NSLocalizedDescriptionKey: "Already downloading"])
             }
-            defer { self.stateQueue.sync { self.isDownloading = false } }
+            defer {
+                // Clear isDownloading + the snapshot anchor under the same
+                // queue so didEnterBackground's paired read can't observe
+                // a half-cleared state (isDownloading=true while
+                // activeDownloadFilePath=nil, or vice versa).
+                self.stateQueue.sync {
+                    self.isDownloading = false
+                    self.activeDownloadFilePath = nil
+                }
+            }
 
             let appVersion = params.latestVersion
             let bundleVersion = params.bundleVersion
@@ -1202,6 +1398,10 @@ class ReactNativeBundleUpdate: HybridReactNativeBundleUpdateSpec {
 
             let fileName = "\(appVersion)-\(bundleVersion).zip"
             let filePath = (BundleUpdateStore.downloadBundleDir() as NSString).appendingPathComponent(fileName)
+            // Persisted resume blob from a previous failed attempt. Lives next
+            // to the bundle so it shares fate (delete-with-bundle) without
+            // polluting the bundle dir's cache lookup.
+            let resumeDataPath = "\(filePath).resume"
 
             let result = BundleDownloadResult(
                 downloadedFile: filePath,
@@ -1218,6 +1418,8 @@ class ReactNativeBundleUpdate: HybridReactNativeBundleUpdateSpec {
                 OneKeyLog.info("BundleUpdate", "downloadBundle: file already exists, verifying SHA256...")
                 if self.verifyBundleSHA256(filePath, sha256: sha256) {
                     OneKeyLog.info("BundleUpdate", "downloadBundle: existing file SHA256 valid, skipping download")
+                    // A valid completed bundle invalidates any stale resume blob.
+                    try? FileManager.default.removeItem(atPath: resumeDataPath)
                     DispatchQueue.main.asyncAfter(deadline: .now() + 1.0) { [weak self] in
                         self?.sendEvent(type: "update/complete")
                     }
@@ -1225,6 +1427,8 @@ class ReactNativeBundleUpdate: HybridReactNativeBundleUpdateSpec {
                 } else {
                     OneKeyLog.warn("BundleUpdate", "downloadBundle: existing file SHA256 mismatch, re-downloading")
                     try? FileManager.default.removeItem(atPath: filePath)
+                    // Hash-failed bundle means the resume blob is also poisoned.
+                    try? FileManager.default.removeItem(atPath: resumeDataPath)
                 }
             }
 
@@ -1239,8 +1443,17 @@ class ReactNativeBundleUpdate: HybridReactNativeBundleUpdateSpec {
                 throw NSError(domain: "BundleUpdate", code: -1, userInfo: [NSLocalizedDescriptionKey: "URLSession not initialized"])
             }
 
+            // Resume blob from a prior failed attempt; iOS rebuilds the byte
+            // offset internally so we never need a Range header on this path.
+            let persistedResumeData: Data? = {
+                guard FileManager.default.fileExists(atPath: resumeDataPath),
+                      let data = try? Data(contentsOf: URL(fileURLWithPath: resumeDataPath)),
+                      data.count > 0 else { return nil }
+                return data
+            }()
+
             self.sendEvent(type: "update/start")
-            OneKeyLog.info("BundleUpdate", "downloadBundle: starting download...")
+            OneKeyLog.info("BundleUpdate", "downloadBundle: starting download (resumeBytes=\(persistedResumeData?.count ?? 0))...")
 
             let request = URLRequest(url: url)
 
@@ -1254,51 +1467,103 @@ class ReactNativeBundleUpdate: HybridReactNativeBundleUpdateSpec {
                 self?.sendEvent(type: "update/downloading", progress: progress)
             }
 
-            let (tempURL, response) = try await withCheckedThrowingContinuation { (continuation: CheckedContinuation<(URL, URLResponse), Error>) in
-                delegate.setContinuation(continuation)
-                let task = session.downloadTask(with: request)
-                task.resume()
-            }
+            // Anchor for the background-snapshot handler. Set BEFORE
+            // task.resume() so a foreground→background transition that
+            // races the very first bytes still finds a path to write the
+            // resume blob to. Pair the write with the same stateQueue the
+            // snapshot reader uses, so isDownloading=true and a non-nil
+            // activeDownloadFilePath always go together.
+            self.stateQueue.sync { self.activeDownloadFilePath = filePath }
 
-            // Verify HTTPS was maintained (no HTTP redirect)
-            if let httpResponse = response as? HTTPURLResponse,
-               let responseUrl = httpResponse.url,
-               responseUrl.scheme?.lowercased() != "https" {
-                OneKeyLog.error("BundleUpdate", "downloadBundle: redirected to non-HTTPS URL: \(responseUrl)")
-                throw NSError(domain: "BundleUpdate", code: -1, userInfo: [NSLocalizedDescriptionKey: "Download was redirected to non-HTTPS URL"])
-            }
+            let downloadOutcome: Result<(URL, URLResponse), Error>
+            do {
+                let value = try await withCheckedThrowingContinuation { (continuation: CheckedContinuation<(URL, URLResponse), Error>) in
+                    delegate.setContinuation(continuation)
+                    let task: URLSessionDownloadTask
+                    if let resumeData = persistedResumeData {
+                        task = session.downloadTask(withResumeData: resumeData)
+                    } else {
+                        task = session.downloadTask(with: request)
+                    }
+                    task.resume()
+                }
+                downloadOutcome = .success(value)
+            } catch {
+                downloadOutcome = .failure(error)
+            }
+
+            switch downloadOutcome {
+            case .failure(let error):
+                // Persist the freshly captured resume blob for the next call.
+                // If iOS gave us nothing usable, clear any stale blob so a
+                // future attempt isn't held back by an unresumable cut point.
+                if let resumeData = delegate.lastResumeData {
+                    do {
+                        let dir = (resumeDataPath as NSString).deletingLastPathComponent
+                        if !FileManager.default.fileExists(atPath: dir) {
+                            try FileManager.default.createDirectory(atPath: dir, withIntermediateDirectories: true)
+                        }
+                        try resumeData.write(to: URL(fileURLWithPath: resumeDataPath), options: .atomic)
+                        OneKeyLog.info("BundleUpdate", "downloadBundle: persisted resumeData (\(resumeData.count) bytes) at \(resumeDataPath)")
+                    } catch {
+                        OneKeyLog.warn("BundleUpdate", "downloadBundle: failed to persist resumeData: \(error)")
+                    }
+                } else {
+                    try? FileManager.default.removeItem(atPath: resumeDataPath)
+                }
+                let nsError = error as NSError
+                OneKeyLog.error("BundleUpdate", "downloadBundle: download failed: \(nsError.domain) \(nsError.code) \(nsError.localizedDescription)")
+                self.sendEvent(type: "update/error", message: "\(nsError.domain) \(nsError.code)")
+                throw error
+
+            case .success(let (tempURL, response)):
+                // Successful completion ⇒ no longer need the resume blob.
+                try? FileManager.default.removeItem(atPath: resumeDataPath)
+
+                // Verify HTTPS was maintained (no HTTP redirect)
+                if let httpResponse = response as? HTTPURLResponse,
+                   let responseUrl = httpResponse.url,
+                   responseUrl.scheme?.lowercased() != "https" {
+                    OneKeyLog.error("BundleUpdate", "downloadBundle: redirected to non-HTTPS URL: \(responseUrl)")
+                    throw NSError(domain: "BundleUpdate", code: -1, userInfo: [NSLocalizedDescriptionKey: "Download was redirected to non-HTTPS URL"])
+                }
 
-            guard let httpResponse = response as? HTTPURLResponse, httpResponse.statusCode == 200 else {
-                let statusCode = (response as? HTTPURLResponse)?.statusCode ?? -1
-                OneKeyLog.error("BundleUpdate", "downloadBundle: HTTP error, statusCode=\(statusCode)")
-                self.sendEvent(type: "update/error", message: "HTTP error \(statusCode)")
-                throw NSError(domain: "BundleUpdate", code: -1, userInfo: [NSLocalizedDescriptionKey: "Download failed with HTTP \(statusCode)"])
-            }
+                // 206 is acceptable when the OS finished a Range-resumed task on
+                // our behalf; everything else non-200 is a real error.
+                guard let httpResponse = response as? HTTPURLResponse,
+                      httpResponse.statusCode == 200 || httpResponse.statusCode == 206 else {
+                    let statusCode = (response as? HTTPURLResponse)?.statusCode ?? -1
+                    OneKeyLog.error("BundleUpdate", "downloadBundle: HTTP error, statusCode=\(statusCode)")
+                    self.sendEvent(type: "update/error", message: "HTTP error \(statusCode)")
+                    throw NSError(domain: "BundleUpdate", code: -1, userInfo: [NSLocalizedDescriptionKey: "Download failed with HTTP \(statusCode)"])
+                }
 
-            OneKeyLog.info("BundleUpdate", "downloadBundle: download finished, HTTP 200, moving to destination...")
+                OneKeyLog.info("BundleUpdate", "downloadBundle: download finished, HTTP \(httpResponse.statusCode), moving to destination...")
 
-            // Move downloaded file to destination
-            let destDir = (filePath as NSString).deletingLastPathComponent
-            if !FileManager.default.fileExists(atPath: destDir) {
-                try FileManager.default.createDirectory(atPath: destDir, withIntermediateDirectories: true)
-            }
-            if FileManager.default.fileExists(atPath: filePath) {
-                try FileManager.default.removeItem(atPath: filePath)
-            }
-            try FileManager.default.moveItem(at: tempURL, to: URL(fileURLWithPath: filePath))
+                // Move downloaded file to destination
+                let destDir = (filePath as NSString).deletingLastPathComponent
+                if !FileManager.default.fileExists(atPath: destDir) {
+                    try FileManager.default.createDirectory(atPath: destDir, withIntermediateDirectories: true)
+                }
+                if FileManager.default.fileExists(atPath: filePath) {
+                    try FileManager.default.removeItem(atPath: filePath)
+                }
+                try FileManager.default.moveItem(at: tempURL, to: URL(fileURLWithPath: filePath))
 
-            // Verify SHA256
-            OneKeyLog.info("BundleUpdate", "downloadBundle: verifying SHA256...")
-            if !self.verifyBundleSHA256(filePath, sha256: sha256) {
-                try? FileManager.default.removeItem(atPath: filePath)
-                OneKeyLog.error("BundleUpdate", "downloadBundle: SHA256 verification failed after download")
-                self.sendEvent(type: "update/error", message: "Bundle signature verification failed")
-                throw NSError(domain: "BundleUpdate", code: -1, userInfo: [NSLocalizedDescriptionKey: "Bundle signature verification failed"])
-            }
+                // Verify SHA256
+                OneKeyLog.info("BundleUpdate", "downloadBundle: verifying SHA256...")
+                if !self.verifyBundleSHA256(filePath, sha256: sha256) {
+                    let reason = BundleUpdateStore.lastSHA256FailureReason() ?? "MISMATCH"
+                    try? FileManager.default.removeItem(atPath: filePath)
+                    OneKeyLog.error("BundleUpdate", "downloadBundle: SHA256 verification failed after download, reason=\(reason)")
+                    self.sendEvent(type: "update/error", message: "SHA256_\(reason)")
+                    throw NSError(domain: "BundleUpdate", code: -1, userInfo: [NSLocalizedDescriptionKey: "Bundle SHA256 verification failed: \(reason)"])
+                }
 
-            self.sendEvent(type: "update/complete")
-            OneKeyLog.info("BundleUpdate", "downloadBundle: completed successfully, appVersion=\(appVersion), bundleVersion=\(bundleVersion)")
-            return result
+                self.sendEvent(type: "update/complete")
+                OneKeyLog.info("BundleUpdate", "downloadBundle: completed successfully, appVersion=\(appVersion), bundleVersion=\(bundleVersion)")
+                return result
+            }
         }
     }
 
@@ -1347,10 +1612,16 @@ class ReactNativeBundleUpdate: HybridReactNativeBundleUpdateSpec {
 
             if !skipGPG {
                 OneKeyLog.info("BundleUpdate", "verifyBundleASC: verifying SHA256 of downloaded file...")
-                guard let calculated = BundleUpdateStore.calculateSHA256(filePath),
-                      calculated.secureCompare(sha256) else {
-                    OneKeyLog.error("BundleUpdate", "verifyBundleASC: SHA256 verification failed for file=\(filePath)")
-                    throw NSError(domain: "BundleUpdate", code: -1, userInfo: [NSLocalizedDescriptionKey: "Bundle signature verification failed"])
+                let calculated = BundleUpdateStore.calculateSHA256(filePath)
+                let isValid = calculated != nil && calculated!.secureCompare(sha256)
+                if !isValid {
+                    // Promote the SHA256 subtype (FILE_TRUNCATED / OOM /
+                    // IO_<class> / MISMATCH) into the thrown message so
+                    // analytics' extractUpdateErrorCode can split this
+                    // bucket the same way the download stage already does.
+                    let reason = BundleUpdateStore.lastSHA256FailureReason() ?? "MISMATCH"
+                    OneKeyLog.error("BundleUpdate", "verifyBundleASC: SHA256 verification failed for file=\(filePath), reason=\(reason)")
+                    throw NSError(domain: "BundleUpdate", code: -1, userInfo: [NSLocalizedDescriptionKey: "Bundle SHA256 verification failed: \(reason)"])
                 }
                 OneKeyLog.info("BundleUpdate", "verifyBundleASC: SHA256 verified OK")
             } else {
@@ -1377,9 +1648,19 @@ class ReactNativeBundleUpdate: HybridReactNativeBundleUpdateSpec {
                 try SSZipArchive.unzipFile(atPath: filePath, toDestination: destination, overwrite: true, password: nil)
                 OneKeyLog.info("BundleUpdate", "verifyBundleASC: extraction completed")
             } catch {
-                OneKeyLog.error("BundleUpdate", "verifyBundleASC: unzip failed: \(error.localizedDescription)")
+                // SSZipArchive's NSError.localizedDescription often embeds the
+                // failing file path which on iOS includes the install UUID
+                // (/var/mobile/Containers/Data/Application/<UUID>/...). Keep
+                // the rich detail in OneKeyLog (local-only), but expose only
+                // a code-shaped tag in the thrown message so the JS-side
+                // analytics layer can never reflect that path back to the
+                // server. Subtypes intentionally low-cardinality so
+                // extractUpdateErrorCode picks them up cleanly:
+                //   IO_<NSError code>
+                let nsErr = error as NSError
+                OneKeyLog.error("BundleUpdate", "verifyBundleASC: unzip failed: domain=\(nsErr.domain) code=\(nsErr.code) desc=\(nsErr.localizedDescription)")
                 try? FileManager.default.removeItem(atPath: destination)
-                throw NSError(domain: "BundleUpdate", code: -1, userInfo: [NSLocalizedDescriptionKey: "Failed to unzip bundle: \(error.localizedDescription)"])
+                throw NSError(domain: "BundleUpdate", code: -1, userInfo: [NSLocalizedDescriptionKey: "Failed to unzip bundle: IO_\(nsErr.code)"])
             }
 
             // Validate extracted paths (symlinks, path traversal)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@onekeyfe/react-native-bundle-update",
-  "version": "3.0.28",
+  "version": "3.0.29",
   "description": "react-native-bundle-update",
   "main": "./lib/module/index.js",
   "types": "./lib/typescript/src/index.d.ts",