Commit 51cbd6b2 authored by Dan Ballard's avatar Dan Ballard Committed by Pier Angelo Vendrame
Browse files

TB 41878: [android] Add standalone Tor Bootstrap

parent b5177089
Loading
Loading
Loading
Loading
+2 −0
Original line number Diff line number Diff line
@@ -2190,6 +2190,8 @@ class GeckoEngine(
        installedExtension.registerTabHandler(webExtensionTabHandler, defaultSettings)
        onSuccess(installedExtension)
    }

    fun getTorIntegrationController() = runtime.getTorIntegrationController()
}

internal fun ContentBlockingController.LogEntry.BlockingData.hasBlockedCookies(): Boolean {
+11 −0
Original line number Diff line number Diff line
@@ -307,6 +307,11 @@ android.applicationVariants.configureEach { variant ->
        System.setProperty("nimbusFml",  gradle.mozconfig.substs.NIMBUS_FML)
    }

    def disableTor = false
    if (project.hasProperty("disableTor")) {
        disableTor = project.getProperty("disableTor")
    }

    project.logger.debug("----------------------------------------------")
    project.logger.debug("Variant name:      " + variant.name)
    project.logger.debug("Application ID:    " + [variant.applicationId, variant.buildType.applicationIdSuffix].findAll().join())
@@ -314,6 +319,9 @@ android.applicationVariants.configureEach { variant ->
    project.logger.debug("Flavor:            " + variant.flavorName)
    project.logger.debug("Telemetry enabled: " + !isDebugOrDCD)
    project.logger.debug("nimbusFml:         " + providers.gradleProperty("nimbusFml").getOrNull())
    project.logger.debug("Tor is disabled:   " + disableTor)

    buildConfigField "boolean", "DISABLE_TOR", "$disableTor"

    if (useReleaseVersioning) {
        // The Google Play Store does not allow multiple APKs for the same app that all have the
@@ -767,6 +775,9 @@ dependencies {
    androidTestUtil libs.androidx.test.orchestrator

    lintChecks project(':components:tooling-lint')

    // Tor Expert Bundle
    implementation files(gradle.mozconfig.substs.TOR_EXPERT_BUNDLE_AAR)
}

protobuf {
+9 −0
Original line number Diff line number Diff line
## torrc-defaults for Tor Browser for Android
AvoidDiskWrites 1
# (stderr|stdout|syslog|file FILENAME).
Log notice stdout
CookieAuthentication 1
DormantCanceledByStartup 1
ClientTransportPlugin meek_lite,obfs2,obfs3,obfs4,scramblesuit,webtunnel exec ./libLyrebird.so
ClientTransportPlugin snowflake exec ./libLyrebird.so
ClientTransportPlugin conjure exec ./libConjure.so -registerURL https://registration.refraction.network/api
+3 −1
Original line number Diff line number Diff line
@@ -417,6 +417,8 @@ open class FenixApplication : Application(), Provider, ThemeProvider {
        CoroutineScope(IO).launch {
            components.useCases.wallpaperUseCases.fetchCurrentWallpaperUseCase.invoke()
        }

        components.torController.start()
    }

    @OptIn(DelicateCoroutinesApi::class) // GlobalScope usage
@@ -1155,7 +1157,7 @@ open class FenixApplication : Application(), Provider, ThemeProvider {
            bookmarksSuggestion.set(settings.shouldShowBookmarkSuggestions)
            clipboardSuggestionsEnabled.set(settings.shouldShowClipboardSuggestions)
            voiceSearchEnabled.set(settings.shouldShowVoiceSearch)
            openLinksInAppEnabled.set(settings.openLinksInExternalApp)
            // openLinksInAppEnabled.set(settings.openLinksInExternalApp)
            signedInSync.set(settings.signedInFxaAccount)
            isolatedContentProcessesEnabled.set(settings.isIsolatedProcessEnabled)
            appZygoteIsolatedContentProcessesEnabled.set(settings.isAppZygoteEnabled)
+31 −7
Original line number Diff line number Diff line
@@ -4,6 +4,7 @@

package org.mozilla.fenix

import android.annotation.SuppressLint
import android.app.assist.AssistContent
import android.app.PendingIntent
import android.content.ComponentName
@@ -186,6 +187,7 @@ import org.mozilla.fenix.theme.StatusBarColorManager
import org.mozilla.fenix.theme.ThemeManager
import org.mozilla.fenix.translations.TranslationsAIControllableFeatureRegistrar
import org.mozilla.fenix.translations.TranslationsEnabledSettings
import org.mozilla.fenix.tor.TorEvents
import org.mozilla.fenix.utils.AccessibilityUtils.announcePrivateModeForAccessibility
import org.mozilla.fenix.utils.Settings
import org.mozilla.fenix.utils.changeAppLauncherIcon
@@ -967,7 +969,26 @@ open class HomeActivity : LocaleAwareAppCompatActivity(), NavHostActivity, Crash
    /**
     * Handles intents received when the activity is open.
     */
    @SuppressLint("MissingSuperCall") // super.onNewIntent is called in [onNewIntentInternal(intent)]
    final override fun onNewIntent(intent: Intent) {
        if (intent.action == ACTION_MAIN || components.torController.isBootstrapped) {
            onNewIntentInternal(intent)
        } else {
            // Wait until Tor is connected to handle intents from external apps for links, search, etc.
            components.torController.registerTorListener(object : TorEvents {
                override fun onTorConnected() {
                    components.torController.unregisterTorListener(this)
                    onNewIntentInternal(intent)
                }
                override fun onTorConnecting() { /* no-op */ }
                override fun onTorStopped() { /* no-op */ }
                override fun onTorStatusUpdate(entry: String?, status: String?, progress: Double?) { /* no-op */ }
            })
            return
        }
    }

    private fun onNewIntentInternal(intent: Intent) {
        super.onNewIntent(intent)
        handleNewIntent(intent)
        startupPathProvider.onIntentReceived(intent)
@@ -1443,11 +1464,11 @@ open class HomeActivity : LocaleAwareAppCompatActivity(), NavHostActivity, Crash

    @VisibleForTesting
    internal fun navigateToHome(navController: NavController) {
        if (this is ExternalAppBrowserActivity) {
            return
        }
        // if (this is ExternalAppBrowserActivity) {
        //     return
        // }

        navController.navigate(NavGraphDirections.actionStartupHome())
        navController.navigate(NavGraphDirections.actionStartupTorbootstrap())
    }

    final override fun attachBaseContext(base: Context) {
@@ -1546,14 +1567,17 @@ open class HomeActivity : LocaleAwareAppCompatActivity(), NavHostActivity, Crash

    /**
     *  Indicates if the user should be redirected to the [BrowserFragment] or to the [HomeFragment],
     *  links from an external apps should always opened in the [BrowserFragment].
     *  links from an external apps should always opened in the [BrowserFragment],
     *  unless Tor is not yet connected.
     */
    @VisibleForTesting
    internal fun shouldStartOnHome(intent: Intent? = this.intent): Boolean {
        return components.strictMode.allowViolation(StrictMode::allowThreadDiskReads) {
            // We only want to open on home when users tap the app,
            // we want to ignore other cases when the app gets open by users clicking on links.
            getSettings().shouldStartOnHome() && intent?.action == ACTION_MAIN
            // we want to ignore other cases when the app gets open by users clicking on links,
            // unless Tor is not yet connected.
            getSettings().shouldStartOnHome() && (intent?.action == ACTION_MAIN ||
                    !components.torController.isBootstrapped)
        }
    }

Loading