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

Commit 79a6da3a authored by Chiachang Wang's avatar Chiachang Wang
Browse files

Move Setting constants for NetworkStack 

From mainline perspective, we should use android flag api
instead of using Settings. Thus, move the definitions into
NetworkStack and apply new flag design.

Bug: 123167629
Test: atest NetworkStackTests
Change-Id: I9602263f0bff5d8e942bd652de69ccfcb3034a2f
parent f51f15e2

src/android/net/util/NetworkStackUtils.java

+28 −0
Original line number Diff line number Diff line
@@ -34,6 +34,34 @@ public class NetworkStackUtils { 
    // TODO: Refer to DeviceConfig definition.

    public static final String NAMESPACE_CONNECTIVITY = "connectivity";



    /**

     * A list of captive portal detection specifications used in addition to the fallback URLs.

     * Each spec has the format url@@/@@statusCodeRegex@@/@@contentRegex. Specs are separated

     * by "@@,@@".

     */

    public static final String CAPTIVE_PORTAL_FALLBACK_PROBE_SPECS =

            "captive_portal_fallback_probe_specs";



    /**

     * A comma separated list of URLs used for captive portal detection in addition to the

     * fallback HTTP url associated with the CAPTIVE_PORTAL_FALLBACK_URL settings.

     */

    public static final String CAPTIVE_PORTAL_OTHER_FALLBACK_URLS =

            "captive_portal_other_fallback_urls";



    /**

     * Which User-Agent string to use in the header of the captive portal detection probes.

     * The User-Agent field is unset when this setting has no value (HttpUrlConnection default).

     */

    public static final String CAPTIVE_PORTAL_USER_AGENT = "captive_portal_user_agent";



    /**

     * Whether to use HTTPS for network validation. This is enabled by default and the setting

     * needs to be set to 0 to disable it. This setting is a misnomer because captive portals

     * don't actually use HTTPS, but it's consistent with the other settings.

     */

    public static final String CAPTIVE_PORTAL_USE_HTTPS = "captive_portal_use_https";



    static {

        System.loadLibrary("networkstackutilsjni");

    }

src/com/android/server/connectivity/NetworkMonitor.java

+13 −7
Original line number Diff line number Diff line
@@ -43,6 +43,10 @@ import static android.net.util.DataStallUtils.DEFAULT_DATA_STALL_EVALUATION_TYPE 
import static android.net.util.DataStallUtils.DEFAULT_DATA_STALL_MIN_EVALUATE_TIME_MS;

import static android.net.util.DataStallUtils.DEFAULT_DATA_STALL_VALID_DNS_TIME_THRESHOLD_MS;

import static android.net.util.DataStallUtils.DEFAULT_DNS_LOG_SIZE;

import static android.net.util.NetworkStackUtils.CAPTIVE_PORTAL_FALLBACK_PROBE_SPECS;

import static android.net.util.NetworkStackUtils.CAPTIVE_PORTAL_OTHER_FALLBACK_URLS;

import static android.net.util.NetworkStackUtils.CAPTIVE_PORTAL_USER_AGENT;

import static android.net.util.NetworkStackUtils.CAPTIVE_PORTAL_USE_HTTPS;

import static android.net.util.NetworkStackUtils.NAMESPACE_CONNECTIVITY;

import static android.net.util.NetworkStackUtils.isEmpty;
@@ -1170,7 +1174,8 @@ public class NetworkMonitor extends StateMachine { 
    }



    private boolean getUseHttpsValidation() {

        return mDependencies.getSetting(mContext, Settings.Global.CAPTIVE_PORTAL_USE_HTTPS, 1) == 1;

        return mDependencies.getDeviceConfigPropertyInt(NAMESPACE_CONNECTIVITY,

                CAPTIVE_PORTAL_USE_HTTPS, 1) == 1;

    }



    private String getCaptivePortalServerHttpsUrl() {
@@ -1223,8 +1228,8 @@ public class NetworkMonitor extends StateMachine { 


            final URL[] settingProviderUrls;

            if (!TextUtils.isEmpty(firstUrl)) {

                final String otherUrls = mDependencies.getSetting(mContext,

                        Settings.Global.CAPTIVE_PORTAL_OTHER_FALLBACK_URLS, "");

                final String otherUrls = mDependencies.getDeviceConfigProperty(

                        NAMESPACE_CONNECTIVITY, CAPTIVE_PORTAL_OTHER_FALLBACK_URLS, "");

                // otherUrls may be empty, but .split() ignores trailing empty strings

                final String separator = ",";

                final String[] urls = (firstUrl + separator + otherUrls).split(separator);
@@ -1244,8 +1249,9 @@ public class NetworkMonitor extends StateMachine { 


    private CaptivePortalProbeSpec[] makeCaptivePortalFallbackProbeSpecs() {

        try {

            final String settingsValue = mDependencies.getSetting(

                    mContext, Settings.Global.CAPTIVE_PORTAL_FALLBACK_PROBE_SPECS, null);

            final String settingsValue = mDependencies.getDeviceConfigProperty(

                    NAMESPACE_CONNECTIVITY, CAPTIVE_PORTAL_FALLBACK_PROBE_SPECS, null);



            final CaptivePortalProbeSpec[] emptySpecs = new CaptivePortalProbeSpec[0];

            final CaptivePortalProbeSpec[] providerValue = TextUtils.isEmpty(settingsValue)

                    ? emptySpecs
@@ -1339,8 +1345,8 @@ public class NetworkMonitor extends StateMachine { 
    }



    private String getCaptivePortalUserAgent() {

        return mDependencies.getSetting(mContext,

                Settings.Global.CAPTIVE_PORTAL_USER_AGENT, DEFAULT_USER_AGENT);

        return mDependencies.getDeviceConfigProperty(NAMESPACE_CONNECTIVITY,

                CAPTIVE_PORTAL_USER_AGENT, DEFAULT_USER_AGENT);

    }



    private URL nextFallbackUrl() {

tests/src/com/android/server/connectivity/NetworkMonitorTest.java

+8 −5
Original line number Diff line number Diff line
@@ -26,6 +26,9 @@ import static android.net.util.DataStallUtils.CONFIG_DATA_STALL_EVALUATION_TYPE; 
import static android.net.util.DataStallUtils.CONFIG_DATA_STALL_MIN_EVALUATE_INTERVAL;

import static android.net.util.DataStallUtils.CONFIG_DATA_STALL_VALID_DNS_TIME_THRESHOLD;

import static android.net.util.DataStallUtils.DATA_STALL_EVALUATION_TYPE_DNS;

import static android.net.util.NetworkStackUtils.CAPTIVE_PORTAL_FALLBACK_PROBE_SPECS;

import static android.net.util.NetworkStackUtils.CAPTIVE_PORTAL_OTHER_FALLBACK_URLS;

import static android.net.util.NetworkStackUtils.CAPTIVE_PORTAL_USE_HTTPS;



import static junit.framework.Assert.assertEquals;

import static junit.framework.Assert.assertFalse;
@@ -159,7 +162,7 @@ public class NetworkMonitorTest { 
        when(mDependencies.getRandom()).thenReturn(mRandom);

        when(mDependencies.getSetting(any(), eq(Settings.Global.CAPTIVE_PORTAL_MODE), anyInt()))

                .thenReturn(Settings.Global.CAPTIVE_PORTAL_MODE_PROMPT);

        when(mDependencies.getSetting(any(), eq(Settings.Global.CAPTIVE_PORTAL_USE_HTTPS),

        when(mDependencies.getDeviceConfigPropertyInt(any(), eq(CAPTIVE_PORTAL_USE_HTTPS),

                anyInt())).thenReturn(1);

        when(mDependencies.getSetting(any(), eq(Settings.Global.CAPTIVE_PORTAL_HTTP_URL), any()))

                .thenReturn(TEST_HTTP_URL);
@@ -682,13 +685,13 @@ public class NetworkMonitorTest { 
    }



    private void setOtherFallbackUrls(String urls) {

        when(mDependencies.getSetting(any(),

                eq(Settings.Global.CAPTIVE_PORTAL_OTHER_FALLBACK_URLS), any())).thenReturn(urls);

        when(mDependencies.getDeviceConfigProperty(any(),

                eq(CAPTIVE_PORTAL_OTHER_FALLBACK_URLS), any())).thenReturn(urls);

    }



    private void setFallbackSpecs(String specs) {

        when(mDependencies.getSetting(any(),

                eq(Settings.Global.CAPTIVE_PORTAL_FALLBACK_PROBE_SPECS), any())).thenReturn(specs);

        when(mDependencies.getDeviceConfigProperty(any(),

                eq(CAPTIVE_PORTAL_FALLBACK_PROBE_SPECS), any())).thenReturn(specs);

    }



    private void setCaptivePortalMode(int mode) {