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

Commit 52396dad authored by Lorenzo Colitti's avatar Lorenzo Colitti Committed by Automerger Merge Worker
Browse files

Merge "Have a new method in NetworkAgentConfig.Builder to set allowBypass" am:... 

Merge "Have a new method in NetworkAgentConfig.Builder to set allowBypass" am: 14f7c027 am: 07a92ecb am: 702f7447

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

Change-Id: Ibc6553b1471ee8e0ab362979ba9525c2e328a275
parents 41b860ee 702f7447

packages/Connectivity/framework/api/module-lib-current.txt

+2 −0
Original line number Diff line number Diff line
@@ -37,9 +37,11 @@ package android.net { 


  public final class NetworkAgentConfig implements android.os.Parcelable {

    method @Nullable public String getSubscriberId();

    method public boolean isBypassableVpn();

  }



  public static final class NetworkAgentConfig.Builder {

    method @NonNull public android.net.NetworkAgentConfig.Builder setBypassableVpn(boolean);

    method @NonNull public android.net.NetworkAgentConfig.Builder setSubscriberId(@Nullable String);

  }

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

+23 −0
Original line number Diff line number Diff line
@@ -63,6 +63,16 @@ public final class NetworkAgentConfig implements Parcelable { 
        return explicitlySelected;

    }



    /**

     * @return whether this VPN connection can be bypassed by the apps.

     *

     * @hide

     */

    @SystemApi(client = MODULE_LIBRARIES)

    public boolean isBypassableVpn() {

        return allowBypass;

    }



    /**

     * Set if the user desires to use this network even if it is unvalidated. This field has meaning

     * only if {@link explicitlySelected} is true. If it is, this field must also be set to the
@@ -381,6 +391,19 @@ public final class NetworkAgentConfig implements Parcelable { 
            return this;

        }



        /**

         * Sets whether the apps can bypass the VPN connection.

         *

         * @return this builder, to facilitate chaining.

         * @hide

         */

        @NonNull

        @SystemApi(client = MODULE_LIBRARIES)

        public Builder setBypassableVpn(boolean allowBypass) {

            mConfig.allowBypass = allowBypass;

            return this;

        }



        /**

         * Returns the constructed {@link NetworkAgentConfig} object.

         */

services/core/java/com/android/server/connectivity/Vpn.java

+3 −2
Original line number Diff line number Diff line
@@ -1248,8 +1248,9 @@ public class Vpn { 
        mLegacyState = LegacyVpnInfo.STATE_CONNECTING;

        updateState(DetailedState.CONNECTING, "agentConnect");



        NetworkAgentConfig networkAgentConfig = new NetworkAgentConfig.Builder().build();

        networkAgentConfig.allowBypass = mConfig.allowBypass && !mLockdown;

        final NetworkAgentConfig networkAgentConfig = new NetworkAgentConfig.Builder()

                .setBypassableVpn(mConfig.allowBypass && !mLockdown)

                .build();



        capsBuilder.setOwnerUid(mOwnerUID);

        capsBuilder.setAdministratorUids(new int[] {mOwnerUID});