tor-commits

tor-commits@lists.torproject.org

1 participants
212989 discussions

[Git][tpo/applications/tor-browser][tor-browser-128.8.0esr-14.5-1] 4 commits: fixup! TB 40562: Added Tor Browser preferences to 000-tor-browser.js
by Pier Angelo Vendrame (＠pierov) 19 Mar '25

19 Mar '25

Pier Angelo Vendrame pushed to branch tor-browser-128.8.0esr-14.5-1 at The Tor Project / Applications / Tor Browser Commits: 231850ac by Pier Angelo Vendrame at 2025-03-19T16:59:00+01:00 fixup! TB 40562: Added Tor Browser preferences to 000-tor-browser.js TB 42720: Show what's new in the about:dialog. Update our comment about app.releaseNotesURL.aboutDialog. - - - - - de58bd66 by Pier Angelo Vendrame at 2025-03-19T16:59:03+01:00 fixup! BB 4234: Use the Firefox Update Process for Base Browser. TB 42720: Show what's new in the about:dialog. Add a BB_VERSION replacement to the URLFormatter service. - - - - - 8f35f42f by Pier Angelo Vendrame at 2025-03-19T16:59:04+01:00 fixup! TB 2176: Rebrand Firefox to TorBrowser TB 42720: Show what's new in the about:dialog. Customize the "What's new" link and cleanup other channel-specific prefs. - - - - - 8d7ba10b by Pier Angelo Vendrame at 2025-03-19T16:59:04+01:00 fixup! TB 41668: Tweaks to the Base Browser updater for Tor Browser TB 42720: Show what's new in the about:dialog. Add a variable for the base browser version without dot, as we use it in blog post URLs. - - - - - 7 changed files: - browser/app/profile/000-tor-browser.js - browser/base/content/aboutDialog.xhtml - browser/base/content/aboutDialogTor.css - browser/branding/tb-alpha/pref/firefox-branding.js - browser/branding/tb-nightly/pref/firefox-branding.js - browser/branding/tb-release/pref/firefox-branding.js - toolkit/components/urlformatter/URLFormatter.sys.mjs Changes: ===================================== browser/app/profile/000-tor-browser.js ===================================== @@ -1,14 +1,7 @@ #include 001-base-profile.js pref("app.update.notifyDuringDownload", true); -pref("app.update.url.manual", "https://www.torproject.org/download/languages/"); -pref("app.update.url.details", "https://www.torproject.org/download/"); pref("app.update.badgeWaitTime", 0); -pref("app.releaseNotesURL", "about:blank"); -// disables the 'What's New?' link in the about dialog, otherwise we need to -// duplicate logic for generating the url to the blog post that is already more -// easily found in about:tor -pref("app.releaseNotesURL.aboutDialog", "about:blank"); // point to our feedback url rather than Mozilla's pref("app.feedback.baseURL", "https://support.torproject.org/%LOCALE%/misc/bug-or-feedback/"); ===================================== browser/base/content/aboutDialog.xhtml ===================================== @@ -115,9 +115,9 @@  <hbox align="baseline"> <label id="version" class="update"/> - <label id="releasenotes" is="text-link" hidden="true" data-l10n-id="releaseNotes-link"/> </hbox> <description class="text-blurb"> + <label id="releasenotes" is="text-link" hidden="true" data-l10n-id="releaseNotes-link"/> <label is="text-link" onclick="openHelpLink('firefox-help')" data-l10n-id="aboutdialog-help-user"/> <label id="submit-feedback" is="text-link" onclick="openFeedbackPage()" data-l10n-id="aboutdialog-submit-feedback"/> </description> ===================================== browser/base/content/aboutDialogTor.css ===================================== @@ -26,6 +26,10 @@ margin-block: 10px; } +#releasenotes { + margin-inline-end: .9em; /* Same as #submit-feedback */ +} + #contributeDesc { display: none; } ===================================== browser/branding/tb-alpha/pref/firefox-branding.js ===================================== @@ -10,30 +10,19 @@ // actions="showURL" // openURL="https://blog.torproject.org/tor-browser-55a2-released" pref("startup.homepage_override_url", "https://blog.torproject.org/category/applications"); -pref("startup.homepage_welcome_url", "about:welcome"); -pref("startup.homepage_welcome_url.additional", ""); +pref("app.update.url.details", "https://www.torproject.org/download/alpha/"); +pref("app.update.url.manual", "https://www.torproject.org/download/alpha/"); +pref("app.releaseNotesURL", "https://blog.torproject.org/new-alpha-release-tor-browser-%BB_VERSION_FOR_U…"); +pref("app.releaseNotesURL.aboutDialog", "https://blog.torproject.org/new-alpha-release-tor-browser-%BB_VERSION_FOR_U…"); + // The time interval between checks for a new version (in seconds) pref("app.update.interval", 43200); // 12 hours // Give the user x seconds to react before showing the big UI. default=12 hours pref("app.update.promptWaitTime", 43200); -// URL user can browse to manually if for some reason all update installation -// attempts fail. -pref("app.update.url.manual", "https://www.mozilla.org/%LOCALE%/firefox/nightly/"); -// A default value for the "More information about this update" link -// supplied in the "An update is available" page of the update wizard. -pref("app.update.url.details", "https://www.mozilla.org/%LOCALE%/firefox/nightly/notes/"); - -pref("app.releaseNotesURL", "https://www.mozilla.org/%LOCALE%/firefox/%VERSION%/releasenotes/?utm_source…"); - // The number of days a binary is permitted to be old // without checking for an update. This assumes that // app.update.checkInstallTime is true. pref("app.update.checkInstallTime.days", 2); - -// Give the user x seconds to reboot before showing a badge on the hamburger -// button. default=immediately -pref("app.update.badgeWaitTime", 0); - // Number of usages of the web console. // If this is less than 5, then pasting code into the web console is disabled pref("devtools.selfxss.count", 5); ===================================== browser/branding/tb-nightly/pref/firefox-branding.js ===================================== @@ -9,32 +9,20 @@ // each update manifest should contain attributes similar to: // actions="showURL" // openURL="https://blog.torproject.org/tor-browser-55a2-released" -pref("startup.homepage_override_url", "https://blog.torproject.org/category/applications"); -pref("startup.homepage_welcome_url", "about:welcome"); -pref("startup.homepage_welcome_url.additional", ""); +pref("startup.homepage_override_url", "https://nightlies.tbb.torproject.org/"); +pref("app.update.url.details", "https://nightlies.tbb.torproject.org/nightly-builds/tor-browser-builds/"); +pref("app.update.url.manual", "https://nightlies.tbb.torproject.org/nightly-builds/tor-browser-builds/"); +pref("app.releaseNotesURL", "about:blank"); +pref("app.releaseNotesURL.aboutDialog", "about:blank"); + // The time interval between checks for a new version (in seconds) pref("app.update.interval", 14400); // 4 hours // Give the user x seconds to react before showing the big UI. default=12 hours pref("app.update.promptWaitTime", 43200); -// URL user can browse to manually if for some reason all update installation -// attempts fail. -pref("app.update.url.manual", "https://www.mozilla.org/%LOCALE%/firefox/nightly/?reason=manual-update"); -// A default value for the "More information about this update" link -// supplied in the "An update is available" page of the update wizard. -pref("app.update.url.details", "https://www.mozilla.org/%LOCALE%/firefox/nightly/notes/"); - -pref("app.releaseNotesURL", "https://www.mozilla.org/%LOCALE%/firefox/%VERSION%/releasenotes/?utm_source…"); -pref("app.releaseNotesURL.aboutDialog", "https://www.mozilla.org/%LOCALE%/firefox/%VERSION%/releasenotes/?utm_source…"); - // The number of days a binary is permitted to be old // without checking for an update. This assumes that // app.update.checkInstallTime is true. pref("app.update.checkInstallTime.days", 2); - -// Give the user x seconds to reboot before showing a badge on the hamburger -// button. default=immediately -pref("app.update.badgeWaitTime", 0); - // Number of usages of the web console. // If this is less than 5, then pasting code into the web console is disabled pref("devtools.selfxss.count", 5); ===================================== browser/branding/tb-release/pref/firefox-branding.js ===================================== @@ -10,43 +10,19 @@ // actions="showURL" // openURL="https://blog.torproject.org/tor-browser-55a2-released" pref("startup.homepage_override_url", "https://blog.torproject.org/category/applications"); -pref("startup.homepage_welcome_url", "about:welcome"); -pref("startup.homepage_welcome_url.additional", ""); +pref("app.update.url.details", "https://www.torproject.org/download/"); +pref("app.update.url.manual", "https://www.torproject.org/download/"); +pref("app.releaseNotesURL", "https://blog.torproject.org/new-release-tor-browser-%BB_VERSION_FOR_URLS%/"); +pref("app.releaseNotesURL.aboutDialog", "https://blog.torproject.org/new-release-tor-browser-%BB_VERSION_FOR_URLS%/"); + // Interval: Time between checks for a new version (in seconds) pref("app.update.interval", 43200); // 12 hours // Give the user x seconds to react before showing the big UI. default=192 hours pref("app.update.promptWaitTime", 691200); -// app.update.url.manual: URL user can browse to manually if for some reason -// all update installation attempts fail. -// app.update.url.details: a default value for the "More information about this -// update" link supplied in the "An update is available" page of the update -// wizard. -#if MOZ_UPDATE_CHANNEL == beta - pref("app.update.url.manual", "https://www.mozilla.org/%LOCALE%/firefox/beta?reason=manual-update"); - pref("app.update.url.details", "https://www.mozilla.org/%LOCALE%/firefox/beta/notes"); - pref("app.releaseNotesURL", "https://www.mozilla.org/%LOCALE%/firefox/%VERSION%beta/releasenotes/?utm_so…"); - pref("app.releaseNotesURL.aboutDialog", "https://www.mozilla.org/%LOCALE%/firefox/%VERSION%beta/releasenotes/?utm_so…"); -#elifdef MOZ_ESR - pref("app.update.url.manual", "https://www.mozilla.org/%LOCALE%/firefox/enterprise?reason=manual-update"); - pref("app.update.url.details", "https://www.mozilla.org/%LOCALE%/firefox/organizations/notes"); - pref("app.releaseNotesURL", "https://www.mozilla.org/%LOCALE%/firefox/%VERSION%/releasenotes/?utm_source…"); - pref("app.releaseNotesURL.aboutDialog", "https://www.mozilla.org/%LOCALE%/firefox/%VERSION%/releasenotes/?utm_source…"); -#else - pref("app.update.url.manual", "https://www.mozilla.org/%LOCALE%/firefox/new?reason=manual-update"); - pref("app.update.url.details", "https://www.mozilla.org/%LOCALE%/firefox/notes"); - pref("app.releaseNotesURL", "https://www.mozilla.org/%LOCALE%/firefox/%VERSION%/releasenotes/?utm_source…"); - pref("app.releaseNotesURL.aboutDialog", "https://www.mozilla.org/%LOCALE%/firefox/%VERSION%/releasenotes/?utm_source…"); -#endif - // The number of days a binary is permitted to be old // without checking for an update. This assumes that // app.update.checkInstallTime is true. pref("app.update.checkInstallTime.days", 63); - -// Give the user x seconds to reboot before showing a badge on the hamburger -// button. default=4 days -pref("app.update.badgeWaitTime", 345600); - // Number of usages of the web console. // If this is less than 5, then pasting code into the web console is disabled pref("devtools.selfxss.count", 0); ===================================== toolkit/components/urlformatter/URLFormatter.sys.mjs ===================================== @@ -134,6 +134,16 @@ nsURLFormatterService.prototype = { DISTRIBUTION_VERSION() { return this.distribution.version; }, + BB_VERSION() { + return AppConstants.BASE_BROWSER_VERSION; + }, + BB_VERSION_FOR_URLS() { + let version = AppConstants.BASE_BROWSER_VERSION; + if (/^[0-9a\.]+$/.test(version)) { + version = version.replaceAll(".", ""); + } + return version; + }, }, formatURL: function uf_formatURL(aFormat) { View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/compare/b0a250… -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/compare/b0a250… You're receiving this email because of your account on gitlab.torproject.org.

1 0

[Git][tpo/applications/tor-browser][tor-browser-128.8.0esr-14.5-1] fixup! TB 27476: Implement about:torconnect captive portal within Tor Browser
by morgan (＠morgan) 19 Mar '25

19 Mar '25

