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

Commit 1335aea3 authored by Junyu Lai's avatar Junyu Lai Committed by Automerger Merge Worker
Browse files

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

Merge "[VCN07] Bypass VCN for non-internet app accessible cellular services" am: 4cfe6d8f am: 711ce9c7 am: 54319cbd am: 5b4c4fea

Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1597954

MUST ONLY BE SUBMITTED BY AUTOMERGER

Change-Id: I147e75d829435b17242e8b87417f9bd63016a8cc
parents ce83b58a 5b4c4fea

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;
@@ -1248,6 +1249,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;
@@ -1262,6 +1264,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
@@ -2800,6 +2800,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(),










@@ -4147,6 +4151,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, mCsHandlerThread);










@@ -6051,6 +6056,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)









@@ -6059,6 +6065,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 */);










@@ -7751,19 +7758,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);