react-native-tvos 0.81.3-0 → 0.82.0-0

package/ReactAndroid/src/main/java/com/facebook/react/modules/network/NetworkingModule.kt

@@ -11,11 +11,9 @@
 package com.facebook.react.modules.network
 
 import android.net.Uri
-import android.os.Bundle
 import android.util.Base64
 import com.facebook.common.logging.FLog
 import com.facebook.fbreact.specs.NativeNetworkingAndroidSpec
-import com.facebook.react.bridge.Arguments
 import com.facebook.react.bridge.ReactApplicationContext
 import com.facebook.react.bridge.ReactMethod
 import com.facebook.react.bridge.ReadableArray
@@ -36,6 +34,7 @@ import okhttp3.JavaNetCookieJar
 import okhttp3.MediaType
 import okhttp3.MultipartBody
 import okhttp3.OkHttpClient
+import okhttp3.Protocol
 import okhttp3.Request
 import okhttp3.RequestBody
 import okhttp3.Response
@@ -50,7 +49,7 @@ public class NetworkingModule(
     reactContext: ReactApplicationContext,
     defaultUserAgent: String?,
     client: OkHttpClient,
-    networkInterceptorCreators: List<NetworkInterceptorCreator>?
+    networkInterceptorCreators: List<NetworkInterceptorCreator>?,
 ) : NativeNetworkingAndroidSpec(reactContext) {
 
   /**
@@ -61,8 +60,10 @@ public class NetworkingModule(
     /** Returns if the handler should be used for an URI. */
     public fun supports(uri: Uri, responseType: String): Boolean
 
-    /** Fetch the URI and return the JS body payload. */
-    @Throws(IOException::class) public fun fetch(uri: Uri): WritableMap
+    /**
+     * Fetch the URI and return a tuple containing the JS body payload and the raw response body.
+     */
+    @Throws(IOException::class) public fun fetch(uri: Uri): Pair<WritableMap, ByteArray>
   }
 
   /** Allows adding custom handling to build the [RequestBody] from the JS body payload. */
@@ -80,7 +81,7 @@ public class NetworkingModule(
     public fun supports(responseType: String): Boolean
 
     /** Returns the JS body payload for the [ResponseBody]. */
-    @Throws(IOException::class) public fun toResponseData(body: ResponseBody): WritableMap
+    @Throws(IOException::class) public fun toResponseData(data: ByteArray): WritableMap
   }
 
   private val client: OkHttpClient