morgan pushed to branch tor-browser-128.8.0esr-14.5-1 at The Tor Project / Applications / Tor Browser Commits: b0a250cd by Henry Wilkes at 2025-03-19T11:15:37+00:00 fixup! TB 27476: Implement about:torconnect captive portal within Tor Browser TB 42656: Drop maybeUpdateOpenLocationForTorConnect. - - - - - 1 changed file: - browser/components/urlbar/UrlbarInput.sys.mjs Changes: ===================================== browser/components/urlbar/UrlbarInput.sys.mjs ===================================== @@ -20,7 +20,6 @@ ChromeUtils.defineESModuleGetters(lazy, { ReaderMode: "resource://gre/modules/ReaderMode.sys.mjs", SearchUIUtils: "resource:///modules/SearchUIUtils.sys.mjs", SearchUtils: "resource://gre/modules/SearchUtils.sys.mjs", - TorConnect: "resource://gre/modules/TorConnect.sys.mjs", UrlbarController: "resource:///modules/UrlbarController.sys.mjs", UrlbarEventBufferer: "resource:///modules/UrlbarEventBufferer.sys.mjs", UrlbarPrefs: "resource:///modules/UrlbarPrefs.sys.mjs", @@ -276,36 +275,6 @@ export class UrlbarInput { ); } - // in certain scenarios we want user input uris to open in a new tab if they do so from the - // about:torconnect tab - #maybeUpdateOpenLocationForTorConnect( - openUILinkWhere, - currentURI, - destinationURI - ) { - try { - // only open in new tab if: - if ( - // user is navigating away from about:torconnect - currentURI === "about:torconnect" && - // we are trying to open in same tab - openUILinkWhere === "current" && - // only if user still has not bootstrapped - lazy.TorConnect.shouldShowTorConnect && - // and user is not just navigating to about:torconnect - destinationURI !== "about:torconnect" - ) { - return "tab"; - } - } catch (e) { - // swallow exception and fall through returning original so we don't accidentally break - // anything if an exception is thrown - this.logger.error(e?.message ? e.message : e); - } - - return openUILinkWhere; - } - /** * Applies styling to the text in the urlbar input, depending on the text. */ @@ -3017,11 +2986,6 @@ export class UrlbarInput { this.inputField.setSelectionRange(0, 0); } - openUILinkWhere = this.#maybeUpdateOpenLocationForTorConnect( - openUILinkWhere, - this.window.gBrowser.currentURI.asciiSpec, - url - ); if (openUILinkWhere != "current") { this.handleRevert(); } View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/commit/b0a250c… -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/commit/b0a250c… You're receiving this email because of your account on gitlab.torproject.org.

1 0

[Git][tpo/applications/tor-browser][tor-browser-128.8.0esr-14.5-1] 2 commits: fixup! [android] TBA strings
by Dan Ballard (＠dan) 19 Mar '25

19 Mar '25

Dan Ballard pushed to branch tor-browser-128.8.0esr-14.5-1 at The Tor Project / Applications / Tor Browser Commits: 9078db0b by Dan Ballard at 2025-03-19T00:06:35+00:00 fixup! [android] TBA strings Bug 43505 [android]: Add 2025 UX Survey Campaign - - - - - 359eed56 by Dan Ballard at 2025-03-18T17:10:05-07:00 Bug 43505 [android]: Add 2025 UX Survey Campaign - - - - - 6 changed files: - mobile/android/fenix/app/src/main/java/org/mozilla/fenix/home/HomeFragment.kt - + mobile/android/fenix/app/src/main/java/org/mozilla/fenix/tor/CampaignStrings.kt - mobile/android/fenix/app/src/main/java/org/mozilla/fenix/utils/Settings.kt - + mobile/android/fenix/app/src/main/res/drawable/campaign_hand.xml - mobile/android/fenix/app/src/main/res/layout/fragment_home.xml - mobile/android/fenix/app/src/main/res/values/preference_keys.xml Changes: ===================================== mobile/android/fenix/app/src/main/java/org/mozilla/fenix/home/HomeFragment.kt ===================================== @@ -12,33 +12,65 @@ import android.content.res.Configuration import android.graphics.drawable.ColorDrawable import android.net.Uri import android.os.Bundle +import android.util.Log import android.view.LayoutInflater import android.view.View import android.view.ViewGroup +import android.widget.ImageView import androidx.activity.result.ActivityResultLauncher import androidx.annotation.VisibleForTesting +import androidx.compose.foundation.Image +import androidx.compose.foundation.background +import androidx.compose.foundation.border +import androidx.compose.foundation.layout.Arrangement +import androidx.compose.foundation.layout.Box +import androidx.compose.foundation.layout.BoxWithConstraints import androidx.compose.foundation.layout.Column +import androidx.compose.foundation.layout.Row +import androidx.compose.foundation.layout.Spacer import androidx.compose.foundation.layout.fillMaxWidth import androidx.compose.foundation.layout.heightIn import androidx.compose.foundation.layout.padding +import androidx.compose.foundation.layout.size +import androidx.compose.foundation.layout.wrapContentHeight +import androidx.compose.foundation.layout.wrapContentSize +import androidx.compose.foundation.rememberScrollState +import androidx.compose.foundation.shape.CircleShape import androidx.compose.foundation.shape.RoundedCornerShape +import androidx.compose.foundation.verticalScroll +import androidx.compose.material.Button import androidx.compose.material.ButtonDefaults +import androidx.compose.material.Icon +import androidx.compose.material.IconButton import androidx.compose.material.Text import androidx.compose.material.TextButton +import androidx.compose.runtime.Composable +import androidx.compose.runtime.mutableStateOf +import androidx.compose.runtime.remember +import androidx.compose.ui.Alignment import androidx.compose.ui.ExperimentalComposeUiApi import androidx.compose.ui.Modifier +import androidx.compose.ui.draw.clip +import androidx.compose.ui.draw.drawBehind +import androidx.compose.ui.draw.scale +import androidx.compose.ui.graphics.Color +import androidx.compose.ui.layout.ContentScale import androidx.compose.ui.platform.ViewCompositionStrategy +import androidx.compose.ui.res.painterResource +import androidx.compose.ui.semantics.heading import androidx.compose.ui.semantics.semantics import androidx.compose.ui.semantics.testTag import androidx.compose.ui.semantics.testTagsAsResourceId +import androidx.compose.ui.text.TextLayoutResult +import androidx.compose.ui.text.font.FontWeight import androidx.compose.ui.text.style.TextAlign +import androidx.compose.ui.unit.Dp import androidx.compose.ui.unit.dp +import androidx.compose.ui.unit.sp import androidx.compose.ui.viewinterop.AndroidView import androidx.coordinatorlayout.widget.CoordinatorLayout import androidx.core.content.ContextCompat import androidx.core.content.ContextCompat.getColor -import androidx.core.view.children -import androidx.core.view.doOnLayout import androidx.core.view.isGone import androidx.core.view.isVisible import androidx.fragment.app.Fragment @@ -108,6 +140,7 @@ import org.mozilla.fenix.components.FenixSnackbar import org.mozilla.fenix.components.PrivateShortcutCreateManager import org.mozilla.fenix.components.TabCollectionStorage import org.mozilla.fenix.components.appstate.AppAction +import org.mozilla.fenix.components.components import org.mozilla.fenix.components.menu.MenuAccessPoint import org.mozilla.fenix.components.toolbar.IncompleteRedesignToolbarFeature import org.mozilla.fenix.components.toolbar.ToolbarPosition @@ -151,13 +184,13 @@ import org.mozilla.fenix.messaging.DefaultMessageController import org.mozilla.fenix.messaging.FenixMessageSurfaceId import org.mozilla.fenix.messaging.MessagingFeature import org.mozilla.fenix.microsurvey.ui.MicrosurveyRequestPrompt -import org.mozilla.fenix.nimbus.FxNimbus import org.mozilla.fenix.perf.MarkersFragmentLifecycleCallbacks import org.mozilla.fenix.search.toolbar.DefaultSearchSelectorController import org.mozilla.fenix.search.toolbar.SearchSelectorMenu import org.mozilla.fenix.tabstray.Page import org.mozilla.fenix.tabstray.TabsTrayAccessPoint import org.mozilla.fenix.theme.FirefoxTheme +import org.mozilla.fenix.tor.CampaignStrings import org.mozilla.fenix.utils.Settings.Companion.TOP_SITES_PROVIDER_MAX_THRESHOLD import org.mozilla.fenix.utils.allowUndo import org.mozilla.fenix.wallpapers.Wallpaper @@ -165,6 +198,9 @@ import java.lang.ref.WeakReference import org.mozilla.fenix.GleanMetrics.TabStrip as TabStripMetrics import org.mozilla.fenix.tor.UrlQuickLoadViewModel +import java.text.SimpleDateFormat +import java.time.LocalDate +import java.util.Date @Suppress("TooManyFunctions", "LargeClass") class HomeFragment : Fragment(), UserInteractionHandler { @@ -507,6 +543,8 @@ class HomeFragment : Fragment(), UserInteractionHandler { activity.themeManager.applyStatusBarTheme(activity) + tryShowUX2025Survey() + // FxNimbus.features.homescreen.recordExposure() // DO NOT MOVE ANYTHING BELOW THIS addMarker CALL! @@ -518,6 +556,31 @@ class HomeFragment : Fragment(), UserInteractionHandler { return binding.root } + private fun tryShowUX2025Survey() { + val allowedLocales = arrayListOf("en", "es", "ru", "fr", "pt") + val locale = CampaignStrings.getLocale() + + val dateFormat = SimpleDateFormat("yyyy-MM-dd-hh-zzz") + val startDate = dateFormat.parse("2025-04-14-12-UTC") + + val endDate = dateFormat.parse("2025-04-28-00-UTC") + val currentDate = Date() + + if (currentDate.before(startDate) || currentDate.after(endDate)) { + return // comment out to test + } + + if (allowedLocales.contains(locale) && !requireContext().settings().hideCampaign) { + binding.onionPatternImage.visibility = View.GONE + binding.campaignBox.apply { + setViewCompositionStrategy(ViewCompositionStrategy.DisposeOnViewTreeLifecycleDestroyed) + setContent { + CampaignBox() + } + } + } + } + private fun reinitializeNavBar() { initializeNavBar(activity = requireActivity() as HomeActivity) } @@ -1396,4 +1459,213 @@ class HomeFragment : Fragment(), UserInteractionHandler { override fun onBackPressed(): Boolean { (requireActivity() as HomeActivity).shutDown() } + + @Composable + fun CampaignBox() { + BoxWithConstraints( + contentAlignment = Alignment.Center, + modifier = Modifier + .fillMaxWidth() + .wrapContentHeight() + ) { + val alternateLayout = this.maxWidth >= 500.dp + + CampaignLayout( + alternateLayout, + maxWidth = this.maxWidth, + modifier = Modifier + .padding(top = if (alternateLayout) 65.dp else 55.dp, bottom = 56.dp), + ) + } + } + + @Composable + private fun CampaignLayout( + alternateLayout: Boolean, + maxWidth: Dp, + modifier: Modifier + ) { + Column( + modifier = modifier + .padding(horizontal = 22.dp) + .verticalScroll(rememberScrollState()) + .fillMaxWidth(getVariableWidth(maxWidth)), + horizontalAlignment = Alignment.CenterHorizontally, + ) { + PurpleBox(alternateLayout) + } + } + + private fun getVariableWidth(width: Dp): Float = (500.dp / width).coerceIn(0.75f, 1.0f) + + @Composable + private fun PurpleBox( + alternateLayout: Boolean, + ) { + Box( + modifier = Modifier.background(PhotonColors.Violet90, shape = RoundedCornerShape(8.dp)) + .padding(16.dp), + ) { + Row( + modifier = Modifier.fillMaxWidth(), + ) { + Emoji() + Spacer(Modifier.weight(1f)) + ExitIcon() + } + DynamicCampaignContent(alternateLayout) + } + } + + @Composable + private fun Emoji() { + val alpha38Violet40 = Color(PhotonColors.Violet40.red, PhotonColors.Violet40.green, PhotonColors.Violet40.blue, 0.38f) + Image( + painter = painterResource(id = R.drawable.campaign_hand), + contentDescription = null, + modifier = Modifier + .size(64.dp) + .padding(16.dp) + .drawBehind { + drawCircle( + color = alpha38Violet40, + radius = this.size.maxDimension + ) + } + ) + } + + @Composable + private fun ExitIcon() { + IconButton( + onClick = { + binding.campaignBox.visibility = View.GONE + binding.onionPatternImage.visibility = View.VISIBLE + context?.components?.settings?.hideCampaign = true + }, + ) { + Icon( + painter = painterResource(id = R.drawable.ic_close), + tint = Color( + getColor( + requireContext(), + R.color.photonWhite, + ), + ), + contentDescription = CampaignStrings.get(CampaignStrings.CloseKey), + modifier = Modifier + .size(48.dp) + .padding(8.dp) + ) + } + } + + + @Composable + private fun DynamicCampaignContent( + alternateLayout: Boolean + ) { + Row(verticalAlignment = Alignment.CenterVertically) { + Column( + modifier = Modifier.fillMaxWidth() + .padding( top = 88.dp), + horizontalAlignment = Alignment.Start, + ) { + TitleText() + MainText() + + if (alternateLayout) { + Row(modifier = Modifier.fillMaxWidth()) { + Button1(alternateLayout) + Button2() + } + } else { + Button1(alternateLayout) + Button2() + } + + } + } + } + + @Composable + private fun TitleText() { + + Text(text = CampaignStrings.get(CampaignStrings.HeaderKey), + color = PhotonColors.LightGrey05, + textAlign = TextAlign.Left, + fontWeight = FontWeight.Bold, + fontSize = 24.sp, + lineHeight = 34.sp, + modifier = Modifier.padding(bottom = 16.dp) + ) + } + + @Composable + private fun MainText() { + + Text(text = CampaignStrings.get(CampaignStrings.BodyKey), + modifier = Modifier + .fillMaxWidth() + .padding( + start = 0.dp, + end = 0.dp, + bottom = 18.dp, + ), + color = PhotonColors.LightGrey05, + fontSize = 18.sp, + textAlign = TextAlign.Left, + ) + } + + @Composable + private fun Button1(alternateLayout: Boolean) { + Button( + onClick = { + var locale = CampaignStrings.getLocale() + if (locale == "pt") { + locale = "pt-BR" + } + (activity as HomeActivity).openToBrowserAndLoad( + searchTermOrURL = "https://survey.torproject.org/index.php/923269?lang=${locale}", + newTab = true, + from = BrowserDirection.FromHome, + ) + }, + colors = ButtonDefaults.buttonColors( + backgroundColor = PhotonColors.Violet60), + shape = RoundedCornerShape(4.dp), + modifier = Modifier.padding(0.dp) + .fillMaxWidth(fraction = if (alternateLayout) 0.5f else 1f), + + ) { + Text(text = CampaignStrings.get(CampaignStrings.CTAKey), + color = PhotonColors.LightGrey05, + textAlign = TextAlign.Center, + fontSize = 18.sp, + modifier = Modifier.padding(8.dp)) + } + } + + @Composable + private fun Button2() { + Button( + onClick = { + binding.campaignBox.visibility = View.GONE + binding.onionPatternImage.visibility = View.VISIBLE + context?.components?.settings?.hideCampaign = true + }, + colors = ButtonDefaults.buttonColors( + backgroundColor = PhotonColors.Violet90), + shape = RoundedCornerShape(4.dp), + modifier = Modifier.padding(0.dp) + .fillMaxWidth() + ) { + Text(text = CampaignStrings.get(CampaignStrings.DismissKey), + color = PhotonColors.Violet20, + textAlign = TextAlign.Center, + fontSize = 18.sp, + modifier = Modifier.padding(8.dp)) + } + } } ===================================== mobile/android/fenix/app/src/main/java/org/mozilla/fenix/tor/CampaignStrings.kt ===================================== @@ -0,0 +1,64 @@ +package org.mozilla.fenix.tor + +import java.util.Locale + +object CampaignStrings { + + val HeaderKey = "key_header" + val BodyKey = "key_body" + val CTAKey = "key_cta" + val DismissKey = "key_dismiss" + val CloseKey = "key_close" + + private val translations: HashMap<String, HashMap<String, String>> = hashMapOf( + "en" to hashMapOf( + HeaderKey to "We’d love your feedback", + BodyKey to "Help us improve Tor Browser by completing this 10-minute survey.", + CTAKey to "Launch the survey", + DismissKey to "Dismiss", + CloseKey to "Close", + ), + "es" to hashMapOf( + HeaderKey to "Danos tu opinión", + BodyKey to "Ayúdanos a mejorar el Navegador Tor completando esta encuesta de 10 minutos.", + CTAKey to "Iniciar la encuesta", + DismissKey to "Cerrar", + CloseKey to "Cerrar", + ), + "ru" to hashMapOf( + HeaderKey to "Мы будем рады вашим отзывам", + BodyKey to "Помогите нам улучшить браузер Tor, пройдя 10-минутный опрос.", + CTAKey to "Начать опрос", + DismissKey to "Закрыть", + CloseKey to "Закрыть", + ), + "fr" to hashMapOf( + HeaderKey to "Nous serions ravis d’avoir votre avis !", + BodyKey to "Aidez-nous à améliorer le navigateur Tor en répondant à cette enquête de 10 minutes.", + CTAKey to "Lancer l'enquête", + DismissKey to "Fermer", + CloseKey to "Fermer", + ), + "pt" to hashMapOf( + HeaderKey to "Adoraríamos ouvir sua opinião", + BodyKey to "Ajude-nos a melhorar o Navegador Tor respondendo a esta pesquisa de 10 minutos.", + CTAKey to "Iniciar a pesquisa", + DismissKey to "Dispensar", + CloseKey to "Fechar" + ), + ) + + fun getLocale(): String { + // TODO: do we care about spoofEnglish setting? + return Locale.getDefault().getLanguage(); + } + + + fun get(key: String): String { + val localeStrings = translations.get(getLocale()) + if (localeStrings == null) { + return "" + } + return localeStrings.get(key) ?: "" + } +} ===================================== mobile/android/fenix/app/src/main/java/org/mozilla/fenix/utils/Settings.kt ===================================== @@ -43,15 +43,7 @@ import org.mozilla.fenix.components.settings.lazyFeatureFlagPreference import org.mozilla.fenix.components.toolbar.ToolbarPosition import org.mozilla.fenix.ext.components import org.mozilla.fenix.ext.getPreferenceKey -import org.mozilla.fenix.nimbus.CookieBannersSection import org.mozilla.fenix.nimbus.FxNimbus -import org.mozilla.fenix.nimbus.HomeScreenSection -import org.mozilla.fenix.nimbus.Mr2022Section -import org.mozilla.fenix.nimbus.QueryParameterStrippingSection -import org.mozilla.fenix.nimbus.QueryParameterStrippingSection.QUERY_PARAMETER_STRIPPING -import org.mozilla.fenix.nimbus.QueryParameterStrippingSection.QUERY_PARAMETER_STRIPPING_ALLOW_LIST -import org.mozilla.fenix.nimbus.QueryParameterStrippingSection.QUERY_PARAMETER_STRIPPING_PMB -import org.mozilla.fenix.nimbus.QueryParameterStrippingSection.QUERY_PARAMETER_STRIPPING_STRIP_LIST import org.mozilla.fenix.settings.PhoneFeature import org.mozilla.fenix.settings.deletebrowsingdata.DeleteBrowsingDataOnQuitType import org.mozilla.fenix.settings.logins.SavedLoginsSortingStrategyMenu @@ -2116,4 +2108,9 @@ class Settings(private val appContext: Context) : PreferencesHolder { appContext.getPreferenceKey(R.string.pref_key_quick_start), default = false, ) + + var hideCampaign by booleanPreference( + appContext.getPreferenceKey(R.string.pref_key_hide_campaign_2025_ux_survey), + default = false, + ) } ===================================== mobile/android/fenix/app/src/main/res/drawable/campaign_hand.xml ===================================== @@ -0,0 +1,21 @@ +<vector xmlns:android="http://schemas.android.com/apk/res/android" xmlns:aapt="http://schemas.android.com/aapt" + android:viewportWidth="36" + android:viewportHeight="36" + android:width="36dp" + android:height="36dp"> + <path + android:pathData="M4.861 9.147c0.94 -0.657 2.357 -0.531 3.201 0.166l-0.968 -1.407c-0.779 -1.111 -0.5 -2.313 0.612 -3.093 1.112 -0.777 4.263 1.312 4.263 1.312 -0.786 -1.122 -0.639 -2.544 0.483 -3.331 1.122 -0.784 2.67 -0.513 3.456 0.611l10.42 14.72L25 31l-11.083 -4.042L4.25 12.625c-0.793 -1.129 -0.519 -2.686 0.611 -3.478z" + android:fillColor="#EF9645" /> + <path + android:pathData="M2.695 17.336s-1.132 -1.65 0.519 -2.781c1.649 -1.131 2.78 0.518 2.78 0.518l5.251 7.658c0.181 -0.302 0.379 -0.6 0.6 -0.894L4.557 11.21s-1.131 -1.649 0.519 -2.78c1.649 -1.131 2.78 0.518 2.78 0.518l6.855 9.997c0.255 -0.208 0.516 -0.417 0.785 -0.622L7.549 6.732s-1.131 -1.649 0.519 -2.78c1.649 -1.131 2.78 0.518 2.78 0.518l7.947 11.589c0.292 -0.179 0.581 -0.334 0.871 -0.498L12.238 4.729s-1.131 -1.649 0.518 -2.78c1.649 -1.131 2.78 0.518 2.78 0.518l7.854 11.454 1.194 1.742c-4.948 3.394 -5.419 9.779 -2.592 13.902 0.565 0.825 1.39 0.26 1.39 0.26 -3.393 -4.949 -2.357 -10.51 2.592 -13.903L24.515 8.62s-0.545 -1.924 1.378 -2.47c1.924 -0.545 2.47 1.379 2.47 1.379l1.685 5.004c0.668 1.984 1.379 3.961 2.32 5.831 2.657 5.28 1.07 11.842 -3.94 15.279 -5.465 3.747 -12.936 2.354 -16.684 -3.11L2.695 17.336z" + android:fillColor="#FFDC5D" /> + <path + android:pathData="M12 32.042C8 32.042 3.958 28 3.958 24c0 -0.553 -0.405 -1 -0.958 -1s-1.042 0.447 -1.042 1C1.958 30 6 34.042 12 34.042c0.553 0 1 -0.489 1 -1.042s-0.447 -0.958 -1 -0.958z" + android:fillColor="#5DADEC" /> + <path + android:pathData="M7 34c-3 0 -5 -2 -5 -5 0 -0.553 -0.447 -1 -1 -1s-1 0.447 -1 1c0 4 3 7 7 7 0.553 0 1 -0.447 1 -1s-0.447 -1 -1 -1zM24 2c-0.552 0 -1 0.448 -1 1s0.448 1 1 1c4 0 8 3.589 8 8 0 0.552 0.448 1 1 1s1 -0.448 1 -1c0 -5.514 -4 -10 -10 -10z" + android:fillColor="#5DADEC" /> + <path + android:pathData="M29 0.042c-0.552 0 -1 0.406 -1 0.958s0.448 1.042 1 1.042c3 0 4.958 2.225 4.958 4.958 0 0.552 0.489 1 1.042 1s0.958 -0.448 0.958 -1C35.958 3.163 33 0.042 29 0.042z" + android:fillColor="#5DADEC" /> +</vector> \ No newline at end of file ===================================== mobile/android/fenix/app/src/main/res/layout/fragment_home.xml ===================================== @@ -134,6 +134,12 @@ </com.google.android.material.appbar.AppBarLayout> + <androidx.compose.ui.platform.ComposeView + android:id="@+id/campaignBox" + android:layout_width="match_parent" + android:layout_height="wrap_content" + android:layout_gravity="center"/> + <androidx.recyclerview.widget.RecyclerView android:id="@+id/sessionControlRecyclerView" android:layout_width="match_parent" ===================================== mobile/android/fenix/app/src/main/res/values/preference_keys.xml ===================================== @@ -430,4 +430,5 @@ <string name="pref_key_tor_network_settings_bridges_enabled">pref_key_tor_network_settings_bridges_enabled</string> <string name="pref_key_spoof_english" translatable="false">pref_key_spoof_english</string> + <string name="pref_key_hide_campaign_2025_ux_survey" translatable="false">pref_key_hide_campaign_2025_ux_survey_test</string> </resources> View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/compare/b23a82… -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/compare/b23a82… You're receiving this email because of your account on gitlab.torproject.org.

1 0

[Git][tpo/applications/torbrowser-launcher] Deleted branch AppArmor-updates-for-current-Debian
by intrigeri (＠intrigeri) 18 Mar '25

18 Mar '25

intrigeri deleted branch AppArmor-updates-for-current-Debian at The Tor Project / Applications / torbrowser-launcher -- You're receiving this email because of your account on gitlab.torproject.org.

1 0

[Git][tpo/applications/torbrowser-launcher][main] 5 commits: AppArmor: allow unprivileged user namespaces
by intrigeri (＠intrigeri) 18 Mar '25

18 Mar '25

intrigeri pushed to branch main at The Tor Project / Applications / torbrowser-launcher Commits: 692e28a1 by intrigeri at 2025-03-18T13:49:53+00:00 AppArmor: allow unprivileged user namespaces Firefox uses userns to set up its own sandboxing. On Debian, AppArmor was already allowing this by default, until a recent upload (that is now in Trixie) updated the features pinning to a version that now mediates usage of userns, so this functionality is now blocked by profiles that don't explicitly allow it. Let's repair this. Also reported as Debian#1098845. - - - - - 91db109a by intrigeri at 2025-03-18T14:02:01+00:00 AppArmor: allow reading cgroups-v2 CPU bandwidth quota information Firefox uses this info to determine how many CPUs the current thread actually has access to, which seems like a reasonable thing to do for an app like Firefox which manages a bunch of child processes. The call chain is: get_num_cpus → cgroups_num_cpus → init_cgroups → load_cgroups → cpu_quota → max → "cpu.max". - - - - - 7772a1ea by intrigeri at 2025-03-18T14:15:17+00:00 AppArmor: allow executing Firefox' own VA-API probe utility This is necessary for Tor Browser to determine if VA-API is supported by the host system, which in turn is needed to enable video hardware decoding. - - - - - 9eb8686d by intrigeri at 2025-03-18T14:18:41+00:00 AppArmor: allow reading intel-media-driver feature files Firefox reads these files when it runs the vaapitest tool and the VAAPI driver for the Intel GEN8+ Graphics family is installed. - - - - - 479b8f53 by intrigeri at 2025-03-18T17:00:32+00:00 Merge branch 'AppArmor-updates-for-current-Debian' into 'main' AppArmor: various updates including 1 important fix for Debian Trixie See merge request tpo/applications/torbrowser-launcher!24 - - - - - 1 changed file: - apparmor/torbrowser.Browser.firefox Changes: ===================================== apparmor/torbrowser.Browser.firefox ===================================== @@ -13,6 +13,8 @@ profile torbrowser_firefox @{torbrowser_firefox_executable} { #include if exists <abstractions/vulkan> #include if exists <abstractions/dbus-session-strict> + userns, + deny capability sys_ptrace, # Uncomment the following lines if you want to give the Tor Browser read-write @@ -94,6 +96,10 @@ profile torbrowser_firefox @{torbrowser_firefox_executable} { owner @{torbrowser_home_dir}/TorBrowser/Tor/*.so.* mr, owner @{torbrowser_home_dir}/TorBrowser/Tor/libstdc++/*.so mr, owner @{torbrowser_home_dir}/TorBrowser/Tor/libstdc++/*.so.* mr, + owner @{torbrowser_home_dir}/vaapitest ix, + + # intel-media-driver + /etc/igfx_user_feature*.txt r, # parent Firefox process when restarting after upgrade, Web Content processes owner @{torbrowser_firefox_executable} pxmr -> torbrowser_firefox, @@ -121,6 +127,7 @@ profile torbrowser_firefox @{torbrowser_firefox_executable} { /sys/devices/system/node/ r, /sys/devices/system/node/node[0-9]*/meminfo r, /sys/fs/cgroup/cpu,cpuacct/{,user.slice/}cpu.cfs_quota_us r, + /sys/fs/cgroup/user.slice/user-[0-9]*.slice/user(a)[0-9]*.service/app.slice/app-gnome-torbrowser-[0-9]*.scope/cpu.max r, deny /sys/class/input/ r, deny /sys/devices/virtual/block/*/uevent r, View it on GitLab: https://gitlab.torproject.org/tpo/applications/torbrowser-launcher/-/compar… -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/torbrowser-launcher/-/compar… You're receiving this email because of your account on gitlab.torproject.org.

1 0

[Git][tpo/applications/torbrowser-launcher][main] fix: window icon under wayland
by boklm (＠boklm) 18 Mar '25

18 Mar '25

boklm pushed to branch main at The Tor Project / Applications / torbrowser-launcher Commits: a734238b by Integral at 2025-03-18T17:55:44+01:00 fix: window icon under wayland Currently, when running the launcher on Wayland, the window icon will fallback to the generic Wayland icon. Set desktop filename to solve this problem. - - - - - 1 changed file: - torbrowser_launcher/__init__.py Changes: ===================================== torbrowser_launcher/__init__.py ===================================== @@ -81,6 +81,9 @@ def main(): common = Common(tor_browser_launcher_version) app = Application() + if "WAYLAND_DISPLAY" in os.environ: + app.setDesktopFileName("torbrowser") + # Open the window gui = None View it on GitLab: https://gitlab.torproject.org/tpo/applications/torbrowser-launcher/-/commit… -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/torbrowser-launcher/-/commit… You're receiving this email because of your account on gitlab.torproject.org.

1 0

[Git][tpo/applications/tor-browser][tor-browser-128.8.0esr-14.5-1] fixup! TB 40041 [android]: Implement Tor Network Settings
by Dan Ballard (＠dan) 18 Mar '25

18 Mar '25

Dan Ballard pushed to branch tor-browser-128.8.0esr-14.5-1 at The Tor Project / Applications / Tor Browser Commits: b23a826c by clairehurst at 2025-03-18T16:30:23+00:00 fixup! TB 40041 [android]: Implement Tor Network Settings - - - - - 3 changed files: - mobile/android/fenix/app/src/main/java/org/mozilla/fenix/settings/SettingsFragment.kt - mobile/android/fenix/app/src/main/res/values/preference_keys.xml - mobile/android/fenix/app/src/main/res/xml/preferences.xml Changes: ===================================== mobile/android/fenix/app/src/main/java/org/mozilla/fenix/settings/SettingsFragment.kt ===================================== @@ -796,6 +796,17 @@ class SettingsFragment : PreferenceFragmentCompat(), UserInteractionHandler { true } } + requirePreference<Preference>(R.string.pref_key_about_config_shortcut).apply { + isVisible = requireContext().settings().showSecretDebugMenuThisSession || Config.channel == ReleaseChannel.Debug + setOnPreferenceClickListener { + (requireActivity() as HomeActivity).openToBrowserAndLoad( + searchTermOrURL = "about:config", + from = BrowserDirection.FromSettings, + newTab = true, + ) + true + } + } } @VisibleForTesting ===================================== mobile/android/fenix/app/src/main/res/values/preference_keys.xml ===================================== @@ -415,6 +415,7 @@ <string name="pref_key_tor_network_settings_bridge_config" translatable="false">pref_key_tor_network_settings_bridge_config</string> <string name="pref_key_tor_logs" translatable="false">pref_key_tor_logs</string> <string name="pref_key_use_html_connection_ui" translatable="false">pref_key_use_html_connection_ui</string>  + <string name="pref_key_about_config_shortcut" translatable="false">pref_key_about_config_shortcut</string> <string name="pref_key_tor_network_settings_bridge_config_explanation" translatable="false">pref_key_tor_network_settings_bridge_config_explanation</string> <string name="pref_key_tor_network_settings_bridge_config_toggle" translatable="false">pref_key_tor_network_settings_bridge_config_toggle</string> <string name="pref_key_tor_network_settings_bridge_config_builtin_bridge_obfs4" translatable="false">pref_key_tor_network_settings_bridge_config_builtin_bridge_obfs4</string> ===================================== mobile/android/fenix/app/src/main/res/xml/preferences.xml ===================================== @@ -194,6 +194,12 @@ android:title="@string/preferences_tor_logs" android:summary="@string/preferences_tor_logs_description" /> + <Preference + android:key="@string/pref_key_about_config_shortcut" + app:iconSpaceReserved="false" + android:title="about:config"/> + +   View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/commit/b23a826… -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/commit/b23a826… You're receiving this email because of your account on gitlab.torproject.org.

1 0

[Git][tpo/applications/tor-browser][tor-browser-128.8.0esr-14.5-1] 6 commits: fixup! TB 40933: Add tor-launcher functionality
by henry (＠henry) 18 Mar '25

18 Mar '25

henry pushed to branch tor-browser-128.8.0esr-14.5-1 at The Tor Project / Applications / Tor Browser Commits: 202a08d2 by Henry Wilkes at 2025-03-18T15:25:24+00:00 fixup! TB 40933: Add tor-launcher functionality TB 43405: Split TorProvider writeSettings into separate methods for the proxy, firewall and bridges settings. We also call TorSettings.setTorProvider instead of TorProvider.writeSettings so that TorSettings can handle the application errors. - - - - - a88b1fd3 by Henry Wilkes at 2025-03-18T15:25:25+00:00 fixup! TB 40597: Implement TorSettings module TB 43405: Do not allow string values for proxy and firewall ports. And do not allow a proxy username without a password or vis versa. - - - - - ebfa2591 by Henry Wilkes at 2025-03-18T15:25:26+00:00 fixup! TB 40597: Implement TorSettings module TB 43405: TorSettings handles failures to apply Tor settings. We update TorSettings.#applySettings to catch TorProvider write errors and signal this error with "ApplyError". We also keep track of which group of settings have failed so that we can restore them on the user's request. - - - - - cd596922 by Henry Wilkes at 2025-03-18T15:25:27+00:00 fixup! TB 31286: Implementation of bridge, proxy, and firewall settings in about:preferences#connection TB 43405: Add some validation to the Advanced connection settings dialog. - - - - - 58ad673d by Henry Wilkes at 2025-03-18T15:25:27+00:00 TB 43405: Show a prompt whenever we fail to apply Tor settings. - - - - - eb9525d5 by Henry Wilkes at 2025-03-18T15:25:28+00:00 fixup! Tor Browser strings TB 43405: Add strings for tor settings error notification. - - - - - 8 changed files: - browser/components/BrowserGlue.sys.mjs - browser/components/torpreferences/content/connectionSettingsDialog.js - browser/components/torpreferences/content/connectionSettingsDialog.xhtml - + browser/modules/TorSettingsNotification.sys.mjs - browser/modules/moz.build - toolkit/components/tor-launcher/TorProvider.sys.mjs - toolkit/locales/en-US/toolkit/global/tor-browser.ftl - toolkit/modules/TorSettings.sys.mjs Changes: ===================================== browser/components/BrowserGlue.sys.mjs ===================================== @@ -96,6 +96,8 @@ ChromeUtils.defineESModuleGetters(lazy, { TorConnect: "resource://gre/modules/TorConnect.sys.mjs", TorConnectTopics: "resource://gre/modules/TorConnect.sys.mjs", TorProviderBuilder: "resource://gre/modules/TorProviderBuilder.sys.mjs", + TorSettingsNotification: + "resource:///modules/TorSettingsNotification.sys.mjs", UIState: "resource://services-sync/UIState.sys.mjs", UrlbarPrefs: "resource:///modules/UrlbarPrefs.sys.mjs", WebChannel: "resource://gre/modules/WebChannel.sys.mjs", @@ -2030,6 +2032,8 @@ BrowserGlue.prototype = { lazy.TorProviderBuilder.firstWindowLoaded(); + lazy.TorSettingsNotification.ready(); + ClipboardPrivacy.startup(); this._firstWindowTelemetry(aWindow); ===================================== browser/components/torpreferences/content/connectionSettingsDialog.js ===================================== @@ -5,6 +5,7 @@ const { TorSettings, TorProxyType } = ChromeUtils.importESModule( ); const gConnectionSettingsDialog = { + _acceptButton: null, _useProxyCheckbox: null, _proxyTypeLabel: null, _proxyTypeMenulist: null, @@ -38,25 +39,43 @@ const gConnectionSettingsDialog = { "input#torPreferences-connection-textboxAllowedPorts", }, - // disables the provided list of elements - _setElementsDisabled(elements, disabled) { - for (let currentElement of elements) { - currentElement.disabled = disabled; - } - }, + /** + * The "proxy" and "firewall" settings to pass on to TorSettings. + * + * Each group is `null` whilst the inputs are invalid. + * + * @type {{proxy: ?object, firewall: ?object}} + */ + _settings: { proxy: null, firewall: null }, init() { + const currentSettings = TorSettings.getSettings(); + + const dialog = document.getElementById("torPreferences-connection-dialog"); + dialog.addEventListener("dialogaccept", event => { + if (!this._settings.proxy || !this._settings.firewall) { + // Do not close yet. + event.preventDefault(); + return; + } + // TODO: Maybe wait for the method to resolve before closing. Although + // this can take a few seconds. See tor-browser#43467. + TorSettings.changeSettings(this._settings); + }); + this._acceptButton = dialog.getButton("accept"); + const selectors = this.selectors; // Local Proxy this._useProxyCheckbox = document.querySelector(selectors.useProxyCheckbox); + this._useProxyCheckbox.checked = currentSettings.proxy.enabled; this._useProxyCheckbox.addEventListener("command", () => { - const checked = this._useProxyCheckbox.checked; - this.onToggleProxy(checked); + this.updateProxyType(); }); + this._proxyTypeLabel = document.querySelector(selectors.proxyTypeLabel); - let mockProxies = [ + const mockProxies = [ { value: TorProxyType.Socks4, l10nId: "tor-advanced-dialog-proxy-socks4-menuitem", @@ -72,15 +91,17 @@ const gConnectionSettingsDialog = { ]; this._proxyTypeMenulist = document.querySelector(selectors.proxyTypeList); this._proxyTypeMenulist.addEventListener("command", () => { - const value = this._proxyTypeMenulist.value; - this.onSelectProxyType(value); + this.updateProxyType(); }); - for (let currentProxy of mockProxies) { + for (const currentProxy of mockProxies) { let menuEntry = window.document.createXULElement("menuitem"); menuEntry.setAttribute("value", currentProxy.value); menuEntry.setAttribute("data-l10n-id", currentProxy.l10nId); this._proxyTypeMenulist.querySelector("menupopup").appendChild(menuEntry); } + this._proxyTypeMenulist.value = currentSettings.proxy.enabled + ? currentSettings.proxy.type + : ""; this._proxyAddressLabel = document.querySelector( selectors.proxyAddressLabel @@ -89,13 +110,15 @@ const gConnectionSettingsDialog = { selectors.proxyAddressTextbox ); this._proxyAddressTextbox.addEventListener("blur", () => { + // If the address includes a port move it to the port input instead. let value = this._proxyAddressTextbox.value.trim(); let colon = value.lastIndexOf(":"); if (colon != -1) { - let maybePort = parseInt(value.substr(colon + 1)); - if (!isNaN(maybePort) && maybePort > 0 && maybePort < 65536) { + let maybePort = this.parsePort(value.substr(colon + 1)); + if (maybePort !== null) { this._proxyAddressTextbox.value = value.substr(0, colon); this._proxyPortTextbox.value = maybePort; + this.updateProxy(); } } }); @@ -114,23 +137,36 @@ const gConnectionSettingsDialog = { selectors.proxyPasswordTextbox ); - this.onToggleProxy(false); - if (TorSettings.proxy.enabled) { - this.onToggleProxy(true); - this.onSelectProxyType(TorSettings.proxy.type); - this._proxyAddressTextbox.value = TorSettings.proxy.address; - this._proxyPortTextbox.value = TorSettings.proxy.port; - this._proxyUsernameTextbox.value = TorSettings.proxy.username; - this._proxyPasswordTextbox.value = TorSettings.proxy.password; + if (currentSettings.proxy.enabled) { + this._proxyAddressTextbox.value = currentSettings.proxy.address; + this._proxyPortTextbox.value = currentSettings.proxy.port; + this._proxyUsernameTextbox.value = currentSettings.proxy.username; + this._proxyPasswordTextbox.value = currentSettings.proxy.password; + } else { + this._proxyAddressTextbox.value = ""; + this._proxyPortTextbox.value = ""; + this._proxyUsernameTextbox.value = ""; + this._proxyPasswordTextbox.value = ""; + } + + for (const el of [ + this._proxyAddressTextbox, + this._proxyPortTextbox, + this._proxyUsernameTextbox, + this._proxyPasswordTextbox, + ]) { + el.addEventListener("input", () => { + this.updateProxy(); + }); } // Local firewall this._useFirewallCheckbox = document.querySelector( selectors.useFirewallCheckbox ); + this._useFirewallCheckbox.checked = currentSettings.firewall.enabled; this._useFirewallCheckbox.addEventListener("command", () => { - const checked = this._useFirewallCheckbox.checked; - this.onToggleFirewall(checked); + this.updateFirewallEnabled(); }); this._allowedPortsLabel = document.querySelector( selectors.firewallAllowedPortsLabel @@ -138,182 +174,161 @@ const gConnectionSettingsDialog = { this._allowedPortsTextbox = document.querySelector( selectors.firewallAllowedPortsTextbox ); + this._allowedPortsTextbox.value = currentSettings.firewall.enabled + ? currentSettings.firewall.allowed_ports.join(",") + : "80,443"; - this.onToggleFirewall(false); - if (TorSettings.firewall.enabled) { - this.onToggleFirewall(true); - this._allowedPortsTextbox.value = - TorSettings.firewall.allowed_ports.join(", "); - } - - const dialog = document.getElementById("torPreferences-connection-dialog"); - dialog.addEventListener("dialogaccept", () => { - this._applySettings(); + this._allowedPortsTextbox.addEventListener("input", () => { + this.updateFirewall(); }); - }, - // callback when proxy is toggled - onToggleProxy(enabled) { - this._useProxyCheckbox.checked = enabled; - let disabled = !enabled; + this.updateProxyType(); + this.updateFirewallEnabled(); + }, - this._setElementsDisabled( - [ - this._proxyTypeLabel, - this._proxyTypeMenulist, - this._proxyAddressLabel, - this._proxyAddressTextbox, - this._proxyPortLabel, - this._proxyPortTextbox, - this._proxyUsernameLabel, - this._proxyUsernameTextbox, - this._proxyPasswordLabel, - this._proxyPasswordTextbox, - ], - disabled - ); - if (enabled) { - this.onSelectProxyType(this._proxyTypeMenulist.value); + /** + * Convert a string into a port number. + * + * @param {string} portStr - The string to convert. + * @returns {?integer} - The port number, or `null` if the given string could + * not be converted. + */ + parsePort(portStr) { + const portRegex = /^[1-9][0-9]*$/; // Strictly-positive decimal integer. + if (!portRegex.test(portStr)) { + return null; } + const port = parseInt(portStr, 10); + if (TorSettings.validPort(port)) { + return port; + } + return null; }, - // callback when proxy type is changed - onSelectProxyType(value) { - if (typeof value === "string") { - value = parseInt(value); - } + /** + * Update the disabled state of the accept button. + */ + updateAcceptButton() { + this._acceptButton.disabled = + !this._settings.proxy || !this._settings.firewall; + }, - this._proxyTypeMenulist.value = value; - switch (value) { - case TorProxyType.Invalid: { - this._setElementsDisabled( - [ - this._proxyAddressLabel, - this._proxyAddressTextbox, - this._proxyPortLabel, - this._proxyPortTextbox, - this._proxyUsernameLabel, - this._proxyUsernameTextbox, - this._proxyPasswordLabel, - this._proxyPasswordTextbox, - ], - true - ); // DISABLE + /** + * Update the UI when the proxy setting is enabled or disabled, or the proxy + * type changes. + */ + updateProxyType() { + const enabled = this._useProxyCheckbox.checked; + const haveType = enabled && Boolean(this._proxyTypeMenulist.value); + const type = parseInt(this._proxyTypeMenulist.value, 10); - this._proxyAddressTextbox.value = ""; - this._proxyPortTextbox.value = ""; - this._proxyUsernameTextbox.value = ""; - this._proxyPasswordTextbox.value = ""; - break; - } - case TorProxyType.Socks4: { - this._setElementsDisabled( - [ - this._proxyAddressLabel, - this._proxyAddressTextbox, - this._proxyPortLabel, - this._proxyPortTextbox, - ], - false - ); // ENABLE - this._setElementsDisabled( - [ - this._proxyUsernameLabel, - this._proxyUsernameTextbox, - this._proxyPasswordLabel, - this._proxyPasswordTextbox, - ], - true - ); // DISABLE + this._proxyTypeLabel.disabled = !enabled; + this._proxyTypeMenulist.disabled = !enabled; + this._proxyAddressLabel.disabled = !haveType; + this._proxyAddressTextbox.disabled = !haveType; + this._proxyPortLabel.disabled = !haveType; + this._proxyPortTextbox.disabled = !haveType; + this._proxyUsernameTextbox.disabled = + !haveType || type === TorProxyType.Socks4; + this._proxyPasswordTextbox.disabled = + !haveType || type === TorProxyType.Socks4; + if (type === TorProxyType.Socks4) { + // Clear unused value. + this._proxyUsernameTextbox.value = ""; + this._proxyPasswordTextbox.value = ""; + } - this._proxyUsernameTextbox.value = ""; - this._proxyPasswordTextbox.value = ""; - break; - } - case TorProxyType.Socks5: - case TorProxyType.HTTPS: { - this._setElementsDisabled( - [ - this._proxyAddressLabel, - this._proxyAddressTextbox, - this._proxyPortLabel, - this._proxyPortTextbox, - this._proxyUsernameLabel, - this._proxyUsernameTextbox, - this._proxyPasswordLabel, - this._proxyPasswordTextbox, - ], - false - ); // ENABLE - break; + this.updateProxy(); + }, + + /** + * Update the dialog's stored proxy values. + */ + updateProxy() { + if (this._useProxyCheckbox.checked) { + const typeStr = this._proxyTypeMenulist.value; + const type = parseInt(typeStr, 10); + // TODO: Validate the address. See tor-browser#43467. + const address = this._proxyAddressTextbox.value; + const portStr = this._proxyPortTextbox.value; + const username = + type === TorProxyType.Socks4 ? "" : this._proxyUsernameTextbox.value; + const password = + type === TorProxyType.Socks4 ? "" : this._proxyPasswordTextbox.value; + const port = parseInt(portStr, 10); + if ( + !typeStr || + !address || + !portStr || + !TorSettings.validPort(port) || + // SOCKS5 needs either both username and password, or neither. + (type === TorProxyType.Socks5 && + !TorSettings.validSocks5Credentials(username, password)) + ) { + // Invalid. + this._settings.proxy = null; + } else { + this._settings.proxy = { + enabled: true, + type, + address, + port, + username, + password, + }; } + } else { + this._settings.proxy = { enabled: false }; } + + this.updateAcceptButton(); }, - // callback when firewall proxy is toggled - onToggleFirewall(enabled) { - this._useFirewallCheckbox.checked = enabled; - let disabled = !enabled; + /** + * Update the UI when the firewall setting is enabled or disabled. + */ + updateFirewallEnabled() { + const enabled = this._useFirewallCheckbox.checked; + this._allowedPortsLabel.disabled = !enabled; + this._allowedPortsTextbox.disabled = !enabled; - this._setElementsDisabled( - [this._allowedPortsLabel, this._allowedPortsTextbox], - disabled - ); + this.updateFirewall(); }, - // pushes settings from UI to tor - _applySettings() { - const type = this._useProxyCheckbox.checked - ? parseInt(this._proxyTypeMenulist.value) - : TorProxyType.Invalid; - const address = this._proxyAddressTextbox.value; - const port = this._proxyPortTextbox.value; - const username = this._proxyUsernameTextbox.value; - const password = this._proxyPasswordTextbox.value; - const settings = { proxy: {}, firewall: {} }; - switch (type) { - case TorProxyType.Invalid: - settings.proxy.enabled = false; - break; - case TorProxyType.Socks4: - settings.proxy.enabled = true; - settings.proxy.type = type; - settings.proxy.address = address; - settings.proxy.port = port; - settings.proxy.username = ""; - settings.proxy.password = ""; - break; - case TorProxyType.Socks5: - settings.proxy.enabled = true; - settings.proxy.type = type; - settings.proxy.address = address; - settings.proxy.port = port; - settings.proxy.username = username; - settings.proxy.password = password; - break; - case TorProxyType.HTTPS: - settings.proxy.enabled = true; - settings.proxy.type = type; - settings.proxy.address = address; - settings.proxy.port = port; - settings.proxy.username = username; - settings.proxy.password = password; - break; - } - - let portListString = this._useFirewallCheckbox.checked - ? this._allowedPortsTextbox.value - : ""; - if (portListString) { - settings.firewall.enabled = true; - settings.firewall.allowed_ports = portListString; + /** + * Update the dialog's stored firewall values. + */ + updateFirewall() { + if (this._useFirewallCheckbox.checked) { + const portList = []; + let listInvalid = false; + for (const portStr of this._allowedPortsTextbox.value.split( + /(?:\s*,\s*)+/g + )) { + if (!portStr) { + // Trailing or leading comma. + continue; + } + const port = this.parsePort(portStr); + if (port === null) { + listInvalid = true; + break; + } + portList.push(port); + } + if (!listInvalid && portList.length) { + this._settings.firewall = { + enabled: true, + allowed_ports: portList, + }; + } else { + this._settings.firewall = null; + } } else { - settings.firewall.enabled = false; + this._settings.firewall = { enabled: false }; } - // FIXME: What if this fails? Should we prevent the dialog to close and show - // an error? - TorSettings.changeSettings(settings); + this.updateAcceptButton(); }, }; ===================================== browser/components/torpreferences/content/connectionSettingsDialog.xhtml ===================================== @@ -61,6 +61,7 @@ <html:input id="torPreferences-localProxy-textboxAddress" type="text" + required="required" class="torMarginFix" data-l10n-id="tor-advanced-dialog-proxy-address-input" /> @@ -75,7 +76,8 @@ class="proxy-port-input torMarginFix" hidespinbuttons="true" type="number" - min="0" + required="required" + min="1" max="65535" maxlength="5" /> @@ -121,11 +123,14 @@ /> </hbox> <hbox id="torPreferences-connection-hboxAllowedPorts" align="center"> +  <html:input id="torPreferences-connection-textboxAllowedPorts" type="text" + required="required" + pattern="^(\s*,\s*)*[1-9][0-9]*((\s*,\s*)|([1-9][0-9]*))*$" class="torMarginFix" - value="80,443" data-l10n-id="tor-advanced-dialog-firewall-ports-input" /> </hbox> ===================================== browser/modules/TorSettingsNotification.sys.mjs ===================================== @@ -0,0 +1,167 @@ +const lazy = {}; + +ChromeUtils.defineESModuleGetters(lazy, { + BrowserWindowTracker: "resource:///modules/BrowserWindowTracker.sys.mjs", + TorSettings: "resource://gre/modules/TorSettings.sys.mjs", + TorSettingsTopics: "resource://gre/modules/TorSettings.sys.mjs", +}); + +ChromeUtils.defineLazyGetter(lazy, "NotificationStrings", function () { + return new Localization(["toolkit/global/tor-browser.ftl"]); +}); + +/** + * Shows a notification whenever we get an ApplyError. + */ +export const TorSettingsNotification = { + /** + * Whether we have already been initialised. + * + * @type {boolean} + */ + _initialized: false, + + /** + * Called when the UI is ready to show a notification. + */ + ready() { + if (this._initialized) { + return; + } + this._initialized = true; + Services.obs.addObserver(this, lazy.TorSettingsTopics.ApplyError); + + // Show the notification for each group of settings if they have an error + // that was triggered prior to `ready` being called. + this.showNotification("bridges"); + this.showNotification("proxy"); + this.showNotification("firewall"); + }, + + observe(subject, topic) { + if (topic === lazy.TorSettingsTopics.ApplyError) { + this.showNotification(subject.wrappedJSObject.group); + } + }, + + /** + * A promise for the `showNotification` method to ensure we only show one + * notification at a time. + * + * @type {?Promise} + */ + _notificationPromise: null, + + /** + * Show a notification for the given group of settings if `TorSettings` has an + * error for them. + * + * @param {string} group - The settings group to show the notification for. + */ + async showNotification(group) { + const prevNotificationPromise = this._notificationPromise; + let notificationComplete; + ({ promise: this._notificationPromise, resolve: notificationComplete } = + Promise.withResolvers()); + // Only want to show one notification at a time, so queue behind the + // previous one. + await prevNotificationPromise; + + // NOTE: We only show the notification for a single `group` at a time, even + // when TorSettings has errors for multiple groups. This keeps the strings + // simple and means we can show different buttons depending on `canUndo` for + // each group individually. + // If we do have multiple errors the notification for each group will simply + // queue behind each other. + try { + // Grab the latest error value, which may have changed since + // showNotification was first called. + const error = lazy.TorSettings.getApplyError(group); + if (!error) { + // No current error for this group. + return; + } + + const { canUndo } = error; + + let titleId; + let introId; + switch (group) { + case "bridges": + titleId = "tor-settings-failed-notification-title-bridges"; + introId = "tor-settings-failed-notification-cause-bridges"; + break; + case "proxy": + titleId = "tor-settings-failed-notification-title-proxy"; + introId = "tor-settings-failed-notification-cause-proxy"; + break; + case "firewall": + titleId = "tor-settings-failed-notification-title-firewall"; + introId = "tor-settings-failed-notification-cause-firewall"; + break; + } + + const [ + titleText, + introText, + bodyText, + primaryButtonText, + secondaryButtonText, + ] = await lazy.NotificationStrings.formatValues([ + { id: titleId }, + { id: introId }, + { + id: canUndo + ? "tor-settings-failed-notification-body-undo" + : "tor-settings-failed-notification-body-default", + }, + { + id: canUndo + ? "tor-settings-failed-notification-button-undo" + : "tor-settings-failed-notification-button-clear", + }, + { id: "tor-settings-failed-notification-button-fix-myself" }, + ]); + + const propBag = await Services.prompt.asyncConfirmEx( + lazy.BrowserWindowTracker.getTopWindow()?.browsingContext ?? null, + Services.prompt.MODAL_TYPE_INTERNAL_WINDOW, + titleText, + // Concatenate the intro text and the body text. Really these should be + // separate paragraph elements, but the prompt service does not support + // this. We split them with a double newline, which will hopefully avoid + // the usual problems with concatenating localised strings. + `${introText}\n\n${bodyText}`, + Services.prompt.BUTTON_TITLE_IS_STRING * Services.prompt.BUTTON_POS_0 + + Services.prompt.BUTTON_TITLE_IS_STRING * Services.prompt.BUTTON_POS_1, + primaryButtonText, + secondaryButtonText, + null, + null, + null, + {} + ); + + const buttonNum = propBag.get("buttonNumClicked"); + + if (buttonNum === 0) { + if (canUndo) { + // Wait for these methods in case they resolve the error for a pending + // showNotification call. + await lazy.TorSettings.undoFailedSettings(group); + } else { + await lazy.TorSettings.clearFailedSettings(group); + } + } else if (buttonNum === 1) { + let win = lazy.BrowserWindowTracker.getTopWindow(); + if (!win) { + win = await lazy.BrowserWindowTracker.promiseOpenWindow(); + } + // Open the preferences or switch to its tab and highlight the Tor log. + win.openPreferences("connection-viewlogs"); + } + } finally { + notificationComplete(); + } + }, +}; ===================================== browser/modules/moz.build ===================================== @@ -129,6 +129,7 @@ EXTRA_JS_MODULES += [ "SelectionChangedMenulist.sys.mjs", "SiteDataManager.sys.mjs", "SitePermissions.sys.mjs", + "TorSettingsNotification.sys.mjs", "TorUIUtils.sys.mjs", "TransientPrefs.sys.mjs", "URILoadingHelper.sys.mjs", ===================================== toolkit/components/tor-launcher/TorProvider.sys.mjs ===================================== @@ -210,7 +210,7 @@ export class TorProvider { if (this.ownsTorDaemon) { try { await lazy.TorSettings.initializedPromise; - await this.writeSettings(); + await lazy.TorSettings.setTorProvider(this); } catch (e) { logger.warn( "Failed to initialize TorSettings or to write our initial settings. Continuing the initialization anyway.", @@ -252,44 +252,65 @@ export class TorProvider { /** * Send settings to the tor daemon. * - * This should only be called internally or by the TorSettings module. + * @param {Map<string, ?string>} torSettings - The key value pairs to pass in. */ - async writeSettings() { - // Fetch the current settings. - // We set the useTemporary parameter since we want to apply temporary - // bridges if they are available. - const settings = lazy.TorSettings.getSettings(true); - logger.debug("TorProvider.writeSettings", settings); + async #writeSettings(torSettings) { + logger.debug("Mapped settings object", torSettings); + + // NOTE: The order in which TorProvider.#writeSettings should match the + // order in which the configuration is passed onto setConf. In turn, + // TorControlPort.setConf should similarly ensure that the configuration + // reaches the tor process in the same order. + // In particular, we do not want a race where an earlier call to + // TorProvider.#writeSettings for overlapping settings can be delayed and + // override a later call. + await this.#controller.setConf(Array.from(torSettings)); + } + + /** + * Send bridge settings to the tor daemon. + * + * This should only be called by the `TorSettings` module. + * + * @param {TorBridgeSettings} bridges - The bridge settings to apply. + */ + async writeBridgeSettings(bridges) { + logger.debug("TorProvider.writeBridgeSettings", bridges); const torSettings = new Map(); // Bridges - const haveBridges = - settings.bridges?.enabled && !!settings.bridges.bridge_strings.length; + const haveBridges = bridges?.enabled && !!bridges.bridge_strings.length; torSettings.set(TorConfigKeys.useBridges, haveBridges); - if (haveBridges) { - torSettings.set( - TorConfigKeys.bridgeList, - settings.bridges.bridge_strings - ); - } else { - torSettings.set(TorConfigKeys.bridgeList, null); - } + torSettings.set( + TorConfigKeys.bridgeList, + haveBridges ? bridges.bridge_strings : null + ); + + await this.#writeSettings(torSettings); + } + + /** + * Send proxy settings to the tor daemon. + * + * This should only be called by the `TorSettings` module. + * + * @param {TorProxySettings} proxy - The proxy settings to apply. + */ + async writeProxySettings(proxy) { + logger.debug("TorProvider.writeProxySettings", proxy); + const torSettings = new Map(); - // Proxy torSettings.set(TorConfigKeys.socks4Proxy, null); torSettings.set(TorConfigKeys.socks5Proxy, null); torSettings.set(TorConfigKeys.socks5ProxyUsername, null); torSettings.set(TorConfigKeys.socks5ProxyPassword, null); torSettings.set(TorConfigKeys.httpsProxy, null); torSettings.set(TorConfigKeys.httpsProxyAuthenticator, null); - if (settings.proxy && !settings.proxy.enabled) { - settings.proxy.type = null; - } - const address = settings.proxy?.address; - const port = settings.proxy?.port; - const username = settings.proxy?.username; - const password = settings.proxy?.password; - switch (settings.proxy?.type) { + + const type = proxy.enabled ? proxy.type : null; + const { address, port, username, password } = proxy; + + switch (type) { case lazy.TorProxyType.Socks4: torSettings.set(TorConfigKeys.socks4Proxy, `${address}:${port}`); break; @@ -307,27 +328,28 @@ export class TorProvider { break; } - // Firewall - if (settings.firewall?.enabled) { - const reachableAddresses = settings.firewall.allowed_ports - .map(port => `*:${port}`) - .join(","); - torSettings.set(TorConfigKeys.reachableAddresses, reachableAddresses); - } else { - torSettings.set(TorConfigKeys.reachableAddresses, null); - } + await this.#writeSettings(torSettings); + } - logger.debug("Mapped settings object", settings, torSettings); + /** + * Send firewall settings to the tor daemon. + * + * This should only be called by the `TorSettings` module. + * + * @param {TorFirewallSettings} firewall - The firewall settings to apply. + */ + async writeFirewallSettings(firewall) { + logger.debug("TorProvider.writeFirewallSettings", firewall); + const torSettings = new Map(); - // Send settings to the tor process. - // NOTE: Since everything up to this point has been non-async, the order in - // which TorProvider.writeSettings is called should match the order in which - // the configuration is passed onto setConf. In turn, TorControlPort.setConf - // should similarly ensure that the configuration reaches the tor process in - // the same order. - // In particular, we do not want a race where an earlier call to - // TorProvider.writeSettings can be delayed and override a later call. - await this.#controller.setConf(Array.from(torSettings)); + torSettings.set( + TorConfigKeys.reachableAddresses, + firewall.enabled + ? firewall.allowed_ports.map(port => `*:${port}`).join(",") + : null + ); + + await this.#writeSettings(torSettings); } async flushSettings() { ===================================== toolkit/locales/en-US/toolkit/global/tor-browser.ftl ===================================== @@ -467,6 +467,29 @@ tor-advanced-dialog-firewall-ports-input-label = Allowed ports tor-advanced-dialog-firewall-ports-input = .placeholder = Comma-separated values +## Tor settings error notification. + +# Shown when the user's Tor bridge settings could not be passed on to the Tor daemon. +tor-settings-failed-notification-title-bridges = Your Tor bridge settings could not be applied +# Shown when the user's Tor bridge settings could not be passed on to the Tor daemon. +tor-settings-failed-notification-cause-bridges = This could be due to an invalid bridge address. +# Shown when the user's Tor proxy settings could not be passed on to the Tor daemon. +tor-settings-failed-notification-title-proxy = Your Tor proxy settings could not be applied +# Shown when the user's Tor proxy settings could not be passed on to the Tor daemon. +tor-settings-failed-notification-cause-proxy = This could be due to invalid proxy information. +# Shown when the user's Tor firewall settings could not be passed on to the Tor daemon. +tor-settings-failed-notification-title-firewall = Your Tor firewall settings could not be applied +# Shown when the user's Tor firewall settings could not be passed on to the Tor daemon. +tor-settings-failed-notification-cause-firewall = This could be due to invalid firewall information. +tor-settings-failed-notification-body-undo = Until fixed, your Tor connection will continue to use your previous settings. You can either undo the latest changes to restore the previous working settings or check the Tor log to find and fix the issue yourself. +tor-settings-failed-notification-body-default = Until fixed, your Tor connection will continue to use default settings. You can either clear the problematic settings to restore them to default or check the Tor log to find and fix the issue yourself. +# Button to revert the latest user settings. +tor-settings-failed-notification-button-undo = Undo changes +# Button to clear the user settings. +tor-settings-failed-notification-button-clear = Clear +# Button for the user to declare that they will fix the problematic settings by themself. +tor-settings-failed-notification-button-fix-myself = Fix myself + ## About Tor Browser dialog. # '<label data-l10n-name="project-link">' and '</label>' should wrap the link text for the Tor Project, and will link to the Tor Project web page. ===================================== toolkit/modules/TorSettings.sys.mjs ===================================== @@ -9,7 +9,6 @@ ChromeUtils.defineESModuleGetters(lazy, { Lox: "resource://gre/modules/Lox.sys.mjs", LoxTopics: "resource://gre/modules/Lox.sys.mjs", TorParsers: "resource://gre/modules/TorParsers.sys.mjs", - TorProviderBuilder: "resource://gre/modules/TorProviderBuilder.sys.mjs", }); ChromeUtils.defineLazyGetter(lazy, "logger", () => { @@ -23,6 +22,7 @@ ChromeUtils.defineLazyGetter(lazy, "logger", () => { export const TorSettingsTopics = Object.freeze({ Ready: "torsettings:ready", SettingsChanged: "torsettings:settings-changed", + ApplyError: "torsettings:apply-error", }); /* Prefs used to store settings in TorBrowser prefs */ @@ -162,44 +162,74 @@ function arrayShuffle(array) { return array; } +/** + * @typedef {Object} TorBridgeSettings + * + * Represents the Tor bridge settings. + * + * @property {boolean} enabled - Whether bridges are enabled. + * @property {integer} source - The source of bridges. One of the values in + * `TorBridgeSource`. + * @property {string} lox_id - The ID of the lox credentials to get bridges for. + * Or "" if not using a `Lox` `source`. + * @property {string} builtin_type - The name of the built-in bridge type. Or "" + * if not using a `BuiltIn` `source`. + * @property {string[]} bridge_strings - The bridge lines to be passed to the + * provider. Should be empty if and only if the `source` is `Invalid`. + */ + +/** + * @typedef {Object} TorProxySettings + * + * Represents the Tor proxy settings. + * + * @property {boolean} enabled - Whether the proxy should be enabled. + * @property {integer} type - The proxy type. One of the values in + * `TorProxyType`. + * @property {string} address - The proxy address, or "" if the proxy is not + * being used. + * @property {integer} port - The proxy port, or 0 if the proxy is not being + * used. + * @property {string} username - The proxy user name, or "" if this is not + * needed. + * @property {string} password - The proxy password, or "" if this is not + * needed. + */ + +/** + * @typedef {Object} TorFirewallSettings + * + * Represents the Tor firewall settings. + * + * @property {boolean} enabled - Whether the firewall settings should be + * enabled. + * @property {integer[]} allowed_ports - The list of ports that are allowed. + */ + +/** + * @typedef {Object} TorCombinedSettings + * + * A combination of Tor settings. + * + * @property {TorBridgeSettings} bridges - The bridge settings. + * @property {TorProxySettings} proxy - The proxy settings. + * @property {TorFirewallSettings} firewall - The firewall settings. + */ + /* TorSettings module */ class TorSettingsImpl { /** - * The underlying settings values. + * The default settings to use. * - * @type {object} + * @type {TorCombinedSettings} */ - #settings = { + #defaultSettings = { bridges: { - /** - * Whether the bridges are enabled or not. - * - * @type {boolean} - */ enabled: false, source: TorBridgeSource.Invalid, - /** - * The lox id is used with the Lox "source", and remains set with the - * stored value when other sources are used. - * - * @type {string} - */ lox_id: "", - /** - * The built-in type to use when using the BuiltIn "source", or empty when - * using any other source. - * - * @type {string} - */ builtin_type: "", - /** - * The current bridge strings. - * - * Can only be non-empty if the "source" is not Invalid. - * - * @type {Array<string>} - */ bridge_strings: [], }, proxy: { @@ -216,10 +246,71 @@ class TorSettingsImpl { }, }; + /** + * The underlying settings values. + * + * @type {TorCombinedSettings} + */ + #settings = structuredClone(this.#defaultSettings); + + /** + * The last successfully applied settings for the current `TorProvider`, if + * any. + * + * NOTE: Should only be written to within `#applySettings`. + * + * @type {{ + * bridges: ?TorBridgeSettings, + * proxy: ?TorProxySettings, + * firewall: ?TorFirewallSettings + * }} + */ + #successfulSettings = { bridges: null, proxy: null, firewall: null }; + + /** + * Whether temporary bridge settings have been applied to the current + * `TorProvider`. + * + * @type {boolean} + */ + #temporaryBridgesApplied = false; + + /** + * @typedef {TorSettingsApplyError} + * + * @property {boolean} canUndo - Whether the latest error can be "undone". + * When this is `false`, the TorProvider will be using its default values + * instead. + */ + + /** + * A summary of the latest failures to apply our settings, if any. + * + * NOTE: Should only be written to within `#applySettings`. + * + * @type {{ + * bridges: ?TorSettingsApplyError, + * proxy: ?TorSettingsApplyError, + * firewall: ?TorSettingsApplyError, + * }} + */ + #applyErrors = { bridges: null, proxy: null, firewall: null }; + + /** + * Get the latest failure for the given setting, if any. + * + * @param {string} group - The settings to get the error details for. + * + * @returns {?TorSettingsApplyError} - The error details, if any. + */ + getApplyError(group) { + return structuredClone(this.#applyErrors[group]); + } + /** * Temporary bridge settings to apply instead of #settings.bridges. * - * @type {?Object} + * @type {?TorBridgeSettings} */ #temporaryBridgeSettings = null; @@ -342,38 +433,39 @@ class TorSettingsImpl { } /** - * Regular expression for a decimal non-negative integer. + * Verify a port number is within bounds. * - * @type {RegExp} + * @param {integer} val - The value to verify. + * @return {boolean} - Whether the port is within range. */ - #portRegex = /^[0-9]+$/; + validPort(val) { + return Number.isInteger(val) && val >= 1 && val <= 65535; + } + /** - * Parse a string as a port number. + * Verify that some SOCKS5 credentials are valid. * - * @param {string|integer} val - The value to parse. - * @param {boolean} trim - Whether a string value can be stripped of - * whitespace before parsing. - * - * @return {integer?} - The port number, or null if the given value was not - * valid. + * @param {string} username - The SOCKS5 username. + * @param {string} password - The SOCKS5 password. + * @return {boolean} - Whether the credentials are valid. */ - #parsePort(val, trim) { - if (typeof val === "string") { - if (trim) { - val = val.trim(); + validSocks5Credentials(username, password) { + if (!username && !password) { + // Both empty is valid. + return true; + } + for (const val of [username, password]) { + if (typeof val !== "string") { + return false; } - // ensure port string is a valid positive integer - if (this.#portRegex.test(val)) { - val = Number.parseInt(val, 10); - } else { - throw new Error(`Invalid port string "${val}"`); + const byteLen = new TextEncoder().encode(val).length; + if (byteLen < 1 || byteLen > 255) { + return false; } } - if (!Number.isInteger(val) || val < 1 || val > 65535) { - throw new Error(`Port out of range: ${val}`); - } - return val; + return true; } + /** * Test whether two arrays have equal members and order. * @@ -659,10 +751,11 @@ class TorSettingsImpl { false ); if (firewall.enabled) { - firewall.allowed_ports = Services.prefs.getStringPref( - TorSettingsPrefs.firewall.allowed_ports, - "" - ); + firewall.allowed_ports = Services.prefs + .getStringPref(TorSettingsPrefs.firewall.allowed_ports, "") + .split(",") + .filter(p => p.trim()) + .map(p => parseInt(p, 10)); } try { this.#fixupFirewallSettings(firewall); @@ -767,19 +860,221 @@ class TorSettingsImpl { } } + /** + * A blocker promise for the #applySettings method. + * + * Ensures only one active caller to protect the #applyErrors and + * #successfulSettings properties. + * + * @type {?Promise} + */ + #applySettingsTask = null; + /** * Push our settings down to the tor provider. * * Even though this introduces a circular depdency, it makes the API nicer for * frontend consumers. * - * @param {boolean} flush - Whether to also flush the settings to disk. + * @param {Object} apply - The list of settings to apply. + * @param {boolean} [apply.bridges] - Whether to apply our bridge settings. + * @param {boolean} [apply.proxy] - Whether to apply our proxy settings. + * @param {boolean} [apply.firewall] - Whether to apply our firewall settings. + * @param {boolean} [details] - Optional details. + * @param {boolean} [details.useTemporaryBridges] - Whether the caller wants + * to apply temporary bridges. + * @param {boolean} [details.newProvider] - Whether the caller is initialising + * a new `TorProvider`. */ - async #applySettings(flush) { - const provider = await lazy.TorProviderBuilder.build(); - await provider.writeSettings(); - if (flush) { - provider.flushSettings(); + async #applySettings(apply, details) { + // Grab this provider before awaiting. + // In particular, if the provider is changed we do not want to switch to + // writing to the new instance. + const providerRef = this.#providerRef; + const provider = providerRef?.deref(); + if (!provider) { + // Wait until setTorProvider is called. + lazy.logger.info("No TorProvider yet"); + return; + } + + // We only want one instance of #applySettings to be running at any given + // time, so we await the previous call first. + // Read and replace #applySettingsTask before we do any async operations. + // I.e. this is effectively an atomic read and replace. + const prevTask = this.#applySettingsTask; + let taskComplete; + ({ promise: this.#applySettingsTask, resolve: taskComplete } = + Promise.withResolvers()); + await prevTask; + + try { + let flush = false; + const errors = []; + + // Test whether the provider is no longer running or has been replaced. + const providerRunning = () => { + return providerRef === this.#providerRef && provider.isRunning; + }; + + lazy.logger.debug("Passing on settings to the provider", apply, details); + + if (details?.newProvider) { + // If we have a new provider we clear our successful settings. + // In particular, the user may have changed their settings several times + // whilst the tor process was not running. In the event of an + // "ApplyError", we want to correctly show to the user that they are now + // using default settings and we do not want to allow them to "undo" + // since the previous successful settings may be out of date. + // NOTE: We do not do this within `setTorProvider` since some other + // caller's `#applySettingsTask` may still be running and writing to + // these values when `setTorProvider` is called. + this.#successfulSettings.bridges = null; + this.#successfulSettings.proxy = null; + this.#successfulSettings.firewall = null; + this.#applyErrors.bridges = null; + this.#applyErrors.proxy = null; + this.#applyErrors.firewall = null; + // Temporary bridges are not applied to the new provider. + this.#temporaryBridgesApplied = false; + } + + for (const group of ["bridges", "proxy", "firewall"]) { + if (!apply[group]) { + continue; + } + + if (!providerRunning()) { + lazy.logger.info("The TorProvider is no longer running"); + // Bail on this task since the old provider should not accept + // settings. setTorProvider will be called for the new provider and + // will already handle applying the same settings. + return; + } + + let usingSettings = true; + if (group === "bridges") { + // Only record successes or failures when using the user settings. + if (this.#temporaryBridgeSettings && !details?.useTemporaryBridges) { + // #temporaryBridgeSettings were written whilst awaiting the + // previous task. Do nothing and allow applyTemporarySettings to + // apply the temporary bridges instead. + lazy.logger.info( + "Not apply bridges since temporary bridges were applied" + ); + continue; + } + if (!this.#temporaryBridgeSettings && details?.useTemporaryBridges) { + // #temporaryBridgeSettings were cleared whilst awaiting the + // previous task. Do nothing and allow changeSettings or + // clearTemporaryBridges to apply the non-temporary bridges + // instead. + lazy.logger.info( + "Not apply temporary bridges since they were cleared" + ); + continue; + } + usingSettings = !details?.useTemporaryBridges; + } + + try { + switch (group) { + case "bridges": { + const bridges = structuredClone( + usingSettings + ? this.#settings.bridges + : this.#temporaryBridgeSettings + ); + + try { + await provider.writeBridgeSettings(bridges); + } catch (e) { + if ( + usingSettings && + this.#temporaryBridgesApplied && + providerRunning() + ) { + lazy.logger.warn( + "Recovering to clear temporary bridges from the provider" + ); + // The TorProvider is still using the temporary bridges. As a + // priority we want to try and restore the TorProvider to the + // state it was in prior to the temporary bridges being + // applied. + const prevBridges = structuredClone( + this.#successfulSettings.bridges ?? + this.#defaultSettings.bridges + ); + try { + await provider.writeBridgeSettings(prevBridges); + this.#temporaryBridgesApplied = false; + } catch (e) { + lazy.logger.error( + "Failed to clear the temporary bridges from the provider", + e + ); + } + } + throw e; + } + + if (usingSettings) { + this.#successfulSettings.bridges = bridges; + this.#temporaryBridgesApplied = false; + this.#applyErrors.bridges = null; + flush = true; + } else { + this.#temporaryBridgesApplied = true; + // Do not flush the temporary bridge settings until they are + // saved. + } + break; + } + case "proxy": { + const proxy = structuredClone(this.#settings.proxy); + await provider.writeProxySettings(proxy); + this.#successfulSettings.proxy = proxy; + this.#applyErrors.proxy = null; + flush = true; + break; + } + case "firewall": { + const firewall = structuredClone(this.#settings.firewall); + await provider.writeFirewallSettings(firewall); + this.#successfulSettings.firewall = firewall; + this.#applyErrors.firewall = null; + flush = true; + break; + } + } + } catch (e) { + // Store the error and throw later. + errors.push(e); + if (usingSettings && providerRunning()) { + // Record and signal the error. + // NOTE: We do not signal ApplyError when we fail to apply temporary + // bridges. + this.#applyErrors[group] = { + canUndo: Boolean(this.#successfulSettings[group]), + }; + lazy.logger.debug(`Signalling new ApplyError for ${group}`); + Services.obs.notifyObservers( + { group }, + TorSettingsTopics.ApplyError + ); + } + } + } + if (flush && providerRunning()) { + provider.flushSettings(); + } + if (errors.length) { + lazy.logger.error("Failed to apply settings", errors); + throw new Error(`Failed to apply settings. ${errors.join(". ")}.`); + } + } finally { + // Allow the next caller to proceed. + taskComplete(); } } @@ -869,7 +1164,30 @@ class TorSettingsImpl { if (!Object.values(TorProxyType).includes(proxy.type)) { throw new Error(`Invalid proxy type: ${proxy.type}`); } - proxy.port = this.#parsePort(proxy.port, false); + // Do not allow port value to be 0. + // Whilst Socks4Proxy, Socks5Proxy and HTTPSProxyPort allow you to pass in + // `<address>:0` this will select a default port. Our UI does not indicate + // that `0` maps to a different value, so we disallow it. + if (!this.validPort(proxy.port)) { + throw new Error(`Invalid proxy port: ${proxy.port}`); + } + + switch (proxy.type) { + case TorProxyType.Socks4: + // Never use the username or password. + proxy.username = ""; + proxy.password = ""; + break; + case TorProxyType.Socks5: + if (!this.validSocks5Credentials(proxy.username, proxy.password)) { + throw new Error("Invalid SOCKS5 credentials"); + } + break; + case TorProxyType.HTTPS: + // username and password are both optional. + break; + } + proxy.address = String(proxy.address); proxy.username = String(proxy.username); proxy.password = String(proxy.password); @@ -890,22 +1208,82 @@ class TorSettingsImpl { return; } - let allowed_ports = firewall.allowed_ports; - if (!Array.isArray(allowed_ports)) { - allowed_ports = allowed_ports === "" ? [] : allowed_ports.split(","); + if (!Array.isArray(firewall.allowed_ports)) { + throw new Error("allowed_ports should be an array of ports"); } - // parse and remove duplicates - const portSet = new Set(); - - for (const port of allowed_ports) { - try { - portSet.add(this.#parsePort(port, true)); - } catch (e) { - // Do not throw for individual ports. - lazy.logger.error(`Failed to parse the port ${port}. Ignoring.`, e); + for (const port of firewall.allowed_ports) { + if (!this.validPort(port)) { + throw new Error(`Invalid firewall port: ${port}`); } } - firewall.allowed_ports = [...portSet]; + // Remove duplicates + firewall.allowed_ports = [...new Set(firewall.allowed_ports)]; + } + + /** + * The current `TorProvider` instance we are using, if any. + * + * @type {?WeakRef<TorProvider>} + */ + #providerRef = null; + + /** + * Called whenever we have a new provider to send settings to. + * + * @param {TorProvider} provider - The provider to apply our settings to. + */ + async setTorProvider(provider) { + lazy.logger.debug("Applying settings to new provider"); + this.#checkIfInitialized(); + + // Use a WeakRef to not keep the TorProvider instance alive. + this.#providerRef = new WeakRef(provider); + // NOTE: We need the caller to pass in the TorProvider instance because + // TorProvider's initialisation waits for this method. In particular, we + // cannot await TorProviderBuilder.build since it would hang! + await this.#applySettings( + { bridges: true, proxy: true, firewall: true }, + { newProvider: true } + ); + } + + /** + * Undo settings that have failed to be applied by restoring the last + * successfully applied settings instead. + * + * @param {string} group - The group to undo the settings for. + */ + async undoFailedSettings(group) { + if (!this.#applyErrors[group]) { + lazy.logger.warn( + `${group} settings have already been successfully replaced.` + ); + return; + } + if (!this.#successfulSettings[group]) { + // Unexpected. + lazy.logger.warn( + `Cannot undo ${group} settings since we have no successful settings.` + ); + return; + } + await this.changeSettings({ [group]: this.#successfulSettings[group] }); + } + + /** + * Clear settings that have failed to be applied by using the default settings + * instead. + * + * @param {string} group - The group to clear the settings for. + */ + async clearFailedSettings(group) { + if (!this.#applyErrors[group]) { + lazy.logger.warn( + `${group} settings have already been successfully replaced.` + ); + return; + } + await this.changeSettings({ [group]: this.#defaultSettings[group] }); } /** @@ -919,8 +1297,8 @@ class TorSettingsImpl { * + proxy settings can be set as a group. * + firewall settings can be set a group. * - * @param {object} newValues - The new setting values, a subset of the - * complete settings that should be changed. + * @param {object} newValues - The new setting values that should be changed. + * A subset of the `TorCombinedSettings` object. */ async changeSettings(newValues) { lazy.logger.debug("changeSettings()", newValues); @@ -932,6 +1310,7 @@ class TorSettingsImpl { const completeSettings = structuredClone(this.#settings); const changes = []; + const apply = {}; /** * Change the given setting to a new value. Does nothing if the new value @@ -950,10 +1329,11 @@ class TorSettingsImpl { } completeSettings[group][prop] = value; changes.push(`${group}.${prop}`); + // Apply these settings. + apply[group] = true; }; if ("bridges" in newValues) { - const changesLength = changes.length; if ("source" in newValues.bridges) { this.#fixupBridgeSettings(newValues.bridges); changeSetting("bridges", "source", newValues.bridges.source); @@ -984,7 +1364,7 @@ class TorSettingsImpl { changeSetting("bridges", "enabled", newValues.bridges.enabled); } - if (this.#temporaryBridgeSettings && changes.length !== changesLength) { + if (this.#temporaryBridgeSettings && apply.bridges) { // A change in the bridges settings. // We want to clear the temporary bridge settings to ensure that they // cannot be used to overwrite these user-provided settings. @@ -1034,34 +1414,22 @@ class TorSettingsImpl { lazy.logger.debug("setSettings result", this.#settings, changes); - // After we have sent out the notifications for the changed settings and - // saved the preferences we send the new settings to TorProvider. - // Some properties are unread by TorProvider. So if only these values change - // there is no need to re-apply the settings. - const unreadProps = ["bridges.builtin_type", "bridges.lox_id"]; - const shouldApply = changes.some(prop => !unreadProps.includes(prop)); - if (shouldApply) { - await this.#applySettings(true); + if (apply.bridges || apply.proxy || apply.firewall) { + // After we have sent out the notifications for the changed settings and + // saved the preferences we send the new settings to TorProvider. + await this.#applySettings(apply); } } /** * Get a copy of all our settings. * - * @param {boolean} [useTemporary=false] - Whether the returned settings - * should use the temporary bridge settings, if any, instead. - * - * @returns {object} A copy of the settings object + * @returns {TorCombinedSettings} A copy of the current settings. */ - getSettings(useTemporary = false) { + getSettings() { lazy.logger.debug("getSettings()"); this.#checkIfInitialized(); - const settings = structuredClone(this.#settings); - if (useTemporary && this.#temporaryBridgeSettings) { - // Override the bridge settings with our temporary ones. - settings.bridges = structuredClone(this.#temporaryBridgeSettings); - } - return settings; + return structuredClone(this.#settings); } /** @@ -1109,8 +1477,8 @@ class TorSettingsImpl { // After checks are complete, we commit them. this.#temporaryBridgeSettings = bridgeSettings; - // Do not flush the temporary bridge settings until they are saved. - await this.#applySettings(false); + + await this.#applySettings({ bridges: true }, { useTemporaryBridges: true }); } /** @@ -1137,7 +1505,7 @@ class TorSettingsImpl { return; } this.#temporaryBridgeSettings = null; - await this.#applySettings(); + await this.#applySettings({ bridges: true }); } } View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/compare/0f529e… -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/compare/0f529e… You're receiving this email because of your account on gitlab.torproject.org.

1 0

[Git][tpo/applications/tor-browser][tor-browser-128.8.0esr-14.5-1] 2 commits: fixup! TB 40933: Add tor-launcher functionality
by henry (＠henry) 18 Mar '25

18 Mar '25

henry pushed to branch tor-browser-128.8.0esr-14.5-1 at The Tor Project / Applications / Tor Browser Commits: 480957e5 by Henry Wilkes at 2025-03-18T15:21:16+00:00 fixup! TB 40933: Add tor-launcher functionality TB 42300: Store TorProvider log messages in TorProviderBuilder to be used between instances and to be available whilst a provider is not available. - - - - - 0f529e6f by Henry Wilkes at 2025-03-18T15:21:16+00:00 fixup! TB 31286: Implementation of bridge, proxy, and firewall settings in about:preferences#connection TB 42300: Fetch tor logs from TorProviderBuilder. - - - - - 3 changed files: - browser/components/torpreferences/content/torLogDialog.js - toolkit/components/tor-launcher/TorProvider.sys.mjs - toolkit/components/tor-launcher/TorProviderBuilder.sys.mjs Changes: ===================================== browser/components/torpreferences/content/torLogDialog.js ===================================== @@ -66,28 +66,22 @@ const gTorLogDialog = { ); }); - // A waiting state should not be needed at this point. - // Also, we probably cannot even arrive here if the provider failed to - // initialize, otherwise we could use a try/catch, and write the exception - // text in the logs, instead. - TorProviderBuilder.build().then(provider => { - Services.obs.addObserver(this, TorProviderTopics.TorLog); - window.addEventListener( - "unload", - () => { - Services.obs.removeObserver(this, TorProviderTopics.TorLog); - }, - { once: true } - ); - - for (const logEntry of provider.getLog()) { - this.addLogEntry(logEntry, true); - } - // Set the initial scroll to the bottom. - this._logTable.scrollTo({ - top: this._logTable.scrollTopMax, - behaviour: "instant", - }); + Services.obs.addObserver(this, TorProviderTopics.TorLog); + window.addEventListener( + "unload", + () => { + Services.obs.removeObserver(this, TorProviderTopics.TorLog); + }, + { once: true } + ); + + for (const logEntry of TorProviderBuilder.getLog()) { + this.addLogEntry(logEntry, true); + } + // Set the initial scroll to the bottom. + this._logTable.scrollTo({ + top: this._logTable.scrollTopMax, + behaviour: "instant", }); }, ===================================== toolkit/components/tor-launcher/TorProvider.sys.mjs ===================================== @@ -45,12 +45,6 @@ const logger = console.createInstance({ * @property {string} [host] The host to connect for a TCP proxy * @property {number} [port] The port number to use for a TCP proxy */ -/** - * @typedef {object} LogEntry An object with a log message - * @property {string} timestamp The local date-time stamp at which we received the message - * @property {string} type The message level - * @property {string} msg The message - */ /** * Stores the data associated with a circuit node. * @@ -69,7 +63,6 @@ const Preferences = Object.freeze({ ControlUseIpc: "extensions.torlauncher.control_port_use_ipc", ControlHost: "extensions.torlauncher.control_host", ControlPort: "extensions.torlauncher.control_port", - MaxLogEntries: "extensions.torlauncher.max_tor_log_entries", }); /* Config Keys used to configure tor daemon */ @@ -141,15 +134,6 @@ export class TorProvider { */ #socksSettings = null; - /** - * The logs we received over the control port. - * We store a finite number of log entries which can be configured with - * extensions.torlauncher.max_tor_log_entries. - * - * @type {LogEntry[]} - */ - #logs = []; - #isBootstrapDone = false; /** * Keep the last warning to avoid broadcasting an async warning if it is the @@ -511,15 +495,6 @@ export class TorProvider { return this.#controller.onionAuthViewKeys(); } - /** - * Returns captured log messages. - * - * @returns {LogEntry[]} The logs we collected from the tor daemon so far. - */ - getLog() { - return structuredClone(this.#logs); - } - /** * @returns {boolean} true if we launched and control tor, false if we are * using system tor. @@ -1033,15 +1008,6 @@ export class TorProvider { TorProviderTopics.TorLog ); - const maxEntries = Services.prefs.getIntPref( - Preferences.MaxLogEntries, - 1000 - ); - if (maxEntries > 0 && this.#logs.length >= maxEntries) { - this.#logs.splice(0, 1); - } - - this.#logs.push({ type, msg, timestamp }); switch (type) { case "ERR": logger.error(`[Tor error] ${msg}`); ===================================== toolkit/components/tor-launcher/TorProviderBuilder.sys.mjs ===================================== @@ -23,6 +23,13 @@ export const TorProviders = Object.freeze({ tor: 1, }); +/** + * @typedef {object} LogEntry An object with a log message + * @property {string} timestamp The local date-time stamp at which we received the message + * @property {string} type The message level + * @property {string} msg The message + */ + /** * The factory to get a Tor provider. * Currently we support only TorProvider, i.e., the one that interacts with @@ -36,6 +43,48 @@ export class TorProviderBuilder { */ static #provider = null; + /** + * A record of the log messages from all TorProvider instances. + * + * @type {LogEntry[]} + */ + static #log = []; + + /** + * Get a record of historic log entries. + * + * @returns {LogEntry[]} - The record of entries. + */ + static getLog() { + return structuredClone(this.#log); + } + + /** + * The limit on the number of log entries we should store. + * + * @type {integer} + */ + static #logLimit; + + /** + * The observer that checks for new TorLog messages. + * + * @type {Function} + */ + static #logObserver; + + /** + * Add a new log message. + * + * @param {LogEntry} logEntry - The log entry to add. + */ + static #addLogEntry(logEntry) { + if (this.#logLimit > 0 && this.#log.length >= this.#logLimit) { + this.#log.splice(0, 1); + } + this.#log.push(logEntry); + } + /** * The observer that checks when the tor process exits, and reinitializes the * provider. @@ -56,6 +105,15 @@ export class TorProviderBuilder { * Initialize the provider of choice. */ static init() { + this.#logLimit = Services.prefs.getIntPref( + "extensions.torlauncher.max_tor_log_entries", + 1000 + ); + this.#logObserver = subject => { + this.#addLogEntry(subject.wrappedJSObject); + }; + Services.obs.addObserver(this.#logObserver, TorProviderTopics.TorLog); + switch (this.providerType) { case TorProviders.tor: // Even though initialization of the initial TorProvider is @@ -136,6 +194,7 @@ export class TorProviderBuilder { ); this.#exitObserver = null; } + Services.obs.removeObserver(this.#logObserver, TorProviderTopics.TorLog); } /** View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/compare/ad21bd… -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/compare/ad21bd… You're receiving this email because of your account on gitlab.torproject.org.

1 0

[Git][tpo/applications/torbrowser-launcher] Pushed new branch AppArmor-updates-for-current-Debian
by intrigeri (＠intrigeri) 18 Mar '25

18 Mar '25

intrigeri pushed new branch AppArmor-updates-for-current-Debian at The Tor Project / Applications / torbrowser-launcher -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/torbrowser-launcher/-/tree/A… You're receiving this email because of your account on gitlab.torproject.org.

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

tor-commits