@onekeyfe/react-native-network-throttle 3.0.84-alpha.0 → 3.0.86

package/android/build.gradle

@@ -62,4 +62,5 @@ def kotlin_version = getExtOrDefault("kotlinVersion")
 dependencies {
   implementation "com.facebook.react:react-android"
   implementation "org.jetbrains.kotlin:kotlin-stdlib:$kotlin_version"
+  testImplementation "junit:junit:4.13.2"
 }

package/android/src/main/java/com/onekeyfe/reactnativenetworkthrottle/NetworkThrottle.kt

@@ -140,7 +140,7 @@ internal object NetworkThrottle {
      * patterns the desktop app installs, so both platforms throttle the same
      * traffic.
      */
-    private fun matchesHost(host: String, pattern: String): Boolean =
+    internal fun matchesHost(host: String, pattern: String): Boolean =
         if (pattern.startsWith("*.")) {
             host.endsWith(pattern.substring(1))
         } else {

package/android/src/test/java/com/onekeyfe/reactnativenetworkthrottle/NetworkThrottleHostMatchingTest.kt

@@ -0,0 +1,46 @@
+package com.onekeyfe.reactnativenetworkthrottle
+
+import org.junit.Assert.assertFalse
+import org.junit.Assert.assertTrue
+import org.junit.Test
+
+class NetworkThrottleHostMatchingTest {
+
+  @Test
+  fun matchesSubDomainsAtAnyDepth() {
+    assertTrue(NetworkThrottle.matchesHost("wallet.onekeycn.com", "*.onekeycn.com"))
+    assertTrue(NetworkThrottle.matchesHost("swap.onekeycn.com", "*.onekeycn.com"))
+    assertTrue(NetworkThrottle.matchesHost("a.b.onekeycn.com", "*.onekeycn.com"))
+    assertTrue(NetworkThrottle.matchesHost("uni.onekey-asset.com", "*.onekey-asset.com"))
+  }
+
+  @Test
+  fun matchesExactHosts() {
+    assertTrue(NetworkThrottle.matchesHost("app-assets.onekey.so", "app-assets.onekey.so"))
+    assertFalse(NetworkThrottle.matchesHost("other.onekey.so", "app-assets.onekey.so"))
+  }
+
+  @Test
+  fun doesNotMatchTheBareApex() {
+    // The desktop URLPattern wildcard behaves the same way, so a bare apex must
+    // stay untouched on both platforms.
+    assertFalse(NetworkThrottle.matchesHost("onekeycn.com", "*.onekeycn.com"))
+    assertFalse(NetworkThrottle.matchesHost("onekey-asset.com", "*.onekey-asset.com"))
+  }
+
+  @Test
+  fun doesNotMatchLookAlikeHosts() {
+    // A suffix check without the leading dot would wrongly match these, which
+    // would throttle traffic that is not OneKey's.
+    assertFalse(NetworkThrottle.matchesHost("evil-onekeycn.com", "*.onekeycn.com"))
+    assertFalse(NetworkThrottle.matchesHost("notonekeycn.com", "*.onekeycn.com"))
+  }
+
+  @Test
+  fun doesNotMatchUnrelatedHosts() {
+    for (host in listOf("mainnet.infura.io", "api.hyperliquid.xyz", "localhost", "127.0.0.1")) {
+      assertFalse(NetworkThrottle.matchesHost(host, "*.onekeycn.com"))
+      assertFalse(NetworkThrottle.matchesHost(host, "app-assets.onekey.so"))
+    }
+  }
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@onekeyfe/react-native-network-throttle",
-  "version": "3.0.84-alpha.0",
+  "version": "3.0.86",
   "description": "react-native-network-throttle",
   "main": "./lib/module/index.js",
   "types": "./lib/typescript/src/index.d.ts",