@@ -123,7 +124,7 @@ public class NetworkingModule(
   internal constructor(
       context: ReactApplicationContext,
       defaultUserAgent: String?,
-      client: OkHttpClient
+      client: OkHttpClient,
   ) : this(context, defaultUserAgent, client, null)
 
   /** @param context the ReactContext of the application */
@@ -138,12 +139,13 @@ public class NetworkingModule(
    */
   public constructor(
       context: ReactApplicationContext,
-      networkInterceptorCreators: List<NetworkInterceptorCreator>?
+      networkInterceptorCreators: List<NetworkInterceptorCreator>?,
   ) : this(
       context,
       null,
       OkHttpClientProvider.createClient(context.applicationContext),
-      networkInterceptorCreators)
+      networkInterceptorCreators,
+  )
 
   /**
    * @param context the ReactContext of the application
@@ -152,16 +154,18 @@ public class NetworkingModule(
    */
   public constructor(
       context: ReactApplicationContext,
-      defaultUserAgent: String?
+      defaultUserAgent: String?,
   ) : this(
       context,
       defaultUserAgent,
       OkHttpClientProvider.createClient(context.applicationContext),
-      null)
+      null,
+  )
 
   @Deprecated(
       """To be removed in a future release. See
-        https://github.com/facebook/react-native/pull/37798#pullrequestreview-1518338914""")
+        https://github.com/facebook/react-native/pull/37798#pullrequestreview-1518338914"""
+  )
   public interface CustomClientBuilder : com.facebook.react.modules.network.CustomClientBuilder
 
   override fun initialize() {
@@ -180,27 +184,27 @@ public class NetworkingModule(
     uriHandlers.clear()
   }
 
-  internal fun addUriHandler(handler: UriHandler): Unit {
+  internal fun addUriHandler(handler: UriHandler) {
     uriHandlers.add(handler)
   }
 
-  internal fun addRequestBodyHandler(handler: RequestBodyHandler): Unit {
+  internal fun addRequestBodyHandler(handler: RequestBodyHandler) {
     requestBodyHandlers.add(handler)
   }
 
-  internal fun addResponseHandler(handler: ResponseHandler): Unit {
+  internal fun addResponseHandler(handler: ResponseHandler) {
     responseHandlers.add(handler)
   }
 
-  internal fun removeUriHandler(handler: UriHandler): Unit {
+  internal fun removeUriHandler(handler: UriHandler) {
     uriHandlers.remove(handler)
   }
 
-  internal fun removeRequestBodyHandler(handler: RequestBodyHandler): Unit {
+  internal fun removeRequestBodyHandler(handler: RequestBodyHandler) {
     requestBodyHandlers.remove(handler)
   }
 
-  internal fun removeResponseHandler(handler: ResponseHandler): Unit {
+  internal fun removeResponseHandler(handler: ResponseHandler) {
     responseHandlers.remove(handler)
   }
 
@@ -213,7 +217,7 @@ public class NetworkingModule(
       responseType: String,
       useIncrementalUpdates: Boolean,
       timeoutAsDouble: Double,
-      withCredentials: Boolean
+      withCredentials: Boolean,
   ) {
     val requestId = requestIdAsDouble.toInt()
     val timeout = timeoutAsDouble.toInt()
@@ -227,12 +231,17 @@ public class NetworkingModule(
           responseType,
           useIncrementalUpdates,
           timeout,
-          withCredentials)
+          withCredentials,
+      )
     } catch (th: Throwable) {
       FLog.e(TAG, "Failed to send url request: $url", th)
 
-      ResponseUtil.onRequestError(
-          getReactApplicationContextIfActiveOrWarn(), requestId, th.message, th)
+      NetworkEventUtil.onRequestError(
+          getReactApplicationContextIfActiveOrWarn(),
+          requestId,
+          th.message,
+          th,
+      )
     }
   }
 
@@ -246,8 +255,8 @@ public class NetworkingModule(
       responseType: String,
       useIncrementalUpdates: Boolean,
       timeout: Int,
-      withCredentials: Boolean
-  ): Unit {
+      withCredentials: Boolean,
+  ) {
     val reactApplicationContext = getReactApplicationContextIfActiveOrWarn()
     try {
       val uri = Uri.parse(url)
@@ -255,18 +264,29 @@ public class NetworkingModule(
       // Check if a handler is registered
       for (handler in uriHandlers) {
         if (handler.supports(uri, responseType)) {
-          val res = handler.fetch(uri)
+          val (res, rawBody) = handler.fetch(uri)
+          val encodedDataLength = res.toString().toByteArray().size
           // fix: UriHandlers which are not using file:// scheme fail in whatwg-fetch at this line
           // https://github.com/JakeChampion/fetch/blob/main/fetch.js#L547
-          ResponseUtil.onResponseReceived(
-              reactApplicationContext, requestId, 200, Arguments.createMap(), url)
-          ResponseUtil.onDataReceived(reactApplicationContext, requestId, res)
-          ResponseUtil.onRequestSuccess(reactApplicationContext, requestId)
+          val response =
+              Response.Builder()
+                  .protocol(Protocol.HTTP_1_1)
+                  .request(Request.Builder().url(url.orEmpty()).build())
+                  .code(200)
+                  .message("OK")
+                  .build()
+          NetworkEventUtil.onResponseReceived(reactApplicationContext, requestId, url, response)
+          NetworkEventUtil.onDataReceived(reactApplicationContext, requestId, res, rawBody)
+          NetworkEventUtil.onRequestSuccess(
+              reactApplicationContext,
+              requestId,
+              encodedDataLength.toLong(),
+          )
           return
         }
       }
     } catch (e: IOException) {
-      ResponseUtil.onRequestError(reactApplicationContext, requestId, e.message, e)
+      NetworkEventUtil.onRequestError(reactApplicationContext, requestId, e.message, e)
       return
     }
 
@@ -274,7 +294,7 @@ public class NetworkingModule(
     try {
       requestBuilder = Request.Builder().url(url.orEmpty())
     } catch (e: Exception) {
-      ResponseUtil.onRequestError(reactApplicationContext, requestId, e.message, null)
+      NetworkEventUtil.onRequestError(reactApplicationContext, requestId, e.message, e)
       return
     }
 
@@ -313,11 +333,16 @@ public class NetworkingModule(
                       // JS below, so no need to do anything here.
                       return
                     }
-                    ResponseUtil.onDataReceivedProgress(
-                        reactApplicationContext, requestId, bytesWritten, contentLength)
+                    NetworkEventUtil.onDataReceivedProgress(
+                        reactApplicationContext,
+                        requestId,
+                        bytesWritten,
+                        contentLength,
+                    )
                     last = now
                   }
-                })
+                },
+            )
         originalResponse.newBuilder().body(responseBody).build()
       }
     }
@@ -333,8 +358,12 @@ public class NetworkingModule(
 
     val requestHeaders = extractHeaders(headers, data)
     if (requestHeaders == null) {
-      ResponseUtil.onRequestError(
-          reactApplicationContext, requestId, "Unrecognized headers format", null)
+      NetworkEventUtil.onRequestError(
+          reactApplicationContext,
+          requestId,
+          "Unrecognized headers format",
+          null,
+      )
       return
     }
     var contentType = requestHeaders[CONTENT_TYPE_HEADER_NAME]
@@ -359,11 +388,12 @@ public class NetworkingModule(
       handler != null -> requestBody = handler.toRequestBody(data, contentType)
       data.hasKey(REQUEST_BODY_KEY_STRING) -> {
         if (contentType == null) {
-          ResponseUtil.onRequestError(
+          NetworkEventUtil.onRequestError(
               reactApplicationContext,
               requestId,
               "Payload is set but no content-type header specified",
-              null)
+              null,
+          )
           return
         }
         val body = data.getString(REQUEST_BODY_KEY_STRING)
@@ -374,8 +404,12 @@ public class NetworkingModule(
             requestBody = RequestBodyUtil.createGzip(contentMediaType, body)
           }
           if (requestBody == null) {
-            ResponseUtil.onRequestError(
-                reactApplicationContext, requestId, "Failed to gzip request body", null)
+            NetworkEventUtil.onRequestError(
+                reactApplicationContext,
+                requestId,
+                "Failed to gzip request body",
+                null,
+            )
             return
           }
         } else {
@@ -389,8 +423,12 @@ public class NetworkingModule(
                 checkNotNull(contentMediaType.charset(StandardCharsets.UTF_8))
               }
           if (body == null) {
-            ResponseUtil.onRequestError(
-                reactApplicationContext, requestId, "Received request but body was empty", null)
+            NetworkEventUtil.onRequestError(
+                reactApplicationContext,
+                requestId,
+                "Received request but body was empty",
+                null,
+            )
             return
           }
           @Suppress("DEPRECATION")
@@ -399,11 +437,12 @@ public class NetworkingModule(
       }
       data.hasKey(REQUEST_BODY_KEY_BASE64) -> {
         if (contentType == null) {
-          ResponseUtil.onRequestError(
+          NetworkEventUtil.onRequestError(
               reactApplicationContext,
               requestId,
               "Payload is set but no content-type header specified",
-              null)
+              null,
+          )
           return
         }
         val base64String = data.getString(REQUEST_BODY_KEY_BASE64)
@@ -411,17 +450,22 @@ public class NetworkingModule(
 
         val contentMediaType = MediaType.parse(contentType)
         if (contentMediaType == null) {
-          ResponseUtil.onRequestError(
+          NetworkEventUtil.onRequestError(
               reactApplicationContext,
               requestId,
               "Invalid content type specified: $contentType",
-              null)
+              null,
+          )
           return
         }
         val base64DecodedString = ByteString.decodeBase64(base64String)
         if (base64DecodedString == null) {
-          ResponseUtil.onRequestError(
-              reactApplicationContext, requestId, "Request body base64 string was invalid", null)
+          NetworkEventUtil.onRequestError(
+              reactApplicationContext,
+              requestId,
+              "Request body base64 string was invalid",
+              null,
+          )
           return
         }
         @Suppress("DEPRECATION")
@@ -429,23 +473,32 @@ public class NetworkingModule(
       }
       data.hasKey(REQUEST_BODY_KEY_URI) -> {
         if (contentType == null) {
-          ResponseUtil.onRequestError(
+          NetworkEventUtil.onRequestError(
               reactApplicationContext,
               requestId,
               "Payload is set but no content-type header specified",
-              null)
+              null,
+          )
           return
         }
         val uri = data.getString(REQUEST_BODY_KEY_URI)
         if (uri == null) {
-          ResponseUtil.onRequestError(
-              reactApplicationContext, requestId, "Request body URI field was set but null", null)
+          NetworkEventUtil.onRequestError(
+              reactApplicationContext,
+              requestId,
+              "Request body URI field was set but null",
+              null,
+          )
           return
         }
         val fileInputStream = RequestBodyUtil.getFileInputStream(getReactApplicationContext(), uri)
         if (fileInputStream == null) {
-          ResponseUtil.onRequestError(
-              reactApplicationContext, requestId, "Could not retrieve file for uri $uri", null)
+          NetworkEventUtil.onRequestError(
+              reactApplicationContext,
+              requestId,
+              "Could not retrieve file for uri $uri",
+              null,
+          )
           return
         }
         requestBody = RequestBodyUtil.create(MediaType.parse(contentType), fileInputStream)
@@ -456,8 +509,12 @@ public class NetworkingModule(
         }
         val parts = data.getArray(REQUEST_BODY_KEY_FORMDATA)
         if (parts == null) {
-          ResponseUtil.onRequestError(
-              reactApplicationContext, requestId, "Received request but form data was empty", null)
+          NetworkEventUtil.onRequestError(
+              reactApplicationContext,
+              requestId,
+              "Received request but form data was empty",
+              null,
+          )
           return
         }
         val multipartBuilder = constructMultipartBody(parts, contentType, requestId) ?: return
@@ -472,8 +529,11 @@ public class NetworkingModule(
     requestBuilder.method(method, wrapRequestBodyWithProgressEmitter(requestBody, requestId))
 
     addRequest(requestId)
+    val request = requestBuilder.build()
+    NetworkEventUtil.onCreateRequest(requestId, request)
+
     client
-        .newCall(requestBuilder.build())
+        .newCall(request)
         .enqueue(
             object : Callback {
               override fun onFailure(call: Call, e: IOException) {
@@ -483,7 +543,7 @@ public class NetworkingModule(
                 removeRequest(requestId)
                 val errorMessage =
                     e.message ?: ("Error while executing request: ${e.javaClass.simpleName}")
-                ResponseUtil.onRequestError(reactApplicationContext, requestId, errorMessage, e)
+                NetworkEventUtil.onRequestError(reactApplicationContext, requestId, errorMessage, e)
               }
 
               @Throws(IOException::class)
@@ -493,12 +553,12 @@ public class NetworkingModule(
                 }
                 removeRequest(requestId)
                 // Before we touch the body send headers to JS
-                ResponseUtil.onResponseReceived(
+                NetworkEventUtil.onResponseReceived(
                     reactApplicationContext,
                     requestId,
-                    response.code(),
-                    translateHeaders(response.headers()),
-                    response.request().url().toString())
+                    url,
+                    response,
+                )
 
                 try {
                   // OkHttp implements something called transparent gzip, which mean that it will
@@ -517,8 +577,12 @@ public class NetworkingModule(
                   // https://github.com/square/okhttp/blob/5b37cda9e00626f43acf354df145fd452c3031f1/okhttp/src/main/java/okhttp3/internal/http/BridgeInterceptor.java#L76-L111
                   var responseBody: ResponseBody? = response.body()
                   if (responseBody == null) {
-                    ResponseUtil.onRequestError(
-                        reactApplicationContext, requestId, "Response body is null", null)
+                    NetworkEventUtil.onRequestError(
+                        reactApplicationContext,
+                        requestId,
+                        "Response body is null",
+                        null,
+                    )
                     return
                   }
                   if ("gzip".equals(response.header("Content-Encoding"), ignoreCase = true)) {
@@ -537,9 +601,19 @@ public class NetworkingModule(
                   // Check if a handler is registered
                   for (responseHandler in responseHandlers) {
                     if (responseHandler.supports(responseType)) {
-                      val res = responseHandler.toResponseData(responseBody)
-                      ResponseUtil.onDataReceived(reactApplicationContext, requestId, res)
-                      ResponseUtil.onRequestSuccess(reactApplicationContext, requestId)
+                      val responseData = responseBody.bytes()
+                      val res = responseHandler.toResponseData(responseData)
+                      NetworkEventUtil.onDataReceived(
+                          reactApplicationContext,
+                          requestId,
+                          res,
+                          responseData,
+                      )
+                      NetworkEventUtil.onRequestSuccess(
+                          reactApplicationContext,
+                          requestId,
+                          responseBody.contentLength(),
+                      )
                       return
                     }
                   }
@@ -549,7 +623,11 @@ public class NetworkingModule(
                   // periodically send response data updates to JS.
                   if (useIncrementalUpdates && responseType == "text") {
                     readWithProgress(requestId, responseBody)
-                    ResponseUtil.onRequestSuccess(reactApplicationContext, requestId)
+                    NetworkEventUtil.onRequestSuccess(
+                        reactApplicationContext,
+                        requestId,
+                        responseBody.contentLength(),
+                    )
                     return
                   }
 
@@ -566,25 +644,39 @@ public class NetworkingModule(
                         // Javascript layer.
                         // Introduced to fix issue #7463.
                       } else {
-                        ResponseUtil.onRequestError(
-                            reactApplicationContext, requestId, e.message, e)
+                        NetworkEventUtil.onRequestError(
+                            reactApplicationContext,
+                            requestId,
+                            e.message,
+                            e,
+                        )
                       }
                     }
                   } else if (responseType == "base64") {
                     responseString = Base64.encodeToString(responseBody.bytes(), Base64.NO_WRAP)
                   }
-                  ResponseUtil.onDataReceived(reactApplicationContext, requestId, responseString)
-                  ResponseUtil.onRequestSuccess(reactApplicationContext, requestId)
+                  NetworkEventUtil.onDataReceived(
+                      reactApplicationContext,
+                      requestId,
+                      responseString,
+                      responseType,
+                  )
+                  NetworkEventUtil.onRequestSuccess(
+                      reactApplicationContext,
+                      requestId,
+                      responseBody.contentLength(),
+                  )
                 } catch (e: IOException) {
-                  ResponseUtil.onRequestError(reactApplicationContext, requestId, e.message, e)
+                  NetworkEventUtil.onRequestError(reactApplicationContext, requestId, e.message, e)
                 }
               }
-            })
+            }
+        )
   }
 
   private fun wrapRequestBodyWithProgressEmitter(
       requestBody: RequestBody?,
-      requestId: Int
+      requestId: Int,
   ): RequestBody? {
     if (requestBody == null) {
       return null
@@ -598,12 +690,17 @@ public class NetworkingModule(
           override fun onProgress(bytesWritten: Long, contentLength: Long, done: Boolean) {
             val now = System.nanoTime()
             if (done || shouldDispatch(now, last)) {
-              ResponseUtil.onDataSend(
-                  reactApplicationContext, requestId, bytesWritten, contentLength)
+              NetworkEventUtil.onDataSend(
+                  reactApplicationContext,
+                  requestId,
+                  bytesWritten,
+                  contentLength,
+              )
               last = now
             }
           }
-        })
+        },
+    )
   }
 
   @Throws(IOException::class)
@@ -633,12 +730,13 @@ public class NetworkingModule(
       var read: Int
       val reactApplicationContext = getReactApplicationContextIfActiveOrWarn()
       while ((inputStream.read(buffer).also { read = it }) != -1) {
-        ResponseUtil.onIncrementalDataReceived(
+        NetworkEventUtil.onIncrementalDataReceived(
             reactApplicationContext,
             requestId,
             streamDecoder.decodeNext(buffer, read),
             totalBytesRead,
-            contentLength)
+            contentLength,
+        )
       }
     } finally {
       inputStream.close()
@@ -674,7 +772,7 @@ public class NetworkingModule(
   }
 
   @ReactMethod
-  public override fun clearCookies(callback: com.facebook.react.bridge.Callback): Unit {
+  public override fun clearCookies(callback: com.facebook.react.bridge.Callback) {
     cookieHandler.clearCookies(callback)
   }
 
@@ -685,13 +783,18 @@ public class NetworkingModule(
   private fun constructMultipartBody(
       body: ReadableArray,
       contentType: String,
-      requestId: Int
+      requestId: Int,
   ): MultipartBody.Builder? {
     val reactApplicationContext = getReactApplicationContextIfActiveOrWarn()
     val multipartBuilder = MultipartBody.Builder()
     val mediaType = MediaType.parse(contentType)
     if (mediaType == null) {
-      ResponseUtil.onRequestError(reactApplicationContext, requestId, "Invalid media type.", null)
+      NetworkEventUtil.onRequestError(
+          reactApplicationContext,
+          requestId,
+          "Invalid media type.",
+          null,
+      )
       return null
     }
     multipartBuilder.setType(mediaType)
@@ -699,8 +802,12 @@ public class NetworkingModule(
     for (i in 0 until body.size()) {
       val bodyPart = body.getMap(i)
       if (bodyPart == null) {
-        ResponseUtil.onRequestError(
-            reactApplicationContext, requestId, "Unrecognized FormData part.", null)
+        NetworkEventUtil.onRequestError(
+            reactApplicationContext,
+            requestId,
+            "Unrecognized FormData part.",
+            null,
+        )
         return null
       }
 
@@ -708,11 +815,12 @@ public class NetworkingModule(
       val headersArray = bodyPart.getArray("headers")
       var headers = extractHeaders(headersArray, null)
       if (headers == null) {
-        ResponseUtil.onRequestError(
+        NetworkEventUtil.onRequestError(
             reactApplicationContext,
             requestId,
             "Missing or invalid header format for FormData part.",
-            null)
+            null,
+        )
         return null
       }
       var partContentType: MediaType? = null
@@ -724,41 +832,54 @@ public class NetworkingModule(
         headers = headers.newBuilder().removeAll(CONTENT_TYPE_HEADER_NAME).build()
       }
 
-      if (bodyPart.hasKey(REQUEST_BODY_KEY_STRING) &&
-          bodyPart.getString(REQUEST_BODY_KEY_STRING) != null) {
+      if (
+          bodyPart.hasKey(REQUEST_BODY_KEY_STRING) &&
+              bodyPart.getString(REQUEST_BODY_KEY_STRING) != null
+      ) {
         val bodyValue = bodyPart.getString(REQUEST_BODY_KEY_STRING).orEmpty()
         @Suppress("DEPRECATION")
         multipartBuilder.addPart(headers, RequestBody.create(partContentType, bodyValue))
-      } else if (bodyPart.hasKey(REQUEST_BODY_KEY_URI) &&
-          bodyPart.getString(REQUEST_BODY_KEY_URI) != null) {
+      } else if (
+          bodyPart.hasKey(REQUEST_BODY_KEY_URI) && bodyPart.getString(REQUEST_BODY_KEY_URI) != null
+      ) {
         if (partContentType == null) {
-          ResponseUtil.onRequestError(
+          NetworkEventUtil.onRequestError(
               reactApplicationContext,
               requestId,
               "Binary FormData part needs a content-type header.",
-              null)
+              null,
+          )
           return null
         }
         val fileContentUriStr = bodyPart.getString(REQUEST_BODY_KEY_URI)
         if (fileContentUriStr == null) {
-          ResponseUtil.onRequestError(
-              reactApplicationContext, requestId, "Body must have a valid file uri", null)
+          NetworkEventUtil.onRequestError(
+              reactApplicationContext,
+              requestId,
+              "Body must have a valid file uri",
+              null,
+          )
           return null
         }
         val fileInputStream =
             RequestBodyUtil.getFileInputStream(getReactApplicationContext(), fileContentUriStr)
         if (fileInputStream == null) {
-          ResponseUtil.onRequestError(
+          NetworkEventUtil.onRequestError(
               reactApplicationContext,
               requestId,
               "Could not retrieve file for uri $fileContentUriStr",
-              null)
+              null,
+          )
           return null
         }
         multipartBuilder.addPart(headers, RequestBodyUtil.create(partContentType, fileInputStream))
       } else {
-        ResponseUtil.onRequestError(
-            reactApplicationContext, requestId, "Unrecognized FormData part.", null)
+        NetworkEventUtil.onRequestError(
+            reactApplicationContext,
+            requestId,
+            "Unrecognized FormData part.",
+            null,
+        )
       }
     }
     return multipartBuilder
@@ -818,7 +939,7 @@ public class NetworkingModule(
     @JvmStatic
     public fun setCustomClientBuilder(
         ccb: com.facebook.react.modules.network.CustomClientBuilder?
-    ): Unit {
+    ) {
       customClientBuilder = ccb
     }
 
@@ -827,20 +948,5 @@ public class NetworkingModule(
     }
 
     private fun shouldDispatch(now: Long, last: Long): Boolean = last + CHUNK_TIMEOUT_NS < now
-
-    private fun translateHeaders(headers: Headers): WritableMap {
-      val responseHeaders = Bundle()
-      for (i in 0..<headers.size()) {
-        val headerName = headers.name(i)
-        // multiple values for the same header
-        if (responseHeaders.containsKey(headerName)) {
-          responseHeaders.putString(
-              headerName, "${responseHeaders.getString(headerName)}, ${headers.value(i)}")
-        } else {
-          responseHeaders.putString(headerName, headers.value(i))
-        }
-      }
-      return Arguments.fromBundle(responseHeaders)
-    }
   }
 }