Revert "Check if VPN network has internet"

This reverts commit 6df8613004.

app/src/main/java/eu/faircode/email/ConnectionHelper.java

@@ -300,10 +300,9 @@ public class ConnectionHelper {
 
         if (caps.hasCapability(NetworkCapabilities.NET_CAPABILITY_NOT_VPN)) {
             if (!caps.hasCapability(NetworkCapabilities.NET_CAPABILITY_INTERNET)) {
-                Log.i("isMetered: active no internet");
+                Log.i("isMetered: no internet");
                 return null;
             }
-
             boolean captive = caps.hasCapability(NetworkCapabilities.NET_CAPABILITY_CAPTIVE_PORTAL);
             if ((require_validated || (require_validated_captive && captive)) &&
                     Build.VERSION.SDK_INT >= Build.VERSION_CODES.M &&
@@ -331,23 +330,17 @@ public class ConnectionHelper {
                 return null;
         }
 
-        if (caps.hasCapability(NetworkCapabilities.NET_CAPABILITY_NOT_VPN)) {
+        if (standalone_vpn ||
+                caps.hasCapability(NetworkCapabilities.NET_CAPABILITY_NOT_VPN)) {
             // NET_CAPABILITY_NOT_METERED is unreliable on older Android versions
             boolean metered = cm.isActiveNetworkMetered();
             Log.i("isMetered: active not VPN metered=" + metered);
             return metered;
         }
 
-        // Active network is VPN network
-
         Network[] networks = cm.getAllNetworks();
-        if (standalone_vpn || (networks != null && networks.length == 1)) {
+        if (networks != null && networks.length == 1) {
             // Standalone VPN
-            if (!caps.hasCapability(NetworkCapabilities.NET_CAPABILITY_INTERNET)) {
-                Log.i("isMetered: VPN no internet");
-                return null;
-            }
-
             boolean metered = cm.isActiveNetworkMetered();
             Log.i("isMetered: active VPN metered=" + metered);
             return metered;