Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit b453bb62 authored by junyulai's avatar junyulai
Browse files

[VCN07] Bypass VCN for non-internet app accessible cellular services 

Deduce the NET_CAPABILITY_NOT_VCN_MANAGED capability from other
capabilities and user intention, which includes:
   1. For the requests that don't have anything besides
      VCN_SUPPORTED_CAPABILITIES, add the NOT_VCN_MANAGED to
      allow the callers automatically utilize VCN networks
      if available.
   2. For the requests that explicitly add or remove
      NOT_VCN_MANAGED,  do not alter them to allow user fire
      request that suits their need.

Test: atest NetworkRequestTest#testBypassingVcnForNonInternetRequest
Bug: 175662146
Change-Id: I2876264cee14b624c89ba3b380027a8b521ad8ea
  (cherry-picked from aosp/1549817)
parent ea7513a9

packages/Connectivity/framework/src/android/net/NetworkRequest.java

+69 −0
Original line number Diff line number Diff line
@@ -16,6 +16,22 @@ 


package android.net;



import static android.net.NetworkCapabilities.NET_CAPABILITY_CAPTIVE_PORTAL;

import static android.net.NetworkCapabilities.NET_CAPABILITY_DUN;

import static android.net.NetworkCapabilities.NET_CAPABILITY_FOREGROUND;

import static android.net.NetworkCapabilities.NET_CAPABILITY_INTERNET;

import static android.net.NetworkCapabilities.NET_CAPABILITY_NOT_CONGESTED;

import static android.net.NetworkCapabilities.NET_CAPABILITY_NOT_METERED;

import static android.net.NetworkCapabilities.NET_CAPABILITY_NOT_RESTRICTED;

import static android.net.NetworkCapabilities.NET_CAPABILITY_NOT_ROAMING;

import static android.net.NetworkCapabilities.NET_CAPABILITY_NOT_SUSPENDED;

import static android.net.NetworkCapabilities.NET_CAPABILITY_NOT_VCN_MANAGED;

import static android.net.NetworkCapabilities.NET_CAPABILITY_NOT_VPN;

import static android.net.NetworkCapabilities.NET_CAPABILITY_PARTIAL_CONNECTIVITY;

import static android.net.NetworkCapabilities.NET_CAPABILITY_TEMPORARILY_NOT_METERED;

import static android.net.NetworkCapabilities.NET_CAPABILITY_TRUSTED;

import static android.net.NetworkCapabilities.NET_CAPABILITY_VALIDATED;



import android.annotation.NonNull;

import android.annotation.Nullable;

import android.annotation.RequiresPermission;
@@ -30,6 +46,8 @@ import android.os.Process; 
import android.text.TextUtils;

import android.util.proto.ProtoOutputStream;



import java.util.Arrays;

import java.util.List;

import java.util.Objects;

import java.util.Set;
@@ -154,8 +172,30 @@ public class NetworkRequest implements Parcelable { 
     * needed in terms of {@link NetworkCapabilities} features

     */

    public static class Builder {

        /**

         * Capabilities that are currently compatible with VCN networks.

         */

        private static final List<Integer> VCN_SUPPORTED_CAPABILITIES = Arrays.asList(

                NET_CAPABILITY_CAPTIVE_PORTAL,

                NET_CAPABILITY_DUN,

                NET_CAPABILITY_FOREGROUND,

                NET_CAPABILITY_INTERNET,

                NET_CAPABILITY_NOT_CONGESTED,

                NET_CAPABILITY_NOT_METERED,

                NET_CAPABILITY_NOT_RESTRICTED,

                NET_CAPABILITY_NOT_ROAMING,

                NET_CAPABILITY_NOT_SUSPENDED,

                NET_CAPABILITY_NOT_VPN,

                NET_CAPABILITY_PARTIAL_CONNECTIVITY,

                NET_CAPABILITY_TEMPORARILY_NOT_METERED,

                NET_CAPABILITY_TRUSTED,

                NET_CAPABILITY_VALIDATED);



        private final NetworkCapabilities mNetworkCapabilities;



        // A boolean that represents the user modified NOT_VCN_MANAGED capability.

        private boolean mModifiedNotVcnManaged = false;



        /**

         * Default constructor for Builder.

         */
@@ -177,6 +217,7 @@ public class NetworkRequest implements Parcelable { 
            // maybeMarkCapabilitiesRestricted() doesn't add back.

            final NetworkCapabilities nc = new NetworkCapabilities(mNetworkCapabilities);

            nc.maybeMarkCapabilitiesRestricted();

            deduceNotVcnManagedCapability(nc);

            return new NetworkRequest(nc, ConnectivityManager.TYPE_NONE,

                    ConnectivityManager.REQUEST_ID_UNSET, Type.NONE);

        }
@@ -193,6 +234,9 @@ public class NetworkRequest implements Parcelable { 
         */

        public Builder addCapability(@NetworkCapabilities.NetCapability int capability) {

            mNetworkCapabilities.addCapability(capability);

            if (capability == NetworkCapabilities.NET_CAPABILITY_NOT_VCN_MANAGED) {

                mModifiedNotVcnManaged = true;

            }

            return this;

        }
@@ -204,6 +248,9 @@ public class NetworkRequest implements Parcelable { 
         */

        public Builder removeCapability(@NetworkCapabilities.NetCapability int capability) {

            mNetworkCapabilities.removeCapability(capability);

            if (capability == NetworkCapabilities.NET_CAPABILITY_NOT_VCN_MANAGED) {

                mModifiedNotVcnManaged = true;

            }

            return this;

        }
@@ -261,6 +308,9 @@ public class NetworkRequest implements Parcelable { 
        @NonNull

        public Builder clearCapabilities() {

            mNetworkCapabilities.clearAll();

            // If the caller explicitly clear all capabilities, the NOT_VCN_MANAGED capabilities

            // should not be add back later.

            mModifiedNotVcnManaged = true;

            return this;

        }
@@ -380,6 +430,25 @@ public class NetworkRequest implements Parcelable { 
            mNetworkCapabilities.setSignalStrength(signalStrength);

            return this;

        }



        /**

         * Deduce the NET_CAPABILITY_NOT_VCN_MANAGED capability from other capabilities

         * and user intention, which includes:

         *   1. For the requests that don't have anything besides

         *      {@link #VCN_SUPPORTED_CAPABILITIES}, add the NET_CAPABILITY_NOT_VCN_MANAGED to

         *      allow the callers automatically utilize VCN networks if available.

         *   2. For the requests that explicitly add or remove NET_CAPABILITY_NOT_VCN_MANAGED,

         *      do not alter them to allow user fire request that suits their need.

         *

         * @hide

         */

        private void deduceNotVcnManagedCapability(final NetworkCapabilities nc) {

            if (mModifiedNotVcnManaged) return;

            for (final int cap : nc.getCapabilities()) {

                if (!VCN_SUPPORTED_CAPABILITIES.contains(cap)) return;

            }

            nc.addCapability(NET_CAPABILITY_NOT_VCN_MANAGED);

        }

    }



    // implement the Parcelable interface

services/core/java/com/android/server/ConnectivityService.java

+3 −0
Original line number Diff line number Diff line
@@ -44,6 +44,7 @@ import static android.net.NetworkCapabilities.NET_CAPABILITY_NOT_METERED; 
import static android.net.NetworkCapabilities.NET_CAPABILITY_NOT_RESTRICTED;

import static android.net.NetworkCapabilities.NET_CAPABILITY_NOT_ROAMING;

import static android.net.NetworkCapabilities.NET_CAPABILITY_NOT_SUSPENDED;

import static android.net.NetworkCapabilities.NET_CAPABILITY_NOT_VCN_MANAGED;

import static android.net.NetworkCapabilities.NET_CAPABILITY_NOT_VPN;

import static android.net.NetworkCapabilities.NET_CAPABILITY_OEM_PAID;

import static android.net.NetworkCapabilities.NET_CAPABILITY_OEM_PRIVATE;
@@ -1231,6 +1232,7 @@ public class ConnectivityService extends IConnectivityManager.Stub 
    private static NetworkCapabilities createDefaultNetworkCapabilitiesForUid(int uid) {

        final NetworkCapabilities netCap = new NetworkCapabilities();

        netCap.addCapability(NET_CAPABILITY_INTERNET);

        netCap.addCapability(NET_CAPABILITY_NOT_VCN_MANAGED);

        netCap.removeCapability(NET_CAPABILITY_NOT_VPN);

        netCap.setSingleUid(uid);

        return netCap;
@@ -1245,6 +1247,7 @@ public class ConnectivityService extends IConnectivityManager.Stub 
            int transportType, NetworkRequest.Type type) {

        final NetworkCapabilities netCap = new NetworkCapabilities();

        netCap.addCapability(NET_CAPABILITY_INTERNET);

        netCap.addCapability(NET_CAPABILITY_NOT_VCN_MANAGED);

        netCap.setRequestorUidAndPackageName(Process.myUid(), mContext.getPackageName());

        if (transportType > TYPE_NONE) {

            netCap.addTransportType(transportType);

tests/net/integration/util/com/android/server/NetworkAgentWrapper.java

+2 −0
Original line number Diff line number Diff line
@@ -17,6 +17,7 @@ 
package com.android.server;



import static android.net.NetworkCapabilities.NET_CAPABILITY_NOT_SUSPENDED;

import static android.net.NetworkCapabilities.NET_CAPABILITY_NOT_VCN_MANAGED;

import static android.net.NetworkCapabilities.NET_CAPABILITY_NOT_VPN;

import static android.net.NetworkCapabilities.TRANSPORT_CELLULAR;

import static android.net.NetworkCapabilities.TRANSPORT_ETHERNET;
@@ -84,6 +85,7 @@ public class NetworkAgentWrapper implements TestableNetworkCallback.HasNetwork { 
        final String typeName = ConnectivityManager.getNetworkTypeName(type);

        mNetworkCapabilities = (ncTemplate != null) ? ncTemplate : new NetworkCapabilities();

        mNetworkCapabilities.addCapability(NET_CAPABILITY_NOT_SUSPENDED);

        mNetworkCapabilities.addCapability(NET_CAPABILITY_NOT_VCN_MANAGED);

        mNetworkCapabilities.addTransportType(transport);

        switch (transport) {

            case TRANSPORT_ETHERNET:

tests/net/java/com/android/server/ConnectivityServiceTest.java

+10 −9
Original line number Diff line number Diff line
@@ -2761,6 +2761,10 @@ public class ConnectivityServiceTest { 


        NetworkCapabilities filter = new NetworkCapabilities();

        filter.addCapability(capability);

        // Add NOT_VCN_MANAGED capability into filter unconditionally since some request will add

        // NOT_VCN_MANAGED automatically but not for NetworkCapabilities,

        // see {@code NetworkCapabilities#deduceNotVcnManagedCapability} for more details.

        filter.addCapability(NET_CAPABILITY_NOT_VCN_MANAGED);

        final HandlerThread handlerThread = new HandlerThread("testNetworkFactoryRequests");

        handlerThread.start();

        final MockNetworkFactory testFactory = new MockNetworkFactory(handlerThread.getLooper(),
@@ -4120,6 +4124,7 @@ public class ConnectivityServiceTest { 
        handlerThread.start();

        NetworkCapabilities filter = new NetworkCapabilities()

                .addTransportType(TRANSPORT_CELLULAR)

                .addCapability(NET_CAPABILITY_NOT_VCN_MANAGED)

                .addCapability(NET_CAPABILITY_INTERNET);

        final MockNetworkFactory testFactory = new MockNetworkFactory(handlerThread.getLooper(),

                mServiceContext, "testFactory", filter);
@@ -6023,6 +6028,7 @@ public class ConnectivityServiceTest { 
                .addTransportType(TRANSPORT_CELLULAR)

                .addCapability(NET_CAPABILITY_INTERNET)

                .addCapability(NET_CAPABILITY_NOT_CONGESTED)

                .addCapability(NET_CAPABILITY_NOT_VCN_MANAGED)

                .setLinkDownstreamBandwidthKbps(10);

        final NetworkCapabilities wifiNc = new NetworkCapabilities()

                .addTransportType(TRANSPORT_WIFI)
@@ -6031,6 +6037,7 @@ public class ConnectivityServiceTest { 
                .addCapability(NET_CAPABILITY_NOT_ROAMING)

                .addCapability(NET_CAPABILITY_NOT_CONGESTED)

                .addCapability(NET_CAPABILITY_NOT_SUSPENDED)

                .addCapability(NET_CAPABILITY_NOT_VCN_MANAGED)

                .setLinkUpstreamBandwidthKbps(20);

        mCellNetworkAgent.setNetworkCapabilities(cellNc, true /* sendToConnectivityService */);

        mWiFiNetworkAgent.setNetworkCapabilities(wifiNc, true /* sendToConnectivityService */);
@@ -7718,19 +7725,13 @@ public class ConnectivityServiceTest { 
            mWiFiNetworkAgent.removeCapability(testCap);

            callbackWithCap.expectAvailableCallbacksValidated(mCellNetworkAgent);

            callbackWithoutCap.expectCapabilitiesWithout(testCap, mWiFiNetworkAgent);

            // TODO: Test default network changes for NOT_VCN_MANAGED once the default request has

            //  it.

            if (testCap == NET_CAPABILITY_TRUSTED) {

            verify(mMockNetd).networkSetDefault(eq(mCellNetworkAgent.getNetwork().netId));

            reset(mMockNetd);

            }



            mCellNetworkAgent.removeCapability(testCap);

            callbackWithCap.expectCallback(CallbackEntry.LOST, mCellNetworkAgent);

            callbackWithoutCap.assertNoCallback();

            if (testCap == NET_CAPABILITY_TRUSTED) {

            verify(mMockNetd).networkClearDefault();

            }



            mCm.unregisterNetworkCallback(callbackWithCap);

            mCm.unregisterNetworkCallback(callbackWithoutCap);