Mark VPN as non-metered

httptoolkit · May 7, 2024 · c43315e · c43315e
1 parent 4ce7452
commit c43315e
Showing 1 changed file with 9 additions and 4 deletions.
diff --git a/app/src/main/java/tech/httptoolkit/android/ProxyVpnService.kt b/app/src/main/java/tech/httptoolkit/android/ProxyVpnService.kt
@@ -168,6 +168,15 @@ class ProxyVpnService : VpnService(), IProtectSocket {
         val vpnInterface = Builder()
             .addAddress(VPN_IP_ADDRESS, 32)
             .addRoute(ALL_ROUTES, 0)
+            .apply {
+                // On Android 10+ VPNs are assumed as metered by default, unless we explicitly
+                // specify otherwise:
+                if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.Q) {
+                    setMetered(false)
+                }
+            }
+            .setMtu(MAX_PACKET_LEN) // Limit the packet size to the buffer used by ProxyVpnRunnable
+            .setBlocking(true) // We use a blocking loop to read in ProxyVpnRunnable
             .apply {
                 if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.Q) {
                     // Where possible, we want to explicitly set the proxy in addition to
@@ -177,10 +186,6 @@ class ProxyVpnService : VpnService(), IProtectSocket {
                     setHttpProxy(ProxyInfo.buildDirectProxy(proxyConfig.ip, proxyConfig.port))
                 }
             }
-
-            .setMtu(MAX_PACKET_LEN) // Limit the packet size to the buffer used by ProxyVpnRunnable
-            .setBlocking(true) // We use a blocking loop to read in ProxyVpnRunnable
-
             .apply {
                 // We exclude ourselves from interception, so we can still make network requests
                 // separately, primarily because otherwise pinging with isReachable is recursive.