expo-modules-core 1.12.17 → 1.12.18

package/CHANGELOG.md

@@ -10,6 +10,13 @@
 
 ### 💡 Others
 
+## 1.12.18 — 2024-06-28
+
+### 🐛 Bug fixes
+
+- Fixed `RCTTriggerReloadCommandListeners` not found build error on iOS. ([#30014](https://github.com/expo/expo/pull/30014) by [@kudo](https://github.com/kudo))
+- Fixed blocking SSE responses from network interceptor on Android. ([#30062](https://github.com/expo/expo/pull/30062) by [@kudo](https://github.com/kudo))
+
 ## 1.12.17 — 2024-06-27
 
 ### 🐛 Bug fixes

package/android/build.gradle

@@ -1,7 +1,7 @@
 apply plugin: 'com.android.library'
 
 group = 'host.exp.exponent'
-version = '1.12.17'
+version = '1.12.18'
 
 def expoModulesCorePlugin = new File(project(":expo-modules-core").projectDir.absolutePath, "ExpoModulesCorePlugin.gradle")
 apply from: expoModulesCorePlugin
@@ -63,7 +63,7 @@ android {
   defaultConfig {
     consumerProguardFiles 'proguard-rules.pro'
     versionCode 1
-    versionName "1.12.17"
+    versionName "1.12.18"
     buildConfigField "boolean", "IS_NEW_ARCHITECTURE_ENABLED", isNewArchitectureEnabled.toString()
 
     testInstrumentationRunner "expo.modules.TestRunner"

package/android/src/main/java/expo/modules/kotlin/devtools/ExpoNetworkInspectOkHttpInterceptors.kt

@@ -36,7 +36,7 @@ class ExpoNetworkInspectOkHttpNetworkInterceptor : Interceptor {
           it.priorResponse = response
         }
       } else {
-        val body = peekResponseBody(response)
+        val body = if (shouldParseBody(response)) peekResponseBody(response) else null
         delegate.didReceiveResponse(requestId, request, response, body)
         body?.close()
       }
@@ -123,3 +123,27 @@ internal fun peekResponseBody(
   buffer.write(source, minOf(byteCount, source.buffer.size))
   return buffer.asResponseBody(body.contentType(), buffer.size)
 }
+
+internal fun shouldParseBody(response: Response): Boolean {
+  // Check for Content-Type
+  val skipContentTypes = listOf(
+    "text/event-stream", // Server Sent Events
+    "text/x-component", // React Server Components
+    "audio", // Media might be streaming and not inspectable in DevTools
+    "video" // Media might be streaming and not inspectable in DevTools
+  )
+  val contentType = response.header("Content-Type") ?: ""
+  if (skipContentTypes.any { contentType.startsWith(it) }) {
+    return false
+  }
+
+  // HTTP 1.1 chunked encoding
+  val transferEncoding = response.header("Transfer-Encoding")
+  if ("chunked".equals(transferEncoding, ignoreCase = true)) {
+    return false
+  }
+
+  // If Content-Length is known to exceed the limit
+  val contentLength = response.header("Content-Length")?.toLong() ?: -1
+  return contentLength < 1 || contentLength <= ExpoNetworkInspectOkHttpNetworkInterceptor.MAX_BODY_SIZE
+}

package/ios/Core/Modules/CoreModule.swift

@@ -1,3 +1,7 @@
+// Copyright 2015-present 650 Industries. All rights reserved.
+
+import React
+
 // The core module that describes the `global.expo` object.
 internal final class CoreModule: Module {
   internal func definition() -> ModuleDefinition {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "expo-modules-core",
-  "version": "1.12.17",
+  "version": "1.12.18",
   "description": "The core of Expo Modules architecture",
   "main": "build/index.js",
   "types": "build/index.d.ts",
@@ -44,5 +44,5 @@
     "@testing-library/react-hooks": "^7.0.1",
     "expo-module-scripts": "^3.0.0"
   },
-  "gitHead": "09b2d97bbc0f70f7c811ff9b6c9ad8808c5ad84b"
+  "gitHead": "5e6538be9ebd87877867d0710029a64c883ca00e"
 }