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

Commit b2f960d1 authored by Hugo Benichi's avatar Hugo Benichi
Browse files

Captive portal: fix probe urls at network creation 

This patch changes how url configuration values for captive portal
detection are read: instead of re-reading the settings at every probe
detection, the settings are read once when the NetworkMonitor associated
to a network is created.

If the settings are updated, the new values are picked up by new
networks, but not by existing networks. Since captive portal detection
is most important when joining WiFi networks, the newest available
settings values will still be used.

This change prepares introducing further changes for rotating probe
urls. Especially it helps making the detection strategy stateful with
respect to the lifecycle of a single NetworkMonitor.

Test: built, flashed, tested manually with various portal networks
Bug: 36532213
Merged-In: I71cc8bb8b996462f27f50798d67bceee5ffb898d

(cherry picked from commit b03272c9)

Change-Id: I33ac2b52fea6099969c3fec9b60ba0131f9d672d
parent 19e3cbdb

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

+19 −11
Original line number Diff line number Diff line
@@ -259,6 +259,12 @@ public class NetworkMonitor extends StateMachine { 
    // This variable is set before transitioning to the mCaptivePortalState.

    private CaptivePortalProbeResult mLastPortalProbeResult = CaptivePortalProbeResult.FAILED;



    // Configuration values for captive portal detection probes.

    private final String mCaptivePortalUserAgent;

    private final URL mCaptivePortalHttpsUrl;

    private final URL mCaptivePortalHttpUrl;

    private final URL mCaptivePortalFallbackUrl;



    public NetworkMonitor(Context context, Handler handler, NetworkAgentInfo networkAgentInfo,

            NetworkRequest defaultRequest) {

        this(context, handler, networkAgentInfo, defaultRequest, new IpConnectivityLog());
@@ -293,6 +299,11 @@ public class NetworkMonitor extends StateMachine { 
        mUseHttps = Settings.Global.getInt(mContext.getContentResolver(),

                Settings.Global.CAPTIVE_PORTAL_USE_HTTPS, 1) == 1;



        mCaptivePortalUserAgent = getCaptivePortalUserAgent(context);

        mCaptivePortalHttpsUrl = makeURL(getCaptivePortalServerHttpsUrl(context));

        mCaptivePortalHttpUrl = makeURL(getCaptivePortalServerHttpUrl(context));

        mCaptivePortalFallbackUrl = makeURL(getCaptivePortalFallbackUrl(context));



        start();

    }
@@ -676,7 +687,10 @@ public class NetworkMonitor extends StateMachine { 
            return new CaptivePortalProbeResult(204);

        }



        URL pacUrl = null, httpsUrl = null, httpUrl = null, fallbackUrl = null;

        URL pacUrl = null;

        URL httpsUrl = mCaptivePortalHttpsUrl;

        URL httpUrl = mCaptivePortalHttpUrl;

        URL fallbackUrl = mCaptivePortalFallbackUrl;



        // On networks with a PAC instead of fetching a URL that should result in a 204

        // response, we instead simply fetch the PAC script.  This is done for a few reasons:
@@ -703,14 +717,9 @@ public class NetworkMonitor extends StateMachine { 
            }

        }



        if (pacUrl == null) {

            httpsUrl = makeURL(getCaptivePortalServerHttpsUrl(mContext));

            httpUrl = makeURL(getCaptivePortalServerHttpUrl(mContext));

            fallbackUrl = makeURL(getCaptivePortalFallbackUrl(mContext));

            if (httpUrl == null || httpsUrl == null) {

        if ((pacUrl == null) && (httpUrl == null || httpsUrl == null)) {

            return CaptivePortalProbeResult.FAILED;

        }

        }



        long startTime = SystemClock.elapsedRealtime();
@@ -790,9 +799,8 @@ public class NetworkMonitor extends StateMachine { 
            urlConnection.setConnectTimeout(SOCKET_TIMEOUT_MS);

            urlConnection.setReadTimeout(SOCKET_TIMEOUT_MS);

            urlConnection.setUseCaches(false);

            final String userAgent = getCaptivePortalUserAgent(mContext);

            if (userAgent != null) {

                urlConnection.setRequestProperty("User-Agent", userAgent);

            if (mCaptivePortalUserAgent != null) {

                urlConnection.setRequestProperty("User-Agent", mCaptivePortalUserAgent);

            }

            // cannot read request header after connection

            String requestHeader = urlConnection.getRequestProperties().toString();