@capgo/capacitor-uploader 8.1.10 → 8.1.12

package/android/src/main/java/ee/forgr/capacitor/uploader/Uploader.java

@@ -63,10 +63,14 @@ public class Uploader {
             request.addFileToUpload(filePath, fileField, getFileNameFromUri(Uri.parse(filePath)), mimeType);
 
             for (Map.Entry<String, String> entry : headers.entrySet()) {
-                request.addHeader(entry.getKey(), entry.getValue());
+                if (entry.getKey() != null && entry.getValue() != null) {
+                    request.addHeader(entry.getKey(), entry.getValue());
+                }
             }
             for (Map.Entry<String, String> entry : parameters.entrySet()) {
-                request.addParameter(entry.getKey(), entry.getValue());
+                if (entry.getKey() != null && entry.getValue() != null) {
+                    request.addParameter(entry.getKey(), entry.getValue());
+                }
             }
 
             return request.startUpload();
@@ -91,14 +95,20 @@ public class Uploader {
             .setNotificationConfig((ctx, uploadId) -> notificationConfig)
             .setMaxRetries(maxRetries);
 
-        request.addHeader("Content-Type", mimeType);
+        if (mimeType != null && !mimeType.isEmpty()) {
+            request.addHeader("Content-Type", mimeType);
+        }
 
         for (Map.Entry<String, String> entry : headers.entrySet()) {
-            request.addHeader(entry.getKey(), entry.getValue());
+            if (entry.getKey() != null && entry.getValue() != null) {
+                request.addHeader(entry.getKey(), entry.getValue());
+            }
         }
 
         for (Map.Entry<String, String> entry : parameters.entrySet()) {
-            request.addParameter(entry.getKey(), entry.getValue());
+            if (entry.getKey() != null && entry.getValue() != null) {
+                request.addParameter(entry.getKey(), entry.getValue());
+            }
         }
 
         return request.startUpload();
@@ -132,7 +142,7 @@ public class Uploader {
 
     private String getFileNameFromUri(Uri uri) {
         String result = null;
-        if (uri.getScheme().equals("content")) {
+        if ("content".equals(uri.getScheme())) {
             try (Cursor cursor = context.getContentResolver().query(uri, null, null, null, null)) {
                 if (cursor != null && cursor.moveToFirst()) {
                     int index = cursor.getColumnIndex(OpenableColumns.DISPLAY_NAME);
@@ -145,10 +155,10 @@ public class Uploader {
             }
         }
         if (result == null) {
-            result = uri.getPath();
-            int cut = result.lastIndexOf('/');
-            if (cut != -1) {
-                result = result.substring(cut + 1);
+            String path = uri.getPath();
+            if (path != null) {
+                int cut = path.lastIndexOf('/');
+                result = cut != -1 ? path.substring(cut + 1) : path;
             }
         }
         return result;

package/android/src/main/java/ee/forgr/capacitor/uploader/UploaderPlugin.java

@@ -3,6 +3,7 @@ package ee.forgr.capacitor.uploader;
 import android.app.NotificationChannel;
 import android.app.NotificationManager;
 import android.content.Context;
+import android.net.Uri;
 import android.os.Build;
 import android.webkit.MimeTypeMap;
 import com.getcapacitor.JSObject;
@@ -21,7 +22,11 @@ import net.gotev.uploadservice.observer.request.RequestObserverDelegate;
 @CapacitorPlugin(name = "Uploader")
 public class UploaderPlugin extends Plugin {
 
-    private final String pluginVersion = "8.1.10";
+    private static final String CAPACITOR_FILE_PATH_PREFIX = "/_capacitor_file_";
+
+    private static final String CAPACITOR_CONTENT_PATH_PREFIX = "/_capacitor_content_";
+
+    private final String pluginVersion = "8.1.12";
 
     private Uploader implementation;
 
@@ -123,6 +128,10 @@ public class UploaderPlugin extends Plugin {
             return;
         }
 
+        // Convert Capacitor web-accessible URLs to paths native code can open.
+        // Capacitor 8+ removed Bridge.getLocalUrl(String); mirror AndroidProtocolHandler logic.
+        String localFilePath = resolveCapacitorPath(filePath);
+
         JSObject headersObj = call.getObject("headers", new JSObject());
         JSObject parametersObj = call.getObject("parameters", new JSObject());
         String httpMethod = call.getString("method", "POST");
@@ -135,10 +144,10 @@ public class UploaderPlugin extends Plugin {
         Map<String, String> parameters = JSObjectToMap(parametersObj);
 
         try {
-            String mimeType = call.getString("mimeType", getMimeType(filePath));
+            String mimeType = call.getString("mimeType", getMimeType(localFilePath));
 
             String id = implementation.startUpload(
-                filePath,
+                localFilePath,
                 serverUrl,
                 headers,
                 parameters,
@@ -172,12 +181,50 @@ public class UploaderPlugin extends Plugin {
         }
     }
 
+    /**
+     * Maps WebView URLs (e.g. http(s)://localhost/_capacitor_file_/...) to filesystem or content
+     * paths, matching {@link com.getcapacitor.AndroidProtocolHandler}. Plain absolute paths and
+     * unrecognized URLs are returned unchanged.
+     */
+    private static String resolveCapacitorPath(String filePath) {
+        if (filePath == null || filePath.isEmpty()) {
+            return filePath;
+        }
+        Uri uri = Uri.parse(filePath);
+        String path = uri.getPath();
+        if (path != null) {
+            if (path.startsWith(CAPACITOR_FILE_PATH_PREFIX)) {
+                return path.substring(CAPACITOR_FILE_PATH_PREFIX.length());
+            }
+            if (path.startsWith(CAPACITOR_CONTENT_PATH_PREFIX)) {
+                String scheme = uri.getScheme();
+                String host = uri.getHost();
+                if (scheme != null && host != null) {
+                    String baseUrl = scheme + "://" + host;
+                    if (uri.getPort() != -1) {
+                        baseUrl += ":" + uri.getPort();
+                    }
+                    return filePath.replace(baseUrl + CAPACITOR_CONTENT_PATH_PREFIX, "content:/");
+                }
+                return filePath.replace(CAPACITOR_CONTENT_PATH_PREFIX, "content:/");
+            }
+        }
+        if ("file".equalsIgnoreCase(uri.getScheme()) && uri.getPath() != null) {
+            return uri.getPath();
+        }
+        return filePath;
+    }
+
     private Map<String, String> JSObjectToMap(JSObject object) {
         Map<String, String> map = new HashMap<>();
         if (object != null) {
             for (Iterator<String> it = object.keys(); it.hasNext(); ) {
                 String key = it.next();
-                map.put(key, object.getString(key));
+                String value = object.getString(key);
+                // Only add non-null and non-empty values to prevent upload service errors
+                if (value != null && !value.isEmpty()) {
+                    map.put(key, value);
+                }
             }
         }
         return map;

package/ios/Sources/UploaderPlugin/Uploader.swift

@@ -8,6 +8,15 @@ import MobileCoreServices
     private var responsesData: [Int: Data] = [:]
     private var tasks: [String: URLSessionTask] = [:]
     private var retries: [String: Int] = [:]
+    private var tempBodyFiles: [String: URL] = [:]
+
+    private struct UploadConfig {
+        let filePath: String
+        let serverUrl: String
+        let options: [String: Any]
+    }
+
+    private var uploadConfigs: [String: UploadConfig] = [:]
 
     var eventHandler: (([String: Any]) -> Void)?
 
@@ -15,22 +24,46 @@ import MobileCoreServices
         let id = UUID().uuidString
         print("startUpload: \(id)")
 
-        guard let url = URL(string: serverUrl) else {
+        let config = UploadConfig(filePath: filePath, serverUrl: serverUrl, options: options)
+        do {
+            let task = try createUploadTask(id: id, config: config)
+            uploadConfigs[id] = config
+            tasks[id] = task
+            retries[id] = maxRetries
+            task.resume()
+            return id
+        } catch {
+            if let tempUrl = tempBodyFiles.removeValue(forKey: id) {
+                try? FileManager.default.removeItem(at: tempUrl)
+            }
+            throw error
+        }
+    }
+
+    private func createUploadTask(id: String, config: UploadConfig) throws -> URLSessionTask {
+        guard let url = URL(string: config.serverUrl) else {
             throw NSError(domain: "UploaderPlugin", code: 0, userInfo: [NSLocalizedDescriptionKey: "Invalid URL"])
         }
 
         var request = URLRequest(url: url)
-        request.httpMethod = (options["method"] as? String)?.uppercased() ?? "POST"
+        request.httpMethod = (config.options["method"] as? String)?.uppercased() ?? "POST"
 
-        let headers = options["headers"] as? [String: String] ?? [:]
+        let headers = config.options["headers"] as? [String: String] ?? [:]
         for (key, value) in headers {
             request.setValue(value, forHTTPHeaderField: key)
         }
 
-        guard let fileUrl = URL(string: filePath) else {
+        let filePath = config.filePath
+        let fileUrl: URL
+        if let candidateUrl = URL(string: filePath), candidateUrl.isFileURL {
+            fileUrl = candidateUrl
+        } else {
+            fileUrl = URL(fileURLWithPath: filePath)
+        }
+        guard fileUrl.isFileURL else {
             throw NSError(domain: "UploaderPlugin", code: 1, userInfo: [NSLocalizedDescriptionKey: "Invalid file URL"])
         }
-        let mimeType = options["mimeType"] as? String ?? guessMIMEType(from: filePath)
+        let mimeType = config.options["mimeType"] as? String ?? guessMIMEType(from: filePath)
 
         let task: URLSessionTask
         if request.httpMethod == "PUT" {
@@ -42,19 +75,21 @@ import MobileCoreServices
             let boundary = UUID().uuidString
             request.setValue("multipart/form-data; boundary=\(boundary)", forHTTPHeaderField: "Content-Type")
 
-            let parameters = options["parameters"] as? [String: String] ?? [:]
+            let parameters = config.options["parameters"] as? [String: String] ?? [:]
 
-            let dataBody = createDataBody(withParameters: parameters, filePath: filePath, mimeType: mimeType, boundary: boundary)
+            if let oldTemp = tempBodyFiles[id] {
+                try? FileManager.default.removeItem(at: oldTemp)
+            }
+            let tempDir = FileManager.default.temporaryDirectory
+            let tempFile = tempDir.appendingPathComponent("upload-\(id).tmp")
+            try writeMultipartBodyToFile(at: tempFile, parameters: parameters, fileUrl: fileUrl, mimeType: mimeType, boundary: boundary)
+            tempBodyFiles[id] = tempFile
 
-            task = self.getUrlSession().uploadTask(with: request, from: dataBody)
+            task = self.getUrlSession().uploadTask(with: request, fromFile: tempFile)
         }
 
         task.taskDescription = id
-        tasks[id] = task
-        retries[id] = maxRetries
-        task.resume()
-
-        return id
+        return task
     }
 
     @objc public func removeUpload(_ id: String) async throws {
@@ -62,6 +97,11 @@ import MobileCoreServices
         if let task = tasks[id] {
             task.cancel()
             tasks.removeValue(forKey: id)
+            retries.removeValue(forKey: id)
+            uploadConfigs.removeValue(forKey: id)
+            if let tempUrl = tempBodyFiles.removeValue(forKey: id) {
+                try? FileManager.default.removeItem(at: tempUrl)
+            }
         }
     }
 
@@ -89,21 +129,33 @@ import MobileCoreServices
         }
 
         if let error = error {
-            if let retriesLeft = retries[id], retriesLeft > 0 {
-                retries[id] = retriesLeft - 1
-                print("Retrying upload (retries left: \(retriesLeft - 1))")
-                task.resume()
-                return
+            let isCancelled = (error as NSError).code == NSURLErrorCancelled
+            if !isCancelled, let retriesLeft = retries[id], retriesLeft > 0, let config = uploadConfigs[id] {
+                let newRetries = retriesLeft - 1
+                retries[id] = newRetries
+                print("Retrying upload (retries left: \(newRetries))")
+                do {
+                    let newTask = try createUploadTask(id: id, config: config)
+                    tasks[id] = newTask
+                    newTask.resume()
+                    return
+                } catch {
+                    payload["error"] = error.localizedDescription
+                }
+            } else {
+                payload["error"] = error.localizedDescription
             }
-
-            payload["error"] = error.localizedDescription
             sendEvent(name: "failed", id: id, payload: payload)
         } else {
             sendEvent(name: "completed", id: id, payload: payload)
         }
 
+        if let tempUrl = tempBodyFiles.removeValue(forKey: id) {
+            try? FileManager.default.removeItem(at: tempUrl)
+        }
         tasks.removeValue(forKey: id)
         retries.removeValue(forKey: id)
+        uploadConfigs.removeValue(forKey: id)
     }
 
     public func urlSession(_ session: URLSession, task: URLSessionTask, didSendBodyData bytesSent: Int64, totalBytesSent: Int64, totalBytesExpectedToSend: Int64) {
@@ -126,25 +178,36 @@ import MobileCoreServices
         eventHandler?(event)
     }
 
-    private func createDataBody(withParameters params: [String: String], filePath: String, mimeType: String, boundary: String) -> Data {
-        let data = NSMutableData()
+    // Writes multipart/form-data body directly to a file, streaming the file content in chunks
+    private func writeMultipartBodyToFile(at tempFile: URL, parameters: [String: String], fileUrl: URL, mimeType: String, boundary: String) throws {
+        FileManager.default.createFile(atPath: tempFile.path, contents: nil)
+        let writeHandle = try FileHandle(forWritingTo: tempFile)
+        defer { try? writeHandle.close() }
 
-        for (key, value) in params {
-            data.append("--\(boundary)\r\n".data(using: .utf8)!)
-            data.append("Content-Disposition: form-data; name=\"\(key)\"\r\n\r\n".data(using: .utf8)!)
-            data.append("\(value)\r\n".data(using: .utf8)!)
+        func write(_ string: String) throws {
+            try writeHandle.write(contentsOf: string.data(using: .utf8)!)
         }
 
-        data.append("--\(boundary)\r\n".data(using: .utf8)!)
-        data.append("Content-Disposition: form-data; name=\"file\"; filename=\"\(URL(fileURLWithPath: filePath).lastPathComponent)\"\r\n".data(using: .utf8)!)
-        data.append("Content-Type: \(mimeType)\r\n\r\n".data(using: .utf8)!)
-        data.append(try! Data(contentsOf: URL(fileURLWithPath: filePath)))
-        data.append("\r\n".data(using: .utf8)!)
-        data.append("--\(boundary)--".data(using: .utf8)!)
+        for (key, value) in parameters {
+            try write("--\(boundary)\r\n")
+            try write("Content-Disposition: form-data; name=\"\(key)\"\r\n\r\n")
+            try write("\(value)\r\n")
+        }
 
-        return data as Data
-    }
+        try write("--\(boundary)\r\n")
+        try write("Content-Disposition: form-data; name=\"file\"; filename=\"\(fileUrl.lastPathComponent)\"\r\n")
+        try write("Content-Type: \(mimeType)\r\n\r\n")
+
+        let readHandle = try FileHandle(forReadingFrom: fileUrl)
+        defer { try? readHandle.close() }
+        let chunkSize = 64 * 1024
+        while true {
+            guard let chunk = try readHandle.read(upToCount: chunkSize), !chunk.isEmpty else { break }
+            try writeHandle.write(contentsOf: chunk)
+        }
 
+        try write("\r\n--\(boundary)--")
+    }
 }
 
 extension Uploader: URLSessionDataDelegate {

package/ios/Sources/UploaderPlugin/UploaderPlugin.swift

@@ -3,7 +3,7 @@ import Capacitor
 
 @objc(UploaderPlugin)
 public class UploaderPlugin: CAPPlugin, CAPBridgedPlugin {
-    private let pluginVersion: String = "8.1.10"
+    private let pluginVersion: String = "8.1.12"
     public let identifier = "UploaderPlugin"
     public let jsName = "Uploader"
     public let pluginMethods: [CAPPluginMethod] = [

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@capgo/capacitor-uploader",
-  "version": "8.1.10",
+  "version": "8.1.12",
   "description": "Upload file natively",
   "main": "dist/plugin.cjs.js",
   "module": "dist/esm/index.js",