feat(web/ios): allow string value in body

ios/Plugin/CapacitorUrlRequest.swift

@@ -16,8 +16,12 @@ public class CapacitorUrlRequest: NSObject, URLSessionTaskDelegate {
     }
 
     private func getRequestDataAsJson(_ data: JSValue) throws -> Data? {
-      let jsonData = try JSONSerialization.data(withJSONObject: data)
-      return jsonData
+        // We need to check if the JSON is valid before attempting to serialize, as JSONSerialization.data will not throw an exception that can be caught, and will cause the application to crash if it fails.
+        if JSONSerialization.isValidJSONObject(data) {
+            return try JSONSerialization.data(withJSONObject: data)
+        } else {
+            throw CapacitorUrlRequest.CapacitorUrlRequestError.serializationError("[ data ] argument for request of content-type [ application/json ] must be serializable to JSON")
+        }
     }
 
     private func getRequestDataAsFormUrlEncoded(_ data: JSValue) throws -> Data? {
@@ -84,14 +88,17 @@ public class CapacitorUrlRequest: NSObject, URLSessionTaskDelegate {
     }
 
     func getRequestData(_ body: JSValue, _ contentType: String) throws -> Data? {
-        if contentType.contains("application/json") {
+        // If data can be parsed directly as a string, return that without processing.
+        if let strVal = try? getRequestDataAsString(body) {
+            return strVal
+        } else if contentType.contains("application/json") {
             return try getRequestDataAsJson(body)
         } else if contentType.contains("application/x-www-form-urlencoded") {
             return try getRequestDataAsFormUrlEncoded(body)
         } else if contentType.contains("multipart/form-data") {
             return try getRequestDataAsMultipartFormData(body)
         } else {
-            return try getRequestDataAsString(body)
+            throw CapacitorUrlRequestError.serializationError("[ data ] argument could not be parsed for content type [ \(contentType) ]")
         }
     }
 

ios/Plugin/HttpRequestHandler.swift

@@ -160,11 +160,6 @@ class HttpRequestHandler {
         let connectTimeout = call.getDouble("connectTimeout");
         let readTimeout = call.getDouble("readTimeout");
 
-        let isHttpMutate = method == "DELETE" ||
-            method == "PATCH" ||
-            method == "POST" ||
-            method == "PUT";
-
         let request = try! CapacitorHttpRequestBuilder()
             .setUrl(urlString)
             .setMethod(method)
@@ -178,19 +173,14 @@ class HttpRequestHandler {
         let timeout = (connectTimeout ?? readTimeout ?? 600000.0) / 1000.0;
         request.setTimeout(timeout)
 
-        if isHttpMutate {
+        if let data = call.options["data"] as? JSValue {
             do {
-                guard let data = call.jsObjectRepresentation["data"]
-                else {
-                    throw CapacitorUrlRequest.CapacitorUrlRequestError.serializationError("Invalid [ data ] argument")
-                }
-
                 try request.setRequestBody(data)
             } catch {
                 // Explicitly reject if the http request body was not set successfully,
                 // so as to not send a known malformed request, and to provide the developer with additional context.
                 call.reject("Error", "REQUEST", error, [:])
-                return;
+                return
             }
         }
 
@@ -199,8 +189,7 @@ class HttpRequestHandler {
         let task = urlSession.dataTask(with: urlRequest) { (data, response, error) in
             urlSession.invalidateAndCancel();
             if error != nil {
-                call.reject("Error", "REQUEST", error, [:])
-                return;
+                return
             }
 
             let type = ResponseType(rawValue: responseType) ?? .default

src/request.ts

@@ -75,8 +75,12 @@ export const buildRequestInit = (
   const headers = normalizeHttpHeaders(options.headers);
   const type = headers['content-type'] || '';
 
+  // If body is already a string, then pass it through as-is.
+  if (typeof options.data === 'string') {
+    output.body = options.data;
+  }
   // Build request initializers based off of content-type
-  if (type.includes('application/x-www-form-urlencoded')) {
+  else if (type.includes('application/x-www-form-urlencoded')) {
     const params = new URLSearchParams();
     for (const [key, value] of Object.entries(options.data || {})) {
       params.set(key, value as any);