- tbb-commits - lists.torproject.org

[Git][tpo/applications/tor-browser][base-browser-115.10.0esr-13.5-1] Bug 41966: Allow removing locales from the locale alternatives list.
by Pier Angelo Vendrame (＠pierov) 17 Apr '24

17 Apr '24

Pier Angelo Vendrame pushed to branch base-browser-115.10.0esr-13.5-1 at The Tor Project / Applications / Tor Browser Commits: 168e038e by Henry Wilkes at 2024-04-17T16:52:24+02:00 Bug 41966: Allow removing locales from the locale alternatives list. - - - - - 1 changed file: - browser/components/preferences/dialogs/browserLanguages.js Changes: ===================================== browser/components/preferences/dialogs/browserLanguages.js ===================================== @@ -349,7 +349,7 @@ async function getLocaleDisplayInfo(localeCodes) { id: "locale-" + code, label: localeNames[i], value: code, - canRemove: !packagedLocales.has(code), + canRemove: code !== Services.locale.defaultLocale, installed: availableLocales.has(code), }; }); View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/commit/168e038… -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/commit/168e038… You're receiving this email because of your account on gitlab.torproject.org.

1 0

[Git][tpo/applications/mullvad-browser][mullvad-browser-115.10.0esr-13.5-1] Bug 41966: Allow removing locales from the locale alternatives list.
by Pier Angelo Vendrame (＠pierov) 17 Apr '24

17 Apr '24

Pier Angelo Vendrame pushed to branch mullvad-browser-115.10.0esr-13.5-1 at The Tor Project / Applications / Mullvad Browser Commits: 2970b5b4 by Henry Wilkes at 2024-04-17T16:52:44+02:00 Bug 41966: Allow removing locales from the locale alternatives list. - - - - - 1 changed file: - browser/components/preferences/dialogs/browserLanguages.js Changes: ===================================== browser/components/preferences/dialogs/browserLanguages.js ===================================== @@ -349,7 +349,7 @@ async function getLocaleDisplayInfo(localeCodes) { id: "locale-" + code, label: localeNames[i], value: code, - canRemove: !packagedLocales.has(code), + canRemove: code !== Services.locale.defaultLocale, installed: availableLocales.has(code), }; }); View it on GitLab: https://gitlab.torproject.org/tpo/applications/mullvad-browser/-/commit/297… -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/mullvad-browser/-/commit/297… You're receiving this email because of your account on gitlab.torproject.org.

1 0

[Git][tpo/applications/tor-browser][tor-browser-115.10.0esr-13.5-1] Bug 41966: Allow removing locales from the locale alternatives list.
by Pier Angelo Vendrame (＠pierov) 17 Apr '24

17 Apr '24

Pier Angelo Vendrame pushed to branch tor-browser-115.10.0esr-13.5-1 at The Tor Project / Applications / Tor Browser Commits: 5be723a9 by Henry Wilkes at 2024-04-17T15:37:09+01:00 Bug 41966: Allow removing locales from the locale alternatives list. - - - - - 1 changed file: - browser/components/preferences/dialogs/browserLanguages.js Changes: ===================================== browser/components/preferences/dialogs/browserLanguages.js ===================================== @@ -349,7 +349,7 @@ async function getLocaleDisplayInfo(localeCodes) { id: "locale-" + code, label: localeNames[i], value: code, - canRemove: !packagedLocales.has(code), + canRemove: code !== Services.locale.defaultLocale, installed: availableLocales.has(code), }; }); View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/commit/5be723a… -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/commit/5be723a… You're receiving this email because of your account on gitlab.torproject.org.

1 0

[Git][tpo/applications/tor-browser-build][maint-13.0] Bug 41128: Fix spaces in various config files.
by Pier Angelo Vendrame (＠pierov) 17 Apr '24

17 Apr '24

Pier Angelo Vendrame pushed to branch maint-13.0 at The Tor Project / Applications / tor-browser-build Commits: 24cb8294 by Pier Angelo Vendrame at 2024-04-17T14:41:09+02:00 Bug 41128: Fix spaces in various config files. Some files contain more spaces than needed. After fixing them, we can update these files with ruamel.yaml if needed. - - - - - 30 changed files: - projects/glean/config - projects/goansicolor/config - projects/gobtcd/config - projects/gobtclog/config - projects/gobtcutil/config - projects/gobuildinfo/config - projects/goconfigurable/config - projects/godegoutils/config - projects/godexlogconfig/config - projects/godns/config - projects/goeasyconfig/config - projects/gogroupcache/config - projects/goisatty/config - projects/gokingpin/config - projects/gomadns/config - projects/goncbtcjson/config - projects/goncrpcclient/config - projects/gopflag/config - projects/gopretty/config - projects/goservice/config - projects/gosocks/config - projects/gosvcutils/config - projects/gosystemd/config - projects/gotemplate/config - projects/gotext/config - projects/gotoml/config - projects/gounits/config - projects/gowebsocket/config - projects/goxlog/config - projects/gspt/config The diff was not included because it is too large. View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser-build/-/commit/2… -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser-build/-/commit/2… You're receiving this email because of your account on gitlab.torproject.org.

1 0

[Git][tpo/applications/tor-browser-build][main] Bug 41128: Fix spaces in various config files.
by Pier Angelo Vendrame (＠pierov) 17 Apr '24

17 Apr '24

Pier Angelo Vendrame pushed to branch main at The Tor Project / Applications / tor-browser-build Commits: f8cb6710 by Pier Angelo Vendrame at 2024-04-17T11:26:53+02:00 Bug 41128: Fix spaces in various config files. Some files contain more spaces than needed. After fixing them, we can update these files with ruamel.yaml if needed. - - - - - 30 changed files: - projects/glean/config - projects/goansicolor/config - projects/gobtcd/config - projects/gobtclog/config - projects/gobtcutil/config - projects/gobuildinfo/config - projects/goconfigurable/config - projects/godegoutils/config - projects/godexlogconfig/config - projects/godns/config - projects/goeasyconfig/config - projects/gogroupcache/config - projects/goisatty/config - projects/gokingpin/config - projects/gomadns/config - projects/goncbtcjson/config - projects/goncrpcclient/config - projects/gopflag/config - projects/gopretty/config - projects/goservice/config - projects/gosocks/config - projects/gosvcutils/config - projects/gosystemd/config - projects/gotemplate/config - projects/gotext/config - projects/gotoml/config - projects/gounits/config - projects/gowebsocket/config - projects/goxlog/config - projects/gspt/config The diff was not included because it is too large. View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser-build/-/commit/f… -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser-build/-/commit/f… You're receiving this email because of your account on gitlab.torproject.org.

1 0

[Git][tpo/applications/firefox-android][firefox-android-115.2.1-13.5-1] fixup! Add Tor integration and UI
by Dan Ballard (＠dan) 17 Apr '24

17 Apr '24

Dan Ballard pushed to branch firefox-android-115.2.1-13.5-1 at The Tor Project / Applications / firefox-android Commits: 2095a229 by clairehurst at 2024-04-17T00:27:30+00:00 fixup! Add Tor integration and UI - - - - - 4 changed files: - + fenix/app/src/main/java/org/mozilla/fenix/tor/TorLogsComposeFragment.kt - − fenix/app/src/main/java/org/mozilla/fenix/tor/TorLogsFragment.kt - + fenix/app/src/main/java/org/mozilla/fenix/tor/TorLogsViewModel.kt - fenix/app/src/main/res/navigation/nav_graph.xml Changes: ===================================== fenix/app/src/main/java/org/mozilla/fenix/tor/TorLogsComposeFragment.kt ===================================== @@ -0,0 +1,82 @@ +/* This Source Code Form is subject to the terms of the Mozilla Public + * License, v. 2.0. If a copy of the MPL was not distributed with this + * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ + +package org.mozilla.fenix.tor + +import android.os.Bundle +import android.view.LayoutInflater +import android.view.View +import android.view.ViewGroup +import androidx.compose.foundation.layout.Column +import androidx.compose.foundation.layout.fillMaxSize +import androidx.compose.foundation.layout.fillMaxWidth +import androidx.compose.foundation.layout.padding +import androidx.compose.foundation.rememberScrollState +import androidx.compose.foundation.text.selection.DisableSelection +import androidx.compose.foundation.text.selection.SelectionContainer +import androidx.compose.foundation.verticalScroll +import androidx.compose.material.Text +import androidx.compose.runtime.Composable +import androidx.compose.runtime.Stable +import androidx.compose.ui.Modifier +import androidx.compose.ui.platform.ComposeView +import androidx.compose.ui.unit.dp +import androidx.fragment.app.Fragment +import androidx.fragment.app.viewModels +import mozilla.components.ui.colors.PhotonColors + +class TorLogsComposeFragment : Fragment() { + private val viewModel: TorLogsViewModel by viewModels() + + override fun onCreateView( + inflater: LayoutInflater, + container: ViewGroup?, + savedInstanceState: Bundle?, + ): View { + return ComposeView(requireContext()).apply { + setContent { + SelectionContainer { + Column( + // Column instead of LazyColumn so that you can select all the logs, and not just one "screen" at a time + // The logs won't be too big so loading them all instead of just whats visible shouldn't be a big deal + modifier = Modifier + .fillMaxSize() + .verticalScroll(state = rememberScrollState(), reverseScrolling = true), + ) { + for (log in viewModel.torLogs) { + LogRow(log = log) + } + } + } + } + } + } +} + +@Composable +@Stable +fun LogRow(log: TorLog, modifier: Modifier = Modifier) { + Column( + modifier + .fillMaxWidth() + .padding( + start = 16.dp, + end = 16.dp, + bottom = 16.dp, + ), + ) { + DisableSelection { + Text( + text = log.timestamp.toString(), + color = PhotonColors.LightGrey40, + modifier = modifier + .padding(bottom = 4.dp), + ) + } + Text( + text = log.text, + color = PhotonColors.LightGrey05, + ) + } +} ===================================== fenix/app/src/main/java/org/mozilla/fenix/tor/TorLogsFragment.kt deleted ===================================== @@ -1,81 +0,0 @@ -/* This Source Code Form is subject to the terms of the Mozilla Public - * License, v. 2.0. If a copy of the MPL was not distributed with this - * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ - -package org.mozilla.fenix.tor - -import android.os.Bundle -import android.text.method.ScrollingMovementMethod -import android.view.LayoutInflater -import android.view.View -import android.view.ViewGroup -import androidx.fragment.app.Fragment -import org.mozilla.fenix.R -import org.mozilla.fenix.components.Components -import org.mozilla.fenix.databinding.TorBootstrapLoggerBinding -import org.mozilla.fenix.ext.requireComponents -import org.mozilla.fenix.tor.view.TorBootstrapLoggerViewHolder - -class TorLogsFragment : Fragment(), TorLogs { - - private var entries = mutableListOf<String>() - internal var _binding: TorBootstrapLoggerBinding? = null - private val binding get() = _binding!! - - private var _components: Components? = null - private val components get() = _components!! - - override fun onCreateView( - inflater: LayoutInflater, - container: ViewGroup?, - savedInstanceState: Bundle?, - ): View { - _binding = TorBootstrapLoggerBinding.inflate(inflater) - _components = requireComponents - - components.torController.registerTorLogListener(this) - - val currentEntries = components.torController.logEntries.filter { it.second != null } - .filter { !(it.second!!.startsWith("Circuit") && it.first == "ON") } - // Keep synchronized with format in onTorStatusUpdate - .flatMap { listOf("(${it.first}) '${it.second}'") } - val entriesLen = currentEntries.size - val subListOffset = - if (entriesLen > TorBootstrapLoggerViewHolder.MAX_NEW_ENTRIES) TorBootstrapLoggerViewHolder.MAX_NEW_ENTRIES else entriesLen - entries = - currentEntries.subList((entriesLen - subListOffset), entriesLen) as MutableList<String> - val initLog = - "---------------" + getString(R.string.tor_initializing_log) + "---------------" - entries.add(0, initLog) - - with(binding.torBootstrapLogEntries) { - movementMethod = ScrollingMovementMethod() - text = formatLogEntries(entries) - } - - - return binding.root - } - - // TODO on destroy unregiuster - - private fun formatLogEntries(entries: List<String>) = entries.joinToString("\n") - - override fun onLog(type: String?, message: String?) { - if (message == null || type == null) return - if (type == "ON" && type.startsWith("Circuit")) return - - if (entries.size > TorBootstrapLoggerViewHolder.MAX_LINES) { - entries = entries.drop(1) as MutableList<String> - } - entries.add("($type) '$message'") - - binding.torBootstrapLogEntries.text = formatLogEntries(entries) - } - - override fun onStop() { - super.onStop() - components.torController.unregisterTorLogListener(this) - } - -} ===================================== fenix/app/src/main/java/org/mozilla/fenix/tor/TorLogsViewModel.kt ===================================== @@ -0,0 +1,88 @@ +/* This Source Code Form is subject to the terms of the Mozilla Public + * License, v. 2.0. If a copy of the MPL was not distributed with this + * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ + +package org.mozilla.fenix.tor + +import android.app.Application +import android.content.ClipData +import android.content.ClipboardManager +import android.content.Context +import android.os.Build +import android.widget.Toast +import androidx.compose.runtime.Stable +import androidx.lifecycle.AndroidViewModel +import org.mozilla.fenix.R +import org.mozilla.fenix.ext.components +import java.sql.Timestamp + +class TorLogsViewModel(application: Application) : AndroidViewModel(application), TorLogs { + private val torController = application.components.torController + private val clipboardManager = + application.getSystemService(Context.CLIPBOARD_SERVICE) as ClipboardManager + + val torLogs: MutableList<TorLog> = mutableListOf( + TorLog( + "---------------" + application.getString(R.string.tor_initializing_log) + "---------------", + ), + ) + + init { + setupClipboardListener() + torController.registerTorLogListener(this) + val currentEntries = torController.logEntries.filter { it.second != null } + .filter { !(it.second!!.startsWith("Circuit") && it.first == "ON") } + // Keep synchronized with format in onTorStatusUpdate + .flatMap { listOf(TorLog("[${it.first}] ${it.second}")) } + torLogs.addAll(currentEntries) + } + + override fun onLog(type: String?, message: String?) { + if (message == null || type == null) return + if (type == "ON" && type.startsWith("Circuit")) return + + torLogs.add(TorLog("[$type] $message")) + } + + override fun onCleared() { + super.onCleared() + torController.unregisterTorLogListener(this) + } + + private fun setupClipboardListener() { + clipboardManager.addPrimaryClipChangedListener { + // Only show a toast for Android 12 and lower. + // https://developer.android.com/develop/ui/views/touch-and-input/copy-paste#d… + if (Build.VERSION.SDK_INT <= Build.VERSION_CODES.S_V2) { + Toast.makeText( + getApplication<Application>().applicationContext, + getApplication<Application>().getString(R.string.toast_copy_link_to_clipboard), // "Copied to clipboard" already translated + Toast.LENGTH_SHORT, + ).show() + } + } + } + + fun copyAllLogsToClipboard() { // TODO add kebab menu in top right corner which includes option to "Copy all logs" + clipboardManager.setPrimaryClip( + ClipData.newPlainText( + "Copied Text", + getAllTorLogs(), + ), + ) + } + + private fun getAllTorLogs(): String { + var ret = "" + for (log in torLogs) { + ret += log.text + '\n' + } + return ret + } +} + +@Stable +data class TorLog( + val text: String, + val timestamp: Timestamp = Timestamp(System.currentTimeMillis()), +) ===================================== fenix/app/src/main/res/navigation/nav_graph.xml ===================================== @@ -976,8 +976,7 @@ <fragment android:id="@+id/torBridgeConfigFragment" android:name="org.mozilla.fenix.settings.TorBridgeConfigFragment" - android:label="@string/preferences_tor_network_settings_bridge_config" - tools:layout="@layout/fragment_tor_bridge_config" /> + android:label="@string/preferences_tor_network_settings_bridge_config" /> <fragment android:id="@+id/torBetaConnectionFeaturesFragment" android:name="org.mozilla.fenix.tor.TorBetaConnectionFeaturesFragment" @@ -985,9 +984,8 @@ tools:layout="@layout/tor_network_settings_beta_connection_features" /> <fragment android:id="@+id/torLogsFragment" - android:name="org.mozilla.fenix.tor.TorLogsFragment" - android:label="Tor Logs" - tools:layout="@layout/tor_bootstrap_logger" /> + android:name="org.mozilla.fenix.tor.TorLogsComposeFragment" + android:label="@string/preferences_tor_logs" /> <fragment android:id="@+id/trackingProtectionFragment" View it on GitLab: https://gitlab.torproject.org/tpo/applications/firefox-android/-/commit/209… -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/firefox-android/-/commit/209… You're receiving this email because of your account on gitlab.torproject.org.

1 0

[Git][tpo/applications/tor-browser][tor-browser-115.10.0esr-13.5-1] 4 commits: fixup! Bug 40458: Implement .tor.onion aliases
by richard (＠richard) 16 Apr '24

16 Apr '24

richard pushed to branch tor-browser-115.10.0esr-13.5-1 at The Tor Project / Applications / Tor Browser Commits: 5fd3baa6 by Henry Wilkes at 2024-04-16T21:57:16+00:00 fixup! Bug 40458: Implement .tor.onion aliases Bug 42206: Migrate ruleset strings to Fluent. - - - - - 7b790cf6 by Henry Wilkes at 2024-04-16T21:57:16+00:00 fixup! Tor Browser strings Bug 42206: Migrate ruleset strings to Fluent. - - - - - caed3ae7 by Henry Wilkes at 2024-04-16T21:57:16+00:00 fixup! Add TorStrings module for localization Bug 42206: Migrate ruleset strings to Fluent. - - - - - 8c14330e by Henry Wilkes at 2024-04-16T21:57:16+00:00 fixup! Tor Browser localization migration scripts. Bug 42206: Migrate ruleset strings to Fluent. - - - - - 7 changed files: - browser/components/rulesets/RulesetsParent.sys.mjs - browser/components/rulesets/content/aboutRulesets.html - browser/components/rulesets/content/aboutRulesets.js - browser/locales/en-US/browser/tor-browser.ftl - toolkit/modules/TorStrings.sys.mjs - − toolkit/torbutton/chrome/locale/en-US/rulesets.properties - + tools/torbrowser/l10n/migrations/bug-42206-rulesets.py Changes: ===================================== browser/components/rulesets/RulesetsParent.sys.mjs ===================================== @@ -1,6 +1,5 @@ // Copyright (c) 2022, The Tor Project, Inc. -import { TorStrings } from "resource://gre/modules/TorStrings.sys.mjs"; import { OnionAliasStore, OnionAliasStoreTopics, @@ -8,8 +7,8 @@ import { const kShowWarningPref = "torbrowser.rulesets.show_warning"; -// This class allows about:rulesets to get TorStrings and to load/save the -// preference for skipping the warning +// This class allows about:rulesets to load/save the preference for skipping the +// warning export class RulesetsParent extends JSWindowActorParent { constructor(...args) { super(...args); @@ -53,7 +52,6 @@ export class RulesetsParent extends JSWindowActorParent { return OnionAliasStore.getChannels(); case "rulesets:get-init-args": return { - TorStrings, showWarning: Services.prefs.getBoolPref(kShowWarningPref, true), }; case "rulesets:set-channel": ===================================== browser/components/rulesets/content/aboutRulesets.html ===================================== @@ -11,13 +11,19 @@ rel="stylesheet" href="chrome://browser/content/rulesets/aboutRulesets.css" /> + + <link rel="localization" href="branding/brand.ftl" /> + <link rel="localization" href="browser/tor-browser.ftl" /> </head> <body>  <div id="warning-wrapper"> <div id="warning"> - <h1 id="warning-title"></h1> - <p id="warning-description"></p> + <h1 id="warning-title" data-l10n-id="rulesets-warning-heading"></h1> + <p + id="warning-description" + data-l10n-id="rulesets-warning-description" + ></p> <p> <label> <input @@ -25,11 +31,18 @@ type="checkbox" checked="checked" /> - <span id="warning-enable-label"></span> + <span + id="warning-enable-label" + data-l10n-id="rulesets-warning-checkbox" + ></span> </label> </p> <div id="warning-buttonbar"> - <button id="warning-button" autofocus="autofocus"></button> + <button + id="warning-button" + autofocus="autofocus" + data-l10n-id="rulesets-warning-continue-button" + ></button> </div> </div> </div> @@ -37,11 +50,20 @@ <div id="main-content">  <aside> - <div id="ruleset-heading"></div> + <div + id="ruleset-heading" + data-l10n-id="rulesets-side-panel-heading" + ></div> <div id="ruleset-list-container"> <div id="ruleset-list-empty"> - <p id="ruleset-list-empty-title"></p> - <p id="ruleset-list-empty-description"></p> + <p + id="ruleset-list-empty-title" + data-l10n-id="rulesets-side-panel-no-rules" + ></p> + <p + id="ruleset-list-empty-description" + data-l10n-id="rulesets-side-panel-no-rules-description" + ></p> </div> <ul id="ruleset-list"> <li id="ruleset-template"> @@ -59,24 +81,40 @@ <section id="ruleset-details"> <div class="title"> <h1 id="ruleset-title"></h1> - <button id="ruleset-edit" class="ghost-button"></button> + <button + id="ruleset-edit" + class="ghost-button" + data-l10n-id="rulesets-details-edit-button" + ></button> </div> <dl> - <dt id="ruleset-jwk-label"></dt> + <dt id="ruleset-jwk-label" data-l10n-id="rulesets-details-jwk"></dt> <dd id="ruleset-jwk-value"></dd> - <dt id="ruleset-path-prefix-label"></dt> + <dt + id="ruleset-path-prefix-label" + data-l10n-id="rulesets-details-path" + ></dt> <dd> <a id="ruleset-path-prefix-value" target="_blank"></a> </dd> - <dt id="ruleset-scope-label"></dt> + <dt + id="ruleset-scope-label" + data-l10n-id="rulesets-details-scope" + ></dt> <dd id="ruleset-scope-value"></dd> </dl> <label id="ruleset-enable"> <input type="checkbox" id="ruleset-enable-checkbox" /> - <span id="ruleset-enable-label"></span> + <span + id="ruleset-enable-label" + data-l10n-id="rulesets-details-enable-checkbox" + ></span> </label> <div id="ruleset-buttonbar"> - <button id="ruleset-update-button"></button> + <button + id="ruleset-update-button" + data-l10n-id="rulesets-details-update-button" + ></button> </div> <hr /> <p id="ruleset-updated"></p> @@ -89,24 +127,52 @@ </div> <form id="edit-ruleset-form"> <label> - <div id="edit-jwk-label"></div> - <textarea id="edit-jwk-textarea" rows="10"></textarea> + <div id="edit-jwk-label" data-l10n-id="rulesets-details-jwk"></div> + <textarea + id="edit-jwk-textarea" + rows="10" + data-l10n-id="rulesets-details-jwk-input" + ></textarea> </label> <label> - <div id="edit-path-prefix-label"></div> - <input id="edit-path-prefix-input" type="text" /> + <div + id="edit-path-prefix-label" + data-l10n-id="rulesets-details-path" + ></div> + <input + id="edit-path-prefix-input" + type="text" + data-l10n-id="rulesets-details-path-input" + /> </label> <label> - <div id="edit-scope-label"></div> - <input id="edit-scope-input" type="text" /> + <div + id="edit-scope-label" + data-l10n-id="rulesets-details-scope" + ></div> + <input + id="edit-scope-input" + type="text" + data-l10n-id="rulesets-details-scope-input" + /> </label> <label id="edit-enable"> <input type="checkbox" id="edit-enable-checkbox" /> - <span id="edit-enable-label"></span> + <span + id="edit-enable-label" + data-l10n-id="rulesets-details-enable-checkbox" + ></span> </label> <div id="edit-buttonbar"> - <button id="edit-save" class="primary"></button> - <button id="edit-cancel"></button> + <button + id="edit-save" + class="primary" + data-l10n-id="rulesets-details-save-button" + ></button> + <button + id="edit-cancel" + data-l10n-id="rulesets-details-cancel-button" + ></button> </div> </form> </section> ===================================== browser/components/rulesets/content/aboutRulesets.js ===================================== @@ -2,8 +2,6 @@ /* globals RPMAddMessageListener, RPMSendQuery, RPMSendAsyncMessage */ -let TorStrings; - const Orders = Object.freeze({ Name: "name", NameDesc: "name-desc", @@ -19,55 +17,35 @@ const States = Object.freeze({ function setUpdateDate(ruleset, element) { if (!ruleset.enabled) { - element.textContent = TorStrings.rulesets.disabled; + document.l10n.setAttributes(element, "rulesets-update-rule-disabled"); return; } if (!ruleset.currentTimestamp) { - element.textContent = TorStrings.rulesets.neverUpdated; + document.l10n.setAttributes(element, "rulesets-update-never"); return; } - const formatter = new Intl.DateTimeFormat(navigator.languages, { - year: "numeric", - month: "long", - day: "numeric", + document.l10n.setAttributes(element, "rulesets-update-last", { + date: ruleset.currentTimestamp * 1000, }); - element.textContent = TorStrings.rulesets.lastUpdated.replace( - "%S", - formatter.format(new Date(ruleset.currentTimestamp * 1000)) - ); } class WarningState { - selectors = Object.freeze({ - wrapper: "#warning-wrapper", - title: "#warning-title", - description: "#warning-description", - enableCheckbox: "#warning-enable-checkbox", - enableLabel: "#warning-enable-label", - button: "#warning-button", - }); - - elements = Object.freeze({ - wrapper: document.querySelector(this.selectors.wrapper), - title: document.querySelector(this.selectors.title), - description: document.querySelector(this.selectors.description), - enableCheckbox: document.querySelector(this.selectors.enableCheckbox), - enableLabel: document.querySelector(this.selectors.enableLabel), - button: document.querySelector(this.selectors.button), - }); + elements = { + enableCheckbox: document.getElementById("warning-enable-checkbox"), + button: document.getElementById("warning-button"), + }; constructor() { - const elements = this.elements; - elements.title.textContent = TorStrings.rulesets.warningTitle; - elements.description.textContent = TorStrings.rulesets.warningDescription; - elements.enableLabel.textContent = TorStrings.rulesets.warningEnable; - elements.button.textContent = TorStrings.rulesets.warningButton; - elements.enableCheckbox.addEventListener( + this.elements.enableCheckbox.addEventListener( "change", this.onEnableChange.bind(this) ); - elements.button.addEventListener("click", this.onButtonClick.bind(this)); + + this.elements.button.addEventListener( + "click", + this.onButtonClick.bind(this) + ); } show() { @@ -89,50 +67,28 @@ class WarningState { } class DetailsState { - selectors = Object.freeze({ - title: "#ruleset-title", - edit: "#ruleset-edit", - jwkLabel: "#ruleset-jwk-label", - jwkValue: "#ruleset-jwk-value", - pathPrefixLabel: "#ruleset-path-prefix-label", - pathPrefixValue: "#ruleset-path-prefix-value", - scopeLabel: "#ruleset-scope-label", - scopeValue: "#ruleset-scope-value", - enableCheckbox: "#ruleset-enable-checkbox", - enableLabel: "#ruleset-enable-label", - updateButton: "#ruleset-update-button", - updated: "#ruleset-updated", - }); - - elements = Object.freeze({ - title: document.querySelector(this.selectors.title), - edit: document.querySelector(this.selectors.edit), - jwkLabel: document.querySelector(this.selectors.jwkLabel), - jwkValue: document.querySelector(this.selectors.jwkValue), - pathPrefixLabel: document.querySelector(this.selectors.pathPrefixLabel), - pathPrefixValue: document.querySelector(this.selectors.pathPrefixValue), - scopeLabel: document.querySelector(this.selectors.scopeLabel), - scopeValue: document.querySelector(this.selectors.scopeValue), - enableCheckbox: document.querySelector(this.selectors.enableCheckbox), - enableLabel: document.querySelector(this.selectors.enableLabel), - updateButton: document.querySelector(this.selectors.updateButton), - updated: document.querySelector(this.selectors.updated), - }); + elements = { + title: document.getElementById("ruleset-title"), + jwkValue: document.getElementById("ruleset-jwk-value"), + pathPrefixValue: document.getElementById("ruleset-path-prefix-value"), + scopeValue: document.getElementById("ruleset-scope-value"), + enableCheckbox: document.getElementById("ruleset-enable-checkbox"), + updateButton: document.getElementById("ruleset-update-button"), + updated: document.getElementById("ruleset-updated"), + }; constructor() { - const elements = this.elements; - elements.edit.textContent = TorStrings.rulesets.edit; - elements.edit.addEventListener("click", this.onEdit.bind(this)); - elements.jwkLabel.textContent = TorStrings.rulesets.jwk; - elements.pathPrefixLabel.textContent = TorStrings.rulesets.pathPrefix; - elements.scopeLabel.textContent = TorStrings.rulesets.scope; - elements.enableCheckbox.addEventListener( + document + .getElementById("ruleset-edit") + .addEventListener("click", this.onEdit.bind(this)); + this.elements.enableCheckbox.addEventListener( "change", this.onEnable.bind(this) ); - elements.enableLabel.textContent = TorStrings.rulesets.enable; - elements.updateButton.textContent = TorStrings.rulesets.checkUpdates; - elements.updateButton.addEventListener("click", this.onUpdate.bind(this)); + this.elements.updateButton.addEventListener( + "click", + this.onUpdate.bind(this) + ); } show(ruleset) { @@ -179,61 +135,22 @@ class DetailsState { } class EditState { - selectors = Object.freeze({ - form: "#edit-ruleset-form", - title: "#edit-title", - nameGroup: "#edit-name-group", - nameLabel: "#edit-name-label", - nameInput: "#edit-name-input", - jwkLabel: "#edit-jwk-label", - jwkTextarea: "#edit-jwk-textarea", - pathPrefixLabel: "#edit-path-prefix-label", - pathPrefixInput: "#edit-path-prefix-input", - scopeLabel: "#edit-scope-label", - scopeInput: "#edit-scope-input", - enableCheckbox: "#edit-enable-checkbox", - enableLabel: "#edit-enable-label", - save: "#edit-save", - cancel: "#edit-cancel", - }); - - elements = Object.freeze({ - form: document.querySelector(this.selectors.form), - title: document.querySelector(this.selectors.title), - jwkLabel: document.querySelector(this.selectors.jwkLabel), - jwkTextarea: document.querySelector(this.selectors.jwkTextarea), - pathPrefixLabel: document.querySelector(this.selectors.pathPrefixLabel), - pathPrefixInput: document.querySelector(this.selectors.pathPrefixInput), - scopeLabel: document.querySelector(this.selectors.scopeLabel), - scopeInput: document.querySelector(this.selectors.scopeInput), - enableCheckbox: document.querySelector(this.selectors.enableCheckbox), - enableLabel: document.querySelector(this.selectors.enableLabel), - save: document.querySelector(this.selectors.save), - cancel: document.querySelector(this.selectors.cancel), - }); + elements = { + form: document.getElementById("edit-ruleset-form"), + title: document.getElementById("edit-title"), + jwkTextarea: document.getElementById("edit-jwk-textarea"), + pathPrefixInput: document.getElementById("edit-path-prefix-input"), + scopeInput: document.getElementById("edit-scope-input"), + enableCheckbox: document.getElementById("edit-enable-checkbox"), + }; constructor() { - const elements = this.elements; - elements.jwkLabel.textContent = TorStrings.rulesets.jwk; - elements.jwkTextarea.setAttribute( - "placeholder", - TorStrings.rulesets.jwkPlaceholder - ); - elements.pathPrefixLabel.textContent = TorStrings.rulesets.pathPrefix; - elements.pathPrefixInput.setAttribute( - "placeholder", - TorStrings.rulesets.pathPrefixPlaceholder - ); - elements.scopeLabel.textContent = TorStrings.rulesets.scope; - elements.scopeInput.setAttribute( - "placeholder", - TorStrings.rulesets.scopePlaceholder - ); - elements.enableLabel.textContent = TorStrings.rulesets.enable; - elements.save.textContent = TorStrings.rulesets.save; - elements.save.addEventListener("click", this.onSave.bind(this)); - elements.cancel.textContent = TorStrings.rulesets.cancel; - elements.cancel.addEventListener("click", this.onCancel.bind(this)); + document + .getElementById("edit-save") + .addEventListener("click", this.onSave.bind(this)); + document + .getElementById("edit-cancel") + .addEventListener("click", this.onCancel.bind(this)); } show(ruleset) { @@ -276,7 +193,9 @@ class EditState { elements.jwkTextarea.setCustomValidity(""); } catch (err) { console.error("Invalid JSON or invalid JWK", err); - elements.jwkTextarea.setCustomValidity(TorStrings.rulesets.jwkInvalid); + elements.jwkTextarea.setCustomValidity( + await document.l10n.formatValue("rulesets-details-jwk-input-invalid") + ); valid = false; } @@ -285,7 +204,7 @@ class EditState { const url = new URL(pathPrefix); if (url.protocol !== "http:" && url.protocol !== "https:") { elements.pathPrefixInput.setCustomValidity( - TorStrings.rulesets.pathPrefixInvalid + await document.l10n.formatValue("rulesets-details-path-input-invalid") ); valid = false; } else { @@ -294,7 +213,7 @@ class EditState { } catch (err) { console.error("The path prefix is not a valid URL", err); elements.pathPrefixInput.setCustomValidity( - TorStrings.rulesets.pathPrefixInvalid + await document.l10n.formatValue("rulesets-details-path-input-invalid") ); valid = false; } @@ -304,7 +223,9 @@ class EditState { scope = new RegExp(elements.scopeInput.value.trim()); elements.scopeInput.setCustomValidity(""); } catch (err) { - elements.scopeInput.setCustomValidity(TorStrings.rulesets.scopeInvalid); + elements.scopeInput.setCustomValidity( + await document.l10n.formatValue("rulesets-details-scope-input-invalid") + ); valid = false; } @@ -342,39 +263,17 @@ class NoRulesetsState { } class RulesetList { - selectors = Object.freeze({ - heading: "#ruleset-heading", - list: "#ruleset-list", - emptyContainer: "#ruleset-list-empty", - emptyTitle: "#ruleset-list-empty-title", - emptyDescription: "#ruleset-list-empty-description", - itemTemplate: "#ruleset-template", - itemName: ".name", - itemDescr: ".description", - }); - - elements = Object.freeze({ - heading: document.querySelector(this.selectors.heading), - list: document.querySelector(this.selectors.list), - emptyContainer: document.querySelector(this.selectors.emptyContainer), - emptyTitle: document.querySelector(this.selectors.emptyTitle), - emptyDescription: document.querySelector(this.selectors.emptyDescription), - itemTemplate: document.querySelector(this.selectors.itemTemplate), - }); + elements = { + list: document.getElementById("ruleset-list"), + emptyContainer: document.getElementById("ruleset-list-empty"), + itemTemplate: document.getElementById("ruleset-template"), + }; nameAttribute = "data-name"; rulesets = []; constructor() { - const elements = this.elements; - - // Header - elements.heading.textContent = TorStrings.rulesets.rulesets; - // Empty - elements.emptyTitle.textContent = TorStrings.rulesets.noRulesets; - elements.emptyDescription.textContent = TorStrings.rulesets.noRulesetsDescr; - RPMAddMessageListener( "rulesets:channels-change", this.onRulesetsChanged.bind(this) @@ -438,14 +337,10 @@ class RulesetList { const item = this.elements.itemTemplate.cloneNode(true); item.removeAttribute("id"); item.classList.add("item"); - item.querySelector(this.selectors.itemName).textContent = ruleset.name; - const descr = item.querySelector(this.selectors.itemDescr); - if (ruleset.enabled) { - setUpdateDate(ruleset, descr); - } else { - descr.textContent = TorStrings.rulesets.disabled; - item.classList.add("disabled"); - } + item.querySelector(".name").textContent = ruleset.name; + const descr = item.querySelector(".description"); + setUpdateDate(ruleset, descr); + item.classList.toggle("disabled", !ruleset.enabled); item.setAttribute(this.nameAttribute, ruleset.name); item.addEventListener("click", () => { this.onRulesetClick(ruleset); @@ -478,7 +373,6 @@ class AboutRulesets { async init() { const args = await RPMSendQuery("rulesets:get-init-args"); - TorStrings = args.TorStrings; const showWarning = args.showWarning; this.list = new RulesetList(); ===================================== browser/locales/en-US/browser/tor-browser.ftl ===================================== @@ -557,3 +557,53 @@ downloads-tor-warning-title = Be careful opening downloads downloads-tor-warning-description = Some files may connect to the internet when opened without using Tor. To be safe, open the files while offline or use a portable operating system like <a data-l10n-name="tails-link">Tails</a>. # Button to dismiss the warning forever. downloads-tor-warning-dismiss-button = Got it + +## Initial warning page in about:rulesets. In Tor Browser, each ruleset is a set of rules for converting a ".tor.onion" address to a normal ".onion" address (used by SecureDrop). The feature is taken from the discontinued "HTTPS Everywhere". + +rulesets-warning-heading = Proceed with Caution +rulesets-warning-description = Adding or modifying rulesets can cause attackers to hijack your browser. Proceed only if you know what you are doing. +rulesets-warning-checkbox = Warn me when I attempt to access these preferences +rulesets-warning-continue-button = Accept the Risk and Continue + +## Side panel in about:rulesets. In Tor Browser, each ruleset is a set of rules for converting a ".tor.onion" address to a normal ".onion" address (used by SecureDrop). The feature is taken from the discontinued "HTTPS Everywhere". + +rulesets-side-panel-heading = Rulesets +rulesets-side-panel-no-rules = No rulesets found +# -brand-short-name refers to 'Tor Browser', localized. +rulesets-side-panel-no-rules-description = When you save a ruleset in { -brand-short-name }, it will show up here. + +## Ruleset update date in about:rulesets. + +# $date (Date) - The update date. The DATETIME function will format the $date according to the locale, using a "long" style. E.g. "January 1, 2000" for English (US), "١ يناير ٢٠٠٠" for Arabic, "2000년 1월 1일" in Korean, and "1 января 2000 г." in Russian. +rulesets-update-last = Last updated { DATETIME($date, dateStyle: "long") } +rulesets-update-never = Never updated, or last update failed +# Shown when the ruleset is disabled. +rulesets-update-rule-disabled = Disabled + +## Ruleset details in about:rulesets. In Tor Browser, each ruleset is a set of rules for converting a ".tor.onion" address to a normal ".onion" address (used by SecureDrop). The feature is taken from the discontinued "HTTPS Everywhere". + +rulesets-details-edit-button = Edit +rulesets-details-enable-checkbox = Enable this ruleset +rulesets-details-update-button = Check for Updates +rulesets-details-save-button = Save +rulesets-details-cancel-button = Cancel +# "JWK" refers to "JSON Web Key" and likely should not be translated. +rulesets-details-jwk = JWK +# "JWK" refers to "JSON Web Key" and likely should not be translated. +rulesets-details-jwk-input = + .placeholder = The key used to sign this ruleset in the JWK (JSON Web Key) format +# "JWK" refers to "JSON Web Key" and likely should not be translated. +rulesets-details-jwk-input-invalid = The JWK could not be parsed, or it is not a valid key +# "Path" refers to the URL domain this rule applies to. +rulesets-details-path = Path Prefix +rulesets-details-path-input = + .placeholder = URL prefix that contains the files needed by the ruleset +# "HTTP(S)" refers to "HTTP or HTTPS". +rulesets-details-path-input-invalid = The path prefix is not a valid HTTP(S) URL +# "Scope" refers to the breadth of URLs this rule applies to (as a regular expression). +rulesets-details-scope = Scope +# "Regular expression" refers to the computing term for a special pattern used for matching: https://en.wikipedia.org/wiki/Regular_expression. +rulesets-details-scope-input = + .placeholder = Regular expression for the scope of the rules +# "Regular expression" refers to the computing term for a special pattern used for matching: https://en.wikipedia.org/wiki/Regular_expression. +rulesets-details-scope-input-invalid = The scope could not be parsed as a regular expression ===================================== toolkit/modules/TorStrings.sys.mjs ===================================== @@ -429,54 +429,6 @@ const Loader = { learnMoreURLNotification: `https://tb-manual.torproject.org/${getLocale()}/onion-services/`, }; } /* OnionLocation */, - - /* - Rulesets - */ - rulesets() { - const strings = { - // Initial warning - warningTitle: "Proceed with Caution", - warningDescription: - "Adding or modifying rulesets can cause attackers to hijack your browser. Proceed only if you know what you are doing.", - warningEnable: "Warn me when I attempt to access these preferences", - warningButton: "Accept the Risk and Continue", - // Ruleset list - rulesets: "Rulesets", - noRulesets: "No rulesets found", - noRulesetsDescr: - "When you save a ruleset in Tor Browser, it will show up here.", - lastUpdated: "Last updated %S", - neverUpdated: "Never updated, or last update failed", - enabled: "Enabled", - disabled: "Disabled", - // Ruleset details - edit: "Edit", - name: "Name", - jwk: "JWK", - pathPrefix: "Path Prefix", - scope: "Scope", - enable: "Enable this ruleset", - checkUpdates: "Check for Updates", - // Add ruleset - jwkPlaceholder: - "The key used to sign this ruleset in the JWK (JSON Web Key) format", - jwkInvalid: "The JWK could not be parsed, or it is not a valid key", - pathPrefixPlaceholder: - "URL prefix that contains the files needed by the ruleset", - pathPrefixInvalid: "The path prefix is not a valid HTTP(S) URL", - scopePlaceholder: "Regular expression for the scope of the rules", - scopeInvalid: "The scope could not be parsed as a regular expression", - save: "Save", - cancel: "Cancel", - }; - - const tsb = new TorPropertyStringBundle( - ["chrome://torbutton/locale/rulesets.properties"], - "rulesets." - ); - return tsb.getStrings(strings); - } /* Rulesets */, }; export const TorStrings = { @@ -507,11 +459,4 @@ export const TorStrings = { } return this._onionLocation; }, - - get rulesets() { - if (!this._rulesets) { - this._rulesets = Loader.rulesets(); - } - return this._rulesets; - }, }; ===================================== toolkit/torbutton/chrome/locale/en-US/rulesets.properties deleted ===================================== @@ -1,35 +0,0 @@ -# Copyright (c) 2022, The Tor Project, Inc. -# This Source Code Form is subject to the terms of the Mozilla Public -# License, v. 2.0. If a copy of the MPL was not distributed with this -# file, You can obtain one at http://mozilla.org/MPL/2.0/. - -# about:rulesets strings. -rulesets.warningTitle=Proceed with Caution -rulesets.warningDescription=Adding or modifying rulesets can cause attackers to hijack your browser. Proceed only if you know what you are doing. -rulesets.warningEnable=Warn me when I attempt to access these preferences -rulesets.warningButton=Accept the Risk and Continue -# Ruleset list -rulesets.rulesets=Rulesets -rulesets.noRulesets=No rulesets found -rulesets.noRulesetsDescr=When you save a ruleset in Tor Browser, it will show up here. -# LOCALIZATION NOTE: %S will be replaced by the update date (automatically formatted by Firefox's l10n component) -rulesets.lastUpdated=Last updated %S -rulesets.neverUpdated=Never updated, or last update failed -rulesets.enabled=Enabled -rulesets.disabled=Disabled -# Ruleset details/edit ruleset -rulesets.edit=Edit -rulesets.name=Name -rulesets.jwk=JWK -rulesets.pathPrefix=Path Prefix -rulesets.scope=Scope -rulesets.enable=Enable this ruleset -rulesets.checkUpdates=Check for Updates -rulesets.jwkPlaceholder=The key used to sign this ruleset in the JWK (JSON Web Key) format -rulesets.jwkInvalid=The JWK could not be parsed, or it is not a valid key -rulesets.pathPrefixPlaceholder=URL prefix that contains the files needed by the ruleset -rulesets.pathPrefixInvalid=The path prefix is not a valid HTTP(S) URL -rulesets.scopePlaceholder=Regular expression for the scope of the rules -rulesets.scopeInvalid=The scope could not be parsed as a regular expression -rulesets.save=Save -rulesets.cancel=Cancel ===================================== tools/torbrowser/l10n/migrations/bug-42206-rulesets.py ===================================== @@ -0,0 +1,70 @@ +import fluent.syntax.ast as FTL +from fluent.migrate.helpers import transforms_from +from fluent.migrate.transforms import REPLACE + + +def migrate(ctx): + legacy_path = "rulesets.properties" + + ctx.add_transforms( + "tor-browser.ftl", + "tor-browser.ftl", + transforms_from( + """ +rulesets-warning-heading = { COPY(path, "rulesets.warningTitle") } +rulesets-warning-description = { COPY(path, "rulesets.warningDescription") } +rulesets-warning-checkbox = { COPY(path, "rulesets.warningEnable") } +rulesets-warning-continue-button = { COPY(path, "rulesets.warningButton") } + +rulesets-side-panel-heading = { COPY(path, "rulesets.rulesets") } +rulesets-side-panel-no-rules = { COPY(path, "rulesets.noRulesets") } + +rulesets-update-never = { COPY(path, "rulesets.neverUpdated") } +rulesets-update-rule-disabled = { COPY(path, "rulesets.disabled") } + +rulesets-details-edit-button = { COPY(path, "rulesets.edit") } +rulesets-details-enable-checkbox = { COPY(path, "rulesets.enable") } +rulesets-details-update-button = { COPY(path, "rulesets.checkUpdates") } +rulesets-details-save-button = { COPY(path, "rulesets.save") } +rulesets-details-cancel-button = { COPY(path, "rulesets.cancel") } +rulesets-details-jwk-input = + .placeholder = { COPY(path, "rulesets.jwkPlaceholder") } +rulesets-details-jwk-input-invalid = { COPY(path, "rulesets.jwkInvalid") } +rulesets-details-path = { COPY(path, "rulesets.pathPrefix") } +rulesets-details-path-input = + .placeholder = { COPY(path, "rulesets.pathPrefixPlaceholder") } +rulesets-details-path-input-invalid = { COPY(path, "rulesets.pathPrefixInvalid") } +rulesets-details-scope = { COPY(path, "rulesets.scope") } +rulesets-details-scope-input = + .placeholder = { COPY(path, "rulesets.scopePlaceholder") } +rulesets-details-scope-input-invalid = { COPY(path, "rulesets.scopeInvalid") } +""", + path=legacy_path, + ) + + [ + # Replace "%1$S" with "{ DATETIME($date, dateStyle: "long") }" + FTL.Message( + FTL.Identifier("rulesets-update-last"), + value=REPLACE( + legacy_path, + "rulesets.lastUpdated", + { + "%1$S": FTL.FunctionReference( + FTL.Identifier("DATETIME"), + arguments=FTL.CallArguments( + positional=[ + FTL.VariableReference(FTL.Identifier("date")) + ], + named=[ + FTL.NamedArgument( + FTL.Identifier("dateStyle"), + value=FTL.StringLiteral("long"), + ) + ], + ), + ) + }, + ), + ), + ], + ) View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/compare/08ed48… -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/compare/08ed48… You're receiving this email because of your account on gitlab.torproject.org.

1 0

[Git][tpo/applications/firefox-android][firefox-android-115.2.1-13.5-1] add to fixup! Enable the connect assist experiments on alpha
by Dan Ballard (＠dan) 16 Apr '24

16 Apr '24

Dan Ballard pushed to branch firefox-android-115.2.1-13.5-1 at The Tor Project / Applications / firefox-android Commits: 3768f68b by clairehurst at 2024-04-16T14:32:55-06:00 add to fixup! Enable the connect assist experiments on alpha - - - - - 3 changed files: - android-components/components/concept/engine/src/main/java/mozilla/components/concept/engine/Settings.kt - fenix/app/src/main/java/org/mozilla/fenix/utils/Settings.kt - fenix/app/src/main/res/layout/tor_network_settings_beta_connection_features.xml Changes: ===================================== android-components/components/concept/engine/src/main/java/mozilla/components/concept/engine/Settings.kt ===================================== @@ -258,7 +258,7 @@ data class DefaultSettings( override var cookieBannerHandlingDetectOnlyMode: Boolean = false, override var torSecurityLevel: Int = 4, override var spoofEnglish: Boolean = false, - override var useNewBootstrap: Boolean = false, + override var useNewBootstrap: Boolean = true, ) : Settings() class UnsupportedSetting<T> { ===================================== fenix/app/src/main/java/org/mozilla/fenix/utils/Settings.kt ===================================== @@ -1857,16 +1857,16 @@ class Settings(private val appContext: Context) : PreferencesHolder { var useNewBootstrap by booleanPreference( appContext.getPreferenceKey(R.string.pref_key_use_new_bootstrap), - default = false, + default = true, ) var useNewBootstrapNativeUi by booleanPreference( appContext.getPreferenceKey(R.string.pref_key_use_new_bootstrap_with_android_native), - default = false, + default = true, ) var useNewBootstrapHtmlUi by booleanPreference( appContext.getPreferenceKey(R.string.pref_key_use_new_bootstrap_with_html), - default = true + default = false ) } ===================================== fenix/app/src/main/res/layout/tor_network_settings_beta_connection_features.xml ===================================== @@ -60,7 +60,7 @@ app:layout_constraintTop_toBottomOf="@id/enable_beta_connection_features_summary"> <org.mozilla.fenix.settings.PreferenceBackedRadioButton - android:id="@+id/use_new_bootstrap_with_html_ui_radio_button" + android:id="@+id/use_new_bootstrap_with_native_ui_radio_button" android:layout_width="match_parent" android:layout_height="wrap_content" android:background="?android:attr/selectableItemBackground" @@ -70,15 +70,15 @@ android:paddingTop="@dimen/radio_button_preference_vertical" android:paddingEnd="@dimen/radio_button_preference_horizontal" android:paddingBottom="@dimen/radio_button_preference_vertical" - android:text="HTML UI" + android:text="Native Android UI" android:textAppearance="?android:attr/textAppearanceListItem" android:textSize="16sp" app:drawableStartCompat="?android:attr/listChoiceIndicatorSingle" - app:preferenceKey="@string/pref_key_use_new_bootstrap_with_html" + app:preferenceKey="@string/pref_key_use_new_bootstrap_with_android_native" app:preferenceKeyDefaultValue="true" /> <org.mozilla.fenix.settings.PreferenceBackedRadioButton - android:id="@+id/use_new_bootstrap_with_native_ui_radio_button" + android:id="@+id/use_new_bootstrap_with_html_ui_radio_button" android:layout_width="match_parent" android:layout_height="48dp" android:background="?android:attr/selectableItemBackground" @@ -88,11 +88,11 @@ android:paddingTop="@dimen/radio_button_preference_vertical" android:paddingEnd="@dimen/radio_button_preference_horizontal" android:paddingBottom="@dimen/radio_button_preference_vertical" - android:text="Native Android UI" + android:text="HTML UI" android:textAppearance="?android:attr/textAppearanceListItem" android:textSize="16sp" app:drawableStartCompat="?android:attr/listChoiceIndicatorSingle" - app:preferenceKey="@string/pref_key_use_new_bootstrap_with_android_native" + app:preferenceKey="@string/pref_key_use_new_bootstrap_with_html" app:preferenceKeyDefaultValue="false" /> </RadioGroup> View it on GitLab: https://gitlab.torproject.org/tpo/applications/firefox-android/-/commit/376… -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/firefox-android/-/commit/376… You're receiving this email because of your account on gitlab.torproject.org.

1 0

[Git][tpo/applications/mullvad-browser][mullvad-browser-115.10.0esr-13.5-1] fixup! Bug 41631: Prevent weird initial window dimensions caused by subpixel computations
by ma1 (＠ma1) 16 Apr '24

16 Apr '24

ma1 pushed to branch mullvad-browser-115.10.0esr-13.5-1 at The Tor Project / Applications / Mullvad Browser Commits: a17ec8e0 by hackademix at 2024-04-16T22:18:31+02:00 fixup! Bug 41631: Prevent weird initial window dimensions caused by subpixel computations Bug 42520: Correctly record new initial window size after auto-shrinking - - - - - 1 changed file: - toolkit/components/resistfingerprinting/RFPHelper.sys.mjs Changes: ===================================== toolkit/components/resistfingerprinting/RFPHelper.sys.mjs ===================================== @@ -633,9 +633,16 @@ class _RFPHelper { lazy.logConsole.error(e); } } - if (needToShrink) { - win.shrinkToLetterbox(); - this._recordWindowSize(win); + if (needToShrink && win.shrinkToLetterbox()) { + win.addEventListener( + "resize", + () => { + // We need to record the "new" initial size in this listener + // because resized dimensions are not immediately available. + RFPHelper._recordWindowSize(win); + }, + { once: true } + ); } }); }, @@ -741,23 +748,30 @@ class _RFPHelper { } _recordWindowSize(aWindow) { - aWindow._rfpOriginalSize = { - width: aWindow.outerWidth, - height: aWindow.outerHeight, - containerHeight: aWindow.gBrowser.getBrowserContainer()?.clientHeight, - }; - log("Recording original window size", aWindow._rfpOriginalSize); + aWindow.promiseDocumentFlushed(() => { + aWindow._rfpOriginalSize = { + width: aWindow.outerWidth, + height: aWindow.outerHeight, + containerHeight: aWindow.gBrowser.getBrowserContainer()?.clientHeight, + }; + log("Recording original window size", aWindow._rfpOriginalSize); + }); } // We will attach this method to each browser window. When called // it will instantly resize the window to exactly fit the selected // (possibly letterboxed) browser. + // Returns true if a window resize will occur, false otherwise. shrinkToLetterbox() { let { selectedBrowser } = this.gBrowser; let stack = selectedBrowser.closest(".browserStack"); const outer = stack.getBoundingClientRect(); const inner = selectedBrowser.getBoundingClientRect(); - this.resizeBy(inner.width - outer.width, inner.height - outer.height); + if (inner.width !== outer.witdh || inner.height !== outer.height) { + this.resizeBy(inner.width - outer.width, inner.height - outer.height); + return true; + } + return false; } _onWindowDoubleClick(e) { View it on GitLab: https://gitlab.torproject.org/tpo/applications/mullvad-browser/-/commit/a17… -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/mullvad-browser/-/commit/a17… You're receiving this email because of your account on gitlab.torproject.org.

1 0

[Git][tpo/applications/tor-browser][base-browser-115.10.0esr-13.5-1] fixup! Bug 41631: Prevent weird initial window dimensions caused by subpixel computations
by ma1 (＠ma1) 16 Apr '24

16 Apr '24

ma1 pushed to branch base-browser-115.10.0esr-13.5-1 at The Tor Project / Applications / Tor Browser Commits: 97575530 by hackademix at 2024-04-16T22:17:42+02:00 fixup! Bug 41631: Prevent weird initial window dimensions caused by subpixel computations Bug 42520: Correctly record new initial window size after auto-shrinking - - - - - 1 changed file: - toolkit/components/resistfingerprinting/RFPHelper.sys.mjs Changes: ===================================== toolkit/components/resistfingerprinting/RFPHelper.sys.mjs ===================================== @@ -633,9 +633,16 @@ class _RFPHelper { lazy.logConsole.error(e); } } - if (needToShrink) { - win.shrinkToLetterbox(); - this._recordWindowSize(win); + if (needToShrink && win.shrinkToLetterbox()) { + win.addEventListener( + "resize", + () => { + // We need to record the "new" initial size in this listener + // because resized dimensions are not immediately available. + RFPHelper._recordWindowSize(win); + }, + { once: true } + ); } }); }, @@ -741,23 +748,30 @@ class _RFPHelper { } _recordWindowSize(aWindow) { - aWindow._rfpOriginalSize = { - width: aWindow.outerWidth, - height: aWindow.outerHeight, - containerHeight: aWindow.gBrowser.getBrowserContainer()?.clientHeight, - }; - log("Recording original window size", aWindow._rfpOriginalSize); + aWindow.promiseDocumentFlushed(() => { + aWindow._rfpOriginalSize = { + width: aWindow.outerWidth, + height: aWindow.outerHeight, + containerHeight: aWindow.gBrowser.getBrowserContainer()?.clientHeight, + }; + log("Recording original window size", aWindow._rfpOriginalSize); + }); } // We will attach this method to each browser window. When called // it will instantly resize the window to exactly fit the selected // (possibly letterboxed) browser. + // Returns true if a window resize will occur, false otherwise. shrinkToLetterbox() { let { selectedBrowser } = this.gBrowser; let stack = selectedBrowser.closest(".browserStack"); const outer = stack.getBoundingClientRect(); const inner = selectedBrowser.getBoundingClientRect(); - this.resizeBy(inner.width - outer.width, inner.height - outer.height); + if (inner.width !== outer.witdh || inner.height !== outer.height) { + this.resizeBy(inner.width - outer.width, inner.height - outer.height); + return true; + } + return false; } _onWindowDoubleClick(e) { View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/commit/9757553… -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/commit/9757553… You're receiving this email because of your account on gitlab.torproject.org.

1 0

[Git][tpo/applications/tor-browser][tor-browser-115.10.0esr-13.5-1] fixup! Add TorStrings module for localization
by Pier Angelo Vendrame (＠pierov) 16 Apr '24

16 Apr '24

Pier Angelo Vendrame pushed to branch tor-browser-115.10.0esr-13.5-1 at The Tor Project / Applications / Tor Browser Commits: 08ed48fa by Henry Wilkes at 2024-04-16T16:31:30+01:00 fixup! Add TorStrings module for localization Bug 42521: Drop unused onboarding strings. - - - - - 2 changed files: - − toolkit/torbutton/chrome/locale/en-US/browserOnboarding.properties - − toolkit/torbutton/chrome/locale/en-US/onboarding.properties Changes: ===================================== toolkit/torbutton/chrome/locale/en-US/browserOnboarding.properties deleted ===================================== @@ -1,73 +0,0 @@ -# Copyright (c) 2019, The Tor Project, Inc. -# This Source Code Form is subject to the terms of the Mozilla Public -# License, v. 2.0. If a copy of the MPL was not distributed with this -# file, You can obtain one at http://mozilla.org/MPL/2.0/. - -onboarding.tour-tor-welcome=Welcome -onboarding.tour-tor-welcome.title=You’re ready. -onboarding.tour-tor-welcome.description=Tor Browser offers the highest standard of privacy and security while browsing the web. You’re now protected against tracking, surveillance, and censorship. This quick onboarding will show you how. -onboarding.tour-tor-welcome.next-button=Go to Privacy - -onboarding.tour-tor-privacy=Privacy -onboarding.tour-tor-privacy.title=Snub trackers and snoopers. -onboarding.tour-tor-privacy.description=Tor Browser isolates cookies and deletes your browser history after your session. These modifications ensure your privacy and security are protected in the browser. Click ‘Tor Network’ to learn how we protect you on the network level. -onboarding.tour-tor-privacy.button=Go to Tor Network - -onboarding.tour-tor-network=Tor Network -onboarding.tour-tor-network.title=Travel a decentralized network. -onboarding.tour-tor-network.description=Tor Browser connects you to the Tor network run by thousands of volunteers around the world. Unlike a VPN, there’s no single point of failure or centralized entity you need to trust in order to enjoy the internet privately. -onboarding.tour-tor-network.description-para2=NEW: Tor Network Settings, including the ability to request bridges where Tor is blocked, can now be found in Preferences. -onboarding.tour-tor-network.action-button=Adjust Your Tor Network Settings -onboarding.tour-tor-network.button=Go to Circuit Display - -onboarding.tour-tor-circuit-display=Circuit Display -onboarding.tour-tor-circuit-display.title=See your path. -onboarding.tour-tor-circuit-display.description=For each domain you visit, your traffic is relayed and encrypted in a circuit across three Tor relays around the world. No website knows where you are connecting from. You can request a new circuit by clicking ‘New Circuit for this Site’ on our Circuit Display. -onboarding.tour-tor-circuit-display.button=See My Path -onboarding.tour-tor-circuit-display.next-button=Go to Security - -onboarding.tour-tor-security=Security -onboarding.tour-tor-security.title=Choose your experience. -onboarding.tour-tor-security.description=We also provide you with additional settings for bumping up your browser security. Our Security Settings allow you to block elements that could be used to attack your computer. Click below to see what the different options do. -onboarding.tour-tor-security.description-suffix=Note: By default, NoScript and HTTPS Everywhere are not included on the toolbar, but you can customize your toolbar to add them. -onboarding.tour-tor-security-level.button=See Your Security Level -onboarding.tour-tor-security-level.next-button=Go to Experience Tips - -onboarding.tour-tor-expect-differences=Experience Tips -onboarding.tour-tor-expect-differences.title=Expect some differences. -onboarding.tour-tor-expect-differences.description=With all the security and privacy features provided by Tor, your experience while browsing the internet may be a little different. Things may be a bit slower, and depending on your security level, some elements may not work or load. You may also be asked to prove you are a human and not a robot. -onboarding.tour-tor-expect-differences.button=See FAQs -onboarding.tour-tor-expect-differences.next-button=Go to Onion Services - -onboarding.tour-tor-onion-services=Onion Services -onboarding.tour-tor-onion-services.title=Be extra protected. -onboarding.tour-tor-onion-services.description=Onion services are sites that end with a .onion that provide extra protections to publishers and visitors, including added safeguards against censorship. Onion services allow anyone to provide content and services anonymously. Click below to visit the DuckDuckGo onion site. -onboarding.tour-tor-onion-services.button=Visit an Onion -onboarding.tour-tor-onion-services.next-button=Done - -onboarding.overlay-icon-tooltip-updated2=See what's new\nin %S -onboarding.tour-tor-update.prefix-new=New -onboarding.tour-tor-update.prefix-updated=Updated - -onboarding.tour-tor-toolbar=Toolbar -onboarding.tour-tor-toolbar-update-9.0.title=Goodbye Onion Button. -onboarding.tour-tor-toolbar-update-9.0.description=We want your experience using Tor to be fully integrated within Tor Browser. -onboarding.tour-tor-toolbar-update-9.0.description-para2=That's why now, rather than using the onion button, you can see your Tor Circuit via the [i] in the URL bar and request a New Identity using the toolbar button or the [≡] menu. -onboarding.tour-tor-toolbar-update-9.0.button=How to Request a New Identity -onboarding.tour-tor-toolbar-update-9.0.next-button=Go to Tor Network - -# Circuit Display onboarding. -onboarding.tor-circuit-display.next=Next -onboarding.tor-circuit-display.done=Done -onboarding.tor-circuit-display.one-of-three=1 of 3 -onboarding.tor-circuit-display.two-of-three=2 of 3 -onboarding.tor-circuit-display.three-of-three=3 of 3 - -onboarding.tor-circuit-display.intro.title=How do circuits work? -onboarding.tor-circuit-display.intro.msg=Circuits are made up of randomly assigned relays, which are computers around the world configured to forward Tor traffic. Circuits allow you to browse privately and to connect to onion services. - -onboarding.tor-circuit-display.diagram.title=Circuit Display -onboarding.tor-circuit-display.diagram.msg=This diagram shows the relays that make up the circuit for this website. To prevent linking of activity across different sites, each website gets a different circuit. - -onboarding.tor-circuit-display.new-circuit.title=Do you need a new circuit? -onboarding.tor-circuit-display.new-circuit.msg=If you are not able to connect to the website you’re trying to visit or it is not loading properly, then you can use this button to reload the site with a new circuit. ===================================== toolkit/torbutton/chrome/locale/en-US/onboarding.properties deleted ===================================== @@ -1,19 +0,0 @@ -# This Source Code Form is subject to the terms of the Mozilla Public -# License, v. 2.0. If a copy of the MPL was not distributed with this -# file, You can obtain one at http://mozilla.org/MPL/2.0/. -# LOCALIZATION NOTE(onboarding.overlay-icon-tooltip2): This string will be used -# to show the tooltip alongside the notification icon in the overlay tour. %S is -# brandShortName. The tooltip is designed to show in two lines. Please use \n to -# do appropriate line breaking. -onboarding.overlay-icon-tooltip2=New to %S?\nLet’s get started. -# LOCALIZATION NOTE(onboarding.overlay-icon-tooltip-updated2): %S is -# brandShortName. The tooltip is designed to show in two lines. Please use \n to -# do appropriate line breaking. -onboarding.overlay-icon-tooltip-updated2=%S is all new.\nSee what you can do! -# LOCALIZATION NOTE(onboarding.overlay-close-button-tooltip): The overlay close button is an icon button. This tooltip would be shown when mousing hovering on the button. -onboarding.overlay-close-button-tooltip=Close -# LOCALIZATION NOTE(onboarding.notification-close-button-tooltip): The notification close button is an icon button. This tooltip would be shown when mousing hovering on the button. -onboarding.notification-close-button-tooltip=Dismiss -# LOCALIZATION NOTE(onboarding.complete): This string is used to describe an -# onboarding tour item that is complete. -onboarding.complete=Complete View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/commit/08ed48f… -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/commit/08ed48f… You're receiving this email because of your account on gitlab.torproject.org.

1 0

[Git][tpo/applications/tor-browser][tor-browser-115.10.0esr-13.5-1] fixup! Bug 41631: Prevent weird initial window dimensions caused by subpixel computations
by ma1 (＠ma1) 16 Apr '24

16 Apr '24

ma1 pushed to branch tor-browser-115.10.0esr-13.5-1 at The Tor Project / Applications / Tor Browser Commits: 334c638e by hackademix at 2024-04-16T17:24:54+02:00 fixup! Bug 41631: Prevent weird initial window dimensions caused by subpixel computations Bug 42520: Correctly record new initial window size after auto-shrinking - - - - - 1 changed file: - toolkit/components/resistfingerprinting/RFPHelper.sys.mjs Changes: ===================================== toolkit/components/resistfingerprinting/RFPHelper.sys.mjs ===================================== @@ -633,9 +633,16 @@ class _RFPHelper { lazy.logConsole.error(e); } } - if (needToShrink) { - win.shrinkToLetterbox(); - this._recordWindowSize(win); + if (needToShrink && win.shrinkToLetterbox()) { + win.addEventListener( + "resize", + () => { + // We need to record the "new" initial size in this listener + // because resized dimensions are not immediately available. + RFPHelper._recordWindowSize(win); + }, + { once: true } + ); } }); }, @@ -741,23 +748,30 @@ class _RFPHelper { } _recordWindowSize(aWindow) { - aWindow._rfpOriginalSize = { - width: aWindow.outerWidth, - height: aWindow.outerHeight, - containerHeight: aWindow.gBrowser.getBrowserContainer()?.clientHeight, - }; - log("Recording original window size", aWindow._rfpOriginalSize); + aWindow.promiseDocumentFlushed(() => { + aWindow._rfpOriginalSize = { + width: aWindow.outerWidth, + height: aWindow.outerHeight, + containerHeight: aWindow.gBrowser.getBrowserContainer()?.clientHeight, + }; + log("Recording original window size", aWindow._rfpOriginalSize); + }); } // We will attach this method to each browser window. When called // it will instantly resize the window to exactly fit the selected // (possibly letterboxed) browser. + // Returns true if a window resize will occur, false otherwise. shrinkToLetterbox() { let { selectedBrowser } = this.gBrowser; let stack = selectedBrowser.closest(".browserStack"); const outer = stack.getBoundingClientRect(); const inner = selectedBrowser.getBoundingClientRect(); - this.resizeBy(inner.width - outer.width, inner.height - outer.height); + if (inner.width !== outer.witdh || inner.height !== outer.height) { + this.resizeBy(inner.width - outer.width, inner.height - outer.height); + return true; + } + return false; } _onWindowDoubleClick(e) { View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/commit/334c638… -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/commit/334c638… You're receiving this email because of your account on gitlab.torproject.org.

1 0

[Git][tpo/applications/mullvad-browser][mullvad-browser-115.10.0esr-13.5-1] fixup! Base Browser strings
by ma1 (＠ma1) 16 Apr '24

16 Apr '24

ma1 pushed to branch mullvad-browser-115.10.0esr-13.5-1 at The Tor Project / Applications / Mullvad Browser Commits: 9eaf8f6d by hackademix at 2024-04-16T13:14:32+02:00 fixup! Base Browser strings MB 288: Standardize capitalization in Letterboxing preferences - - - - - 1 changed file: - browser/locales/en-US/browser/base-browser.ftl Changes: ===================================== browser/locales/en-US/browser/base-browser.ftl ===================================== @@ -84,7 +84,7 @@ letterboxing-window-size-header = Window size letterboxing-remember-size = .label = Reuse last window size when opening a new window .accesskey = R -letterboxing-alignment-header = Content Alignment +letterboxing-alignment-header = Content alignment letterboxing-alignment-description = Choose where you want to align the website’s content. letterboxing-alignment-top = Top letterboxing-alignment-middle = Middle View it on GitLab: https://gitlab.torproject.org/tpo/applications/mullvad-browser/-/commit/9ea… -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/mullvad-browser/-/commit/9ea… You're receiving this email because of your account on gitlab.torproject.org.

1 0

[Git][tpo/applications/tor-browser][base-browser-115.10.0esr-13.5-1] fixup! Base Browser strings
by ma1 (＠ma1) 16 Apr '24

16 Apr '24

ma1 pushed to branch base-browser-115.10.0esr-13.5-1 at The Tor Project / Applications / Tor Browser Commits: e98e00fa by hackademix at 2024-04-16T13:12:56+02:00 fixup! Base Browser strings MB 288: Standardize capitalization in Letterboxing preferences - - - - - 1 changed file: - browser/locales/en-US/browser/base-browser.ftl Changes: ===================================== browser/locales/en-US/browser/base-browser.ftl ===================================== @@ -84,7 +84,7 @@ letterboxing-window-size-header = Window size letterboxing-remember-size = .label = Reuse last window size when opening a new window .accesskey = R -letterboxing-alignment-header = Content Alignment +letterboxing-alignment-header = Content alignment letterboxing-alignment-description = Choose where you want to align the website’s content. letterboxing-alignment-top = Top letterboxing-alignment-middle = Middle View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/commit/e98e00f… -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/commit/e98e00f… You're receiving this email because of your account on gitlab.torproject.org.

1 0

[Git][tpo/applications/tor-browser][tor-browser-115.10.0esr-13.5-1] fixup! Base Browser strings
by ma1 (＠ma1) 16 Apr '24

16 Apr '24

ma1 pushed to branch tor-browser-115.10.0esr-13.5-1 at The Tor Project / Applications / Tor Browser Commits: 1f1f3ab2 by hackademix at 2024-04-16T13:09:00+02:00 fixup! Base Browser strings MB 288: Standardize capitalization in Letterboxing preferences - - - - - 1 changed file: - browser/locales/en-US/browser/base-browser.ftl Changes: ===================================== browser/locales/en-US/browser/base-browser.ftl ===================================== @@ -84,7 +84,7 @@ letterboxing-window-size-header = Window size letterboxing-remember-size = .label = Reuse last window size when opening a new window .accesskey = R -letterboxing-alignment-header = Content Alignment +letterboxing-alignment-header = Content alignment letterboxing-alignment-description = Choose where you want to align the website’s content. letterboxing-alignment-top = Top letterboxing-alignment-middle = Middle View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/commit/1f1f3ab… -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/commit/1f1f3ab… You're receiving this email because of your account on gitlab.torproject.org.

1 0

[Git][tpo/applications/tor-browser][tor-browser-115.10.0esr-13.5-1] fixup! Bug 41803: Add some developer tools for working on tor-browser.
by ma1 (＠ma1) 16 Apr '24

16 Apr '24

ma1 pushed to branch tor-browser-115.10.0esr-13.5-1 at The Tor Project / Applications / Tor Browser Commits: bbc4a77b by hackademix at 2024-04-16T13:01:03+02:00 fixup! Bug 41803: Add some developer tools for working on tor-browser. Bug 42516: Make tb-dev worktree-compatible - - - - - 1 changed file: - browser/locales/en-US/browser/base-browser.ftl Changes: ===================================== browser/locales/en-US/browser/base-browser.ftl ===================================== @@ -84,7 +84,7 @@ letterboxing-window-size-header = Window size letterboxing-remember-size = .label = Reuse last window size when opening a new window .accesskey = R -letterboxing-alignment-header = Content Alignment +letterboxing-alignment-header = Content alignment letterboxing-alignment-description = Choose where you want to align the website’s content. letterboxing-alignment-top = Top letterboxing-alignment-middle = Middle View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/commit/bbc4a77… -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/commit/bbc4a77… You're receiving this email because of your account on gitlab.torproject.org.

1 0

[Git][tpo/applications/tor-browser-update-responses][main] release: new version, 13.0.14
by richard (＠richard) 16 Apr '24

16 Apr '24

richard pushed to branch main at The Tor Project / Applications / Tor Browser update responses Commits: 56c533bf by Richard Pospesel at 2024-04-16T01:43:32+00:00 release: new version, 13.0.14 - - - - - 30 changed files: - update_3/release/.htaccess - − update_3/release/13.0.10-13.0.13-linux-i686-ALL.xml - − update_3/release/13.0.10-13.0.13-linux-x86_64-ALL.xml - − update_3/release/13.0.10-13.0.13-macos-ALL.xml - − update_3/release/13.0.10-13.0.13-windows-i686-ALL.xml - − update_3/release/13.0.10-13.0.13-windows-x86_64-ALL.xml - − update_3/release/13.0.11-13.0.13-linux-i686-ALL.xml - − update_3/release/13.0.11-13.0.13-linux-x86_64-ALL.xml - − update_3/release/13.0.11-13.0.13-macos-ALL.xml - − update_3/release/13.0.11-13.0.13-windows-i686-ALL.xml - − update_3/release/13.0.11-13.0.13-windows-x86_64-ALL.xml - + update_3/release/13.0.11-13.0.14-linux-i686-ALL.xml - + update_3/release/13.0.11-13.0.14-linux-x86_64-ALL.xml - + update_3/release/13.0.11-13.0.14-macos-ALL.xml - + update_3/release/13.0.11-13.0.14-windows-i686-ALL.xml - + update_3/release/13.0.11-13.0.14-windows-x86_64-ALL.xml - − update_3/release/13.0.12-13.0.13-linux-i686-ALL.xml - − update_3/release/13.0.12-13.0.13-linux-x86_64-ALL.xml - − update_3/release/13.0.12-13.0.13-macos-ALL.xml - − update_3/release/13.0.12-13.0.13-windows-i686-ALL.xml - − update_3/release/13.0.12-13.0.13-windows-x86_64-ALL.xml - + update_3/release/13.0.12-13.0.14-linux-i686-ALL.xml - + update_3/release/13.0.12-13.0.14-linux-x86_64-ALL.xml - + update_3/release/13.0.12-13.0.14-macos-ALL.xml - + update_3/release/13.0.12-13.0.14-windows-i686-ALL.xml - + update_3/release/13.0.12-13.0.14-windows-x86_64-ALL.xml - + update_3/release/13.0.13-13.0.14-linux-i686-ALL.xml - + update_3/release/13.0.13-13.0.14-linux-x86_64-ALL.xml - + update_3/release/13.0.13-13.0.14-macos-ALL.xml - + update_3/release/13.0.13-13.0.14-windows-i686-ALL.xml The diff was not included because it is too large. View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser-update-responses… -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser-update-responses… You're receiving this email because of your account on gitlab.torproject.org.

1 0

[Git][tpo/applications/tor-browser][tor-browser-115.10.0esr-13.5-1] 4 commits: fixup! Bug 40701: Add security warning when downloading a file
by richard (＠richard) 15 Apr '24

15 Apr '24

richard pushed to branch tor-browser-115.10.0esr-13.5-1 at The Tor Project / Applications / Tor Browser Commits: 49db61d8 by Henry Wilkes at 2024-04-15T18:16:31+00:00 fixup! Bug 40701: Add security warning when downloading a file Bug 42221: Migrate downloads warning strings to Fluent. - - - - - 93684ca3 by Henry Wilkes at 2024-04-15T18:16:31+00:00 fixup! Tor Browser strings Bug 42221: Migrate downloads warning strings to Fluent. - - - - - 4e772ce6 by Henry Wilkes at 2024-04-15T18:16:31+00:00 fixup! Add TorStrings module for localization Bug 42221: Migrate downloads warning strings to Fluent. - - - - - e7445294 by Henry Wilkes at 2024-04-15T18:16:31+00:00 fixup! Tor Browser localization migration scripts. Bug 42221: Migrate downloads warning strings to Fluent. - - - - - 9 changed files: - browser/components/downloads/content/contentAreaDownloadsView.js - browser/components/downloads/content/contentAreaDownloadsView.xhtml - browser/components/downloads/content/downloads.js - browser/components/downloads/content/downloadsPanel.inc.xhtml - browser/components/places/content/places.js - browser/components/places/content/places.xhtml - browser/locales/en-US/browser/tor-browser.ftl - toolkit/torbutton/chrome/locale/en-US/torbutton.properties - + tools/torbrowser/l10n/migrations/bug-42210-download-warning.py Changes: ===================================== browser/components/downloads/content/contentAreaDownloadsView.js ===================================== @@ -62,53 +62,6 @@ var ContentAreaDownloadsView = { if (!PrivateBrowsingUtils.isContentWindowPrivate(window)) { view.place = "place:transition=7&sort=4"; } - - torWarningMessage.querySelector( - ".downloads-tor-warning-title" - ).textContent = this._getTorString("torbutton.download.warning.title"); - - const tailsLink = document.createElement("a"); - tailsLink.href = "https://tails.net/"; - tailsLink.target = "_blank"; - tailsLink.textContent = this._getTorString( - "torbutton.download.warning.tails_brand_name" - ); - - const [beforeLink, afterLink] = this._getTorString( - "torbutton.download.warning.description" - ).split("%S"); - - torWarningMessage - .querySelector(".downloads-tor-warning-description") - .append(beforeLink, tailsLink, afterLink); - - torWarningMessage.querySelector( - ".downloads-tor-warning-dismiss-button" - ).textContent = this._getTorString("torbutton.download.warning.dismiss"); - }, - - /** - * Get a string from the properties bundle. - * - * @param {string} name - The string name. - * - * @return {string} The string. - */ - _getTorString(name) { - if (!this._stringBundle) { - this._stringBundle = Services.strings.createBundle( - "chrome://torbutton/locale/torbutton.properties" - ); - } - try { - return this._stringBundle.GetStringFromName(name); - } catch {} - if (!this._fallbackStringBundle) { - this._fallbackStringBundle = Services.strings.createBundle( - "resource://torbutton/locale/en-US/torbutton.properties" - ); - } - return this._fallbackStringBundle.GetStringFromName(name); }, }; ===================================== browser/components/downloads/content/contentAreaDownloadsView.xhtml ===================================== @@ -21,6 +21,7 @@ <linkset> <html:link rel="localization" href="toolkit/global/textActions.ftl"/> <html:link rel="localization" href="browser/downloads.ftl" /> + <html:link rel="localization" href="browser/tor-browser.ftl" /> </linkset> <script src="chrome://global/content/globalOverlay.js"/> @@ -36,20 +37,34 @@ </keyset> #endif - <html:message-bar id="aboutDownloadsTorWarning" - class="downloads-tor-warning-message-bar" - role="alert" - aria-labelledby="aboutDownloadsTorWarningTitle" - aria-describedby="aboutDownloadsTorWarningDescription"> + <html:message-bar + id="aboutDownloadsTorWarning" + class="downloads-tor-warning-message-bar" + role="alert" + aria-labelledby="aboutDownloadsTorWarningTitle" + aria-describedby="aboutDownloadsTorWarningDescription" + > <html:div class="downloads-tor-warning-grid"> - <html:p id="aboutDownloadsTorWarningTitle" - class="downloads-tor-warning-title"> + <html:p + id="aboutDownloadsTorWarningTitle" + class="downloads-tor-warning-title" + data-l10n-id="downloads-tor-warning-title" + ></html:p> + <html:p + id="aboutDownloadsTorWarningDescription" + class="downloads-tor-warning-description" + data-l10n-id="downloads-tor-warning-description" + > + <html:a + href="https://tails.net/" + target="_blank" + data-l10n-name="tails-link" + ></html:a> </html:p> - <html:p id="aboutDownloadsTorWarningDescription" - class="downloads-tor-warning-description"> - </html:p> - <html:button class="downloads-tor-warning-dismiss-button"> - </html:button> + <html:button + class="downloads-tor-warning-dismiss-button" + data-l10n-id="downloads-tor-warning-dismiss-button" + ></html:button> </html:div> </html:message-bar> ===================================== browser/components/downloads/content/downloads.js ===================================== @@ -167,38 +167,29 @@ var DownloadsPanel = { ); if (!this._torWarningInitialized) { - torWarningMessage.querySelector( - ".downloads-tor-warning-title" - ).textContent = this._getTorString("torbutton.download.warning.title"); - - const tailsLink = document.createElement("a"); - tailsLink.href = "https://tails.net"; - tailsLink.textContent = this._getTorString( - "torbutton.download.warning.tails_brand_name" - ); - tailsLink.addEventListener("click", event => { - event.preventDefault(); - this.hidePanel(); - openWebLinkIn(tailsLink.href, "tab"); - }); - - const [beforeLink, afterLink] = this._getTorString( - "torbutton.download.warning.description" - ).split("%S"); - + // Intercept clicks on the tails link. + // NOTE: We listen for clicks on the parent instead of the + // <a data-l10n-name="tails-link"> element because the latter may be + // swapped for a new instance by Fluent when refreshing the parent. torWarningMessage .querySelector(".downloads-tor-warning-description") - .append(beforeLink, tailsLink, afterLink); + .addEventListener("click", event => { + const tailsLink = event.target.closest( + ".downloads-tor-warning-tails-link" + ); + if (!tailsLink) { + return; + } + event.preventDefault(); + this.hidePanel(); + openWebLinkIn(tailsLink.href, "tab"); + }); - let dismissButton = torWarningMessage.querySelector( - ".downloads-tor-warning-dismiss-button" - ); - dismissButton.textContent = this._getTorString( - "torbutton.download.warning.dismiss" - ); - dismissButton.addEventListener("click", event => { - Services.prefs.setBoolPref(PREF_SHOW_DOWNLOAD_WARNING, false); - }); + torWarningMessage + .querySelector(".downloads-tor-warning-dismiss-button") + .addEventListener("click", event => { + Services.prefs.setBoolPref(PREF_SHOW_DOWNLOAD_WARNING, false); + }); this._torWarningInitialized = true; } @@ -725,30 +716,6 @@ var DownloadsPanel = { } }, 0); }, - - /** - * Get a string from the properties bundle. - * - * @param {string} name - The string name. - * - * @return {string} The string. - */ - _getTorString(name) { - if (!this._stringBundle) { - this._stringBundle = Services.strings.createBundle( - "chrome://torbutton/locale/torbutton.properties" - ); - } - try { - return this._stringBundle.GetStringFromName(name); - } catch {} - if (!this._fallbackStringBundle) { - this._fallbackStringBundle = Services.strings.createBundle( - "resource://torbutton/locale/en-US/torbutton.properties" - ); - } - return this._fallbackStringBundle.GetStringFromName(name); - }, }; XPCOMUtils.defineConstant(this, "DownloadsPanel", DownloadsPanel); ===================================== browser/components/downloads/content/downloadsPanel.inc.xhtml ===================================== @@ -125,19 +125,32 @@ <panelview id="downloadsPanel-mainView"> <vbox id="downloadsPanelTorWarning"> - <vbox role="alert" - aria-labelledby="downloadsPanelTorWarningTitle" - aria-describedby="downloadsPanelTorWarningDescription"> - <html:p id="downloadsPanelTorWarningTitle" - class="downloads-tor-warning-title"> + <vbox + role="alert" + aria-labelledby="downloadsPanelTorWarningTitle" + aria-describedby="downloadsPanelTorWarningDescription" + > + <html:p + id="downloadsPanelTorWarningTitle" + class="downloads-tor-warning-title" + data-l10n-id="downloads-tor-warning-title" + ></html:p> + <html:p + id="downloadsPanelTorWarningDescription" + class="downloads-tor-warning-description" + data-l10n-id="downloads-tor-warning-description" + > + <html:a + href="https://tails.net/" + data-l10n-name="tails-link" + class="downloads-tor-warning-tails-link" + ></html:a> </html:p> - <html:p id="downloadsPanelTorWarningDescription" - class="downloads-tor-warning-description"> - </html:p> - <html:div class="panel-footer"> - <html:button class="downloads-tor-warning-dismiss-button"> - </html:button> + <html:button + class="downloads-tor-warning-dismiss-button" + data-l10n-id="downloads-tor-warning-dismiss-button" + ></html:button> </html:div> </vbox> <toolbarseparator /> ===================================== browser/components/places/content/places.js ===================================== @@ -176,32 +176,22 @@ var PlacesOrganizer = { } ); - // Initialize tor warning text content. - torWarningMessage.querySelector( - ".downloads-tor-warning-title" - ).textContent = this._getTorString("torbutton.download.warning.title"); - - const tailsLink = document.createElement("a"); - tailsLink.href = "https://tails.net/"; - tailsLink.textContent = this._getTorString( - "torbutton.download.warning.tails_brand_name" - ); - tailsLink.addEventListener("click", event => { - event.preventDefault(); - openWebLinkIn(tailsLink.href, "tab"); - }); - - const [beforeLink, afterLink] = this._getTorString( - "torbutton.download.warning.description" - ).split("%S"); - - torWarningMessage + // Intercept clicks on the tor warning tails link. + // NOTE: We listen for clicks on the parent instead of the + // <a data-l10n-name="tails-link"> element because the latter may be + // swapped for a new instance by Fluent when refreshing the parent. + document .querySelector(".downloads-tor-warning-description") - .append(beforeLink, tailsLink, afterLink); - - torWarningMessage.querySelector( - ".downloads-tor-warning-dismiss-button" - ).textContent = this._getTorString("torbutton.download.warning.dismiss"); + .addEventListener("click", event => { + const tailsLink = event.target.closest( + ".downloads-tor-warning-tails-link" + ); + if (!tailsLink) { + return; + } + event.preventDefault(); + openWebLinkIn(tailsLink.href, "tab"); + }); ContentArea.init(); @@ -274,30 +264,6 @@ var PlacesOrganizer = { ContentArea.focus(); }, - /** - * Get a string from the properties bundle. - * - * @param {string} name - The string name. - * - * @returns {string} The string. - */ - _getTorString(name) { - if (!this._stringBundle) { - this._stringBundle = Services.strings.createBundle( - "chrome://torbutton/locale/torbutton.properties" - ); - } - try { - return this._stringBundle.GetStringFromName(name); - } catch {} - if (!this._fallbackStringBundle) { - this._fallbackStringBundle = Services.strings.createBundle( - "resource://torbutton/locale/en-US/torbutton.properties" - ); - } - return this._fallbackStringBundle.GetStringFromName(name); - }, - QueryInterface: ChromeUtils.generateQI([]), handleEvent: function PO_handleEvent(aEvent) { ===================================== browser/components/places/content/places.xhtml ===================================== @@ -40,6 +40,7 @@ <html:link rel="localization" href="browser/places.ftl"/> <html:link rel="localization" href="browser/downloads.ftl"/> <html:link rel="localization" href="browser/editBookmarkOverlay.ftl"/> + <html:link rel="localization" href="browser/tor-browser.ftl"/> </linkset> <script src="chrome://browser/content/places/places.js"/> @@ -338,19 +339,33 @@ </tree> <splitter collapse="none" persist="state"></splitter> <vbox id="contentView"> - <html:message-bar id="placesDownloadsTorWarning" - role="alert" - aria-labelledby="placesDownloadsTorWarningTitle" - aria-describedby="placesDownloadsTorWarningDescription"> + <html:message-bar + id="placesDownloadsTorWarning" + role="alert" + aria-labelledby="placesDownloadsTorWarningTitle" + aria-describedby="placesDownloadsTorWarningDescription" + > <html:div class="downloads-tor-warning-grid"> - <html:p id="placesDownloadsTorWarningTitle" - class="downloads-tor-warning-title"> + <html:p + id="placesDownloadsTorWarningTitle" + class="downloads-tor-warning-title" + data-l10n-id="downloads-tor-warning-title" + ></html:p> + <html:p + id="placesDownloadsTorWarningDescription" + class="downloads-tor-warning-description" + data-l10n-id="downloads-tor-warning-description" + > + <html:a + href="https://tails.net/" + class="downloads-tor-warning-tails-link" + data-l10n-name="tails-link" + ></html:a> </html:p> - <html:p id="placesDownloadsTorWarningDescription" - class="downloads-tor-warning-description"> - </html:p> - <html:button class="downloads-tor-warning-dismiss-button"> - </html:button> + <html:button + class="downloads-tor-warning-dismiss-button" + data-l10n-id="downloads-tor-warning-dismiss-button" + ></html:button> </html:div> </html:message-bar> <vbox id="placesViewsBox" flex="1"> ===================================== browser/locales/en-US/browser/tor-browser.ftl ===================================== @@ -548,3 +548,12 @@ crypto-safety-prompt-title = Cryptocurrency address copied from an insecure webs crypto-safety-prompt-body = The copied text ({ $address }) appears to be a cryptocurrency address. Since the connection to { $host } is not secure, the address may have been modified and should not be trusted. You can try establishing a secure connection by reconnecting with a new circuit. crypto-safety-prompt-reload-button = Reload Tab with a New Circuit crypto-safety-prompt-dismiss-button = Dismiss Warning + +## Downloads warning. +## Shown in downloads panel, about:downloads and Library window. + +downloads-tor-warning-title = Be careful opening downloads +# "Tails" is the brand name for the Tails operating system and should be localized appropriately, and will be a link to its website. The name should be wrapped in '<a data-l10n-name="tails-link">' and '</a>'. +downloads-tor-warning-description = Some files may connect to the internet when opened without using Tor. To be safe, open the files while offline or use a portable operating system like <a data-l10n-name="tails-link">Tails</a>. +# Button to dismiss the warning forever. +downloads-tor-warning-dismiss-button = Got it ===================================== toolkit/torbutton/chrome/locale/en-US/torbutton.properties ===================================== @@ -3,14 +3,6 @@ # License, v. 2.0. If a copy of the MPL was not distributed with this # file, You can obtain one at http://mozilla.org/MPL/2.0/. -# Download pane warning -torbutton.download.warning.title = Be careful opening downloads -# %S will be a link to the Tails operating system website. With the content given by torbutton.download.warning.tails_brand_name -torbutton.download.warning.description = Some files may connect to the internet when opened without using Tor. To be safe, open the files while offline or use a portable operating system like %S. -# Locale name for Tails operating system. -torbutton.download.warning.tails_brand_name = Tails -torbutton.download.warning.dismiss = Got it - # .Onion Page Info prompt. pageInfo_OnionEncryptionWithBitsAndProtocol=Connection Encrypted (Onion Service, %1$S, %2$S bit keys, %3$S) pageInfo_OnionEncryption=Connection Encrypted (Onion Service) ===================================== tools/torbrowser/l10n/migrations/bug-42210-download-warning.py ===================================== @@ -0,0 +1,39 @@ +import fluent.syntax.ast as FTL +from fluent.migrate.helpers import transforms_from +from fluent.migrate.transforms import CONCAT, COPY, REPLACE + + +def migrate(ctx): + legacy_path = "torbutton.properties" + + ctx.add_transforms( + "tor-browser.ftl", + "tor-browser.ftl", + transforms_from( + """ +downloads-tor-warning-title = { COPY(path, "torbutton.download.warning.title") } +downloads-tor-warning-dismiss-button = { COPY(path, "torbutton.download.warning.dismiss") } +""", + path=legacy_path, + ) + + [ + # Replace "%S" with link to Tails website. + FTL.Message( + id=FTL.Identifier("downloads-tor-warning-description"), + value=REPLACE( + legacy_path, + "torbutton.download.warning.description", + { + "%1$S": CONCAT( + FTL.TextElement('<a data-l10n-name="tails-link">'), + COPY( + legacy_path, + "torbutton.download.warning.tails_brand_name", + ), + FTL.TextElement("</a>"), + ) + }, + ), + ), + ], + ) View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/compare/a5b9b0… -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/compare/a5b9b0… You're receiving this email because of your account on gitlab.torproject.org.

1 0

[Git][tpo/applications/tor-browser-build] Pushed new tag tbb-13.0.14-build1
by Pier Angelo Vendrame (＠pierov) 15 Apr '24

15 Apr '24

Pier Angelo Vendrame pushed new tag tbb-13.0.14-build1 at The Tor Project / Applications / tor-browser-build -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser-build/-/tree/tbb… You're receiving this email because of your account on gitlab.torproject.org.

1 0

[Git][tpo/applications/tor-browser-build][maint-13.0] Bug 41099: Prepare Tor Browser Stable 13.0.14 (updated)
by Pier Angelo Vendrame (＠pierov) 15 Apr '24

15 Apr '24

Pier Angelo Vendrame pushed to branch maint-13.0 at The Tor Project / Applications / tor-browser-build Commits: a57006e3 by Pier Angelo Vendrame at 2024-04-15T18:44:40+02:00 Bug 41099: Prepare Tor Browser Stable 13.0.14 (updated) Include the fix to #40992/#41122 and updated the changelogs. - - - - - 2 changed files: - projects/browser/Bundle-Data/Docs-TBB/ChangeLog.txt - rbm.conf Changes: ===================================== projects/browser/Bundle-Data/Docs-TBB/ChangeLog.txt ===================================== @@ -4,20 +4,25 @@ Tor Browser 13.0.14 - April 16 2024 * Bug 41676: Set privacy.resistFingerprinting.testing.setTZtoUTC as a defense-in-depth [tor-browser] * Bug 42335: Do not localize the order of locales for app lang [tor-browser] * Bug 42428: Timezone offset leak via document.lastModified [tor-browser] - * Bug 42472: Timezone May leak from XSLT Date function [tor-browser] + * Bug 42472: Timezone may leak from XSLT Date function [tor-browser] * Bug 42508: Rebase Tor Browser stable onto 115.10.0esr [tor-browser] * Windows + macOS + Linux * Updated Firefox to 115.10.0esr + * Bug 42172: browser.startup.homepage and TOR_DEFAULT_HOMEPAGE are ignored for the new window opened by New Identity [tor-browser] * Bug 42236: Let users decide whether to load their home page on new identity. [tor-browser] * Bug 42468: App languages not sorted correctly in stable [tor-browser] - * Android - * Updated GeckoView to 115.10.0esr * Linux * Bug 41110: Avoid Fontconfig warning about "ambiguous path" [tor-browser-build] + * Android + * Updated GeckoView to 115.10.0esr + * Bug 42509: Backport Android security fixes from Firefox 125 [tor-browser] * Build System * All Platforms * Updated Go to 1.21.8 * Bug 41107: Update download-unsigned-sha256sums-gpg-signatures-from-people-tpo for new type of URL [tor-browser-build] + * Bug 41122: Add release date to rbm.conf [tor-browser-build] + * Android + * Bug 40992: Updated torbrowser_version number is not enough to change firefox-android versionCode number [tor-browser-build] Tor Browser 13.5a6 - March 28 2024 * All Platforms ===================================== rbm.conf ===================================== @@ -80,7 +80,7 @@ var: - '13.0.12' - '[% IF c("var/tor-browser") %]13.0.11[% END %]' - '[% IF c("var/mullvad-browser") %]13.0.10[% END %]' - browser_release_date: '2024/03/28 01:02:03' + browser_release_date: '2024/04/16 15:00:00' browser_release_date_timestamp: '[% USE date; date.format(c("var/browser_release_date"), "%s") %]' updater_enabled: 1 build_mar: 1 View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser-build/-/commit/a… -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser-build/-/commit/a… You're receiving this email because of your account on gitlab.torproject.org.

1 0

[Git][tpo/applications/tor-browser][tor-browser-115.10.0esr-13.5-1] fixup! Bug 31286: Implementation of bridge, proxy, and firewall settings in...
by richard (＠richard) 15 Apr '24

15 Apr '24

richard pushed to branch tor-browser-115.10.0esr-13.5-1 at The Tor Project / Applications / Tor Browser Commits: a5b9b03b by Henry Wilkes at 2024-04-15T17:15:04+00:00 fixup! Bug 31286: Implementation of bridge, proxy, and firewall settings in about:preferences#connection Bug 42496: Fix the refresh captcha button's command. - - - - - 2 changed files: - browser/components/torpreferences/content/requestBridgeDialog.js - browser/components/torpreferences/content/requestBridgeDialog.xhtml Changes: ===================================== browser/components/torpreferences/content/requestBridgeDialog.js ===================================== @@ -71,6 +71,9 @@ const gRequestBridgeDialog = { selectors.refreshCaptchaButton ); this._captchaRefreshButton.disabled = true; + this._captchaRefreshButton.addEventListener("command", () => { + this.onRefreshCaptcha(); + }); this._incorrectCaptchaHbox = this._dialog.querySelector( selectors.incorrectCaptchaHbox ===================================== browser/components/torpreferences/content/requestBridgeDialog.xhtml ===================================== @@ -31,7 +31,6 @@ <button id="torPreferences-requestBridge-refreshCaptchaButton" image="chrome://global/skin/icons/reload.svg" - oncommand="requestBridgeDialog.onRefreshCaptcha();" /> </hbox> <hbox id="torPreferences-requestBridge-incorrectCaptchaHbox" align="center"> View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/commit/a5b9b03… -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/commit/a5b9b03… You're receiving this email because of your account on gitlab.torproject.org.

1 0

[Git][tpo/applications/mullvad-browser][mullvad-browser-115.10.0esr-13.5-1] fixup! MB 188: Customize Gitlab Issue and Merge templates
by richard (＠richard) 15 Apr '24

15 Apr '24

richard pushed to branch mullvad-browser-115.10.0esr-13.5-1 at The Tor Project / Applications / Mullvad Browser Commits: 1d3f7ac2 by Richard Pospesel at 2024-04-15T17:13:12+00:00 fixup! MB 188: Customize Gitlab Issue and Merge templates emergency security issue for mullvad browser - - - - - 1 changed file: - .gitlab/issue_templates/Emergency Security Issue.md Changes: ===================================== .gitlab/issue_templates/Emergency Security Issue.md ===================================== @@ -18,9 +18,7 @@ ### Type of Issue: What are we dealing with? - [ ] Security (sandbox escape, remote code execution, etc) -- [ ] Proxy Bypass (traffic contents becoming MITM'able) -- [ ] De-Anonymization (otherwise identifying which website a user is visiting) -- [ ] Cross-Site Linkability (correlating sessions across circuits and websites) +- [ ] Cross-Site Linkability (correlating sessions across websites) - [ ] Disk Leak (persisting session information to disk) - [ ] Other (please explain) @@ -36,16 +34,11 @@ - [ ] **richard** : signing, release - [ ] **thorin** : fingerprinting - [ ] Other Engineering Teams - - [ ] Networking (**ahf**, **dgoulet**) - - [ ] Anti-Censorship (**meskio**, **cohosh**) - [ ] UX (**donuts**) - [ ] TPA (**anarcat**, **lavamind**) - [ ] External Tor Partners - - [ ] Mozilla - - [ ] Mullvad - - [ ] Brave - - [ ] Guardian Project (Orbot, Onion Browser) - - [ ] Tails + - [ ] Mozilla (**tjr**) + - [ ] Mullvad (**ruihildt**) - [ ] Other (please list) ### Urgency: When do we need to act? @@ -72,6 +65,7 @@ Sometimes fixes have side-effects: users lose their data, roadmaps need to be ad - [ ] Start an initial email thread with the following people: - [ ] **bella** + - [ ] **ruihildt**, **support(a)mullvadvpn.net** - [ ] Relevant Applications Developers - [ ] **(Optional)** **micah** - if there are considerations or asks outside the Applications Team @@ -84,6 +78,7 @@ Sometimes fixes have side-effects: users lose their data, roadmaps need to be ad /cc @ma1 /cc @micah /cc @richard +/cc @ruihildt /confidential View it on GitLab: https://gitlab.torproject.org/tpo/applications/mullvad-browser/-/commit/1d3… -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/mullvad-browser/-/commit/1d3… You're receiving this email because of your account on gitlab.torproject.org.

1 0

[Git][tpo/applications/tor-browser-build][main] Bug 41122: Update stable release prep templates
by boklm (＠boklm) 15 Apr '24

15 Apr '24

boklm pushed to branch main at The Tor Project / Applications / tor-browser-build Commits: 8d829699 by Nicolas Vigier at 2024-04-15T17:44:25+02:00 Bug 41122: Update stable release prep templates The patch for #41122 was backported to `maint-13.0`, so we need to update the stable release prep gitlab issue templates. - - - - - 2 changed files: - .gitlab/issue_templates/Release Prep - Mullvad Browser Stable.md - .gitlab/issue_templates/Release Prep - Tor Browser Stable.md Changes: ===================================== .gitlab/issue_templates/Release Prep - Mullvad Browser Stable.md ===================================== @@ -39,6 +39,7 @@ Mullvad Browser Stable lives in the various `maint-$(MULLVAD_BROWSER_MAJOR).$(MU - [ ] Update `rbm.conf` - [ ] `var/torbrowser_version` : update to next version - [ ] `var/torbrowser_build` : update to `$(MULLVAD_BROWSER_BUILD_N)` + - [ ] `var/browser_release_date` : update to build date - [ ] `var/torbrowser_incremental_from` : update to previous Desktop version - **NOTE**: We try to build incrementals for the previous 3 desktop versions except in the case of a watershed update - **IMPORTANT**: Really *actually* make sure this is the previous Desktop version or else the `make mullvadbrowser-incrementals-*` step will fail ===================================== .gitlab/issue_templates/Release Prep - Tor Browser Stable.md ===================================== @@ -38,6 +38,7 @@ Tor Browser Stable lives in the various `maint-$(TOR_BROWSER_MAJOR).$(TOR_BROWSE - [ ] Update `rbm.conf` - [ ] `var/torbrowser_version` : update to next version - [ ] `var/torbrowser_build` : update to `$(TOR_BROWSER_BUILD_N)` + - [ ] `var/browser_release_date` : update to build date - [ ] ***(Desktop Only)***`var/torbrowser_incremental_from` : update to previous Desktop version - **NOTE**: We try to build incrementals for the previous 3 desktop versions except in the case of a watershed update - **IMPORTANT**: Really *actually* make sure this is the previous Desktop version or else the `make torbrowser-incrementals-*` step will fail View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser-build/-/commit/8… -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser-build/-/commit/8… You're receiving this email because of your account on gitlab.torproject.org.

1 0

[Git][tpo/applications/tor-browser][tor-browser-115.10.0esr-13.5-1] 20 commits: fixup! Bug 40597: Implement TorSettings module
by Pier Angelo Vendrame (＠pierov) 15 Apr '24

15 Apr '24

Pier Angelo Vendrame pushed to branch tor-browser-115.10.0esr-13.5-1 at The Tor Project / Applications / Tor Browser Commits: 3865ebab by Pier Angelo Vendrame at 2024-04-15T17:59:33+02:00 fixup! Bug 40597: Implement TorSettings module Move TorSettings and TorLauncherUtil to lazy imnports. Also removed XPCOMUtils in favor of ChromeUtils. - - - - - d53805c2 by Pier Angelo Vendrame at 2024-04-15T17:59:36+02:00 fixup! Temporary changes to about:torconnect for Android. - - - - - 7720009e by Pier Angelo Vendrame at 2024-04-15T17:59:36+02:00 fixup! Bug 40933: Add tor-launcher functionality Rename TorControlPort.sys.mjs's `TorError` to a less generic `TorProtocolError`. - - - - - bb516579 by Pier Angelo Vendrame at 2024-04-15T17:59:37+02:00 dropme! Bug 42247: Android helpers for the TorProvider Revert the changes to TorProvider*.sys.mjs to move them to the correct commit. - - - - - 76623561 by Dan Ballard at 2024-04-15T17:59:37+02:00 fixup! Bug 40933: Add tor-launcher functionality Bug 41187: Add support for tor logs in android integration - - - - - 8d41f191 by Pier Angelo Vendrame at 2024-04-15T17:59:37+02:00 fixup! Bug 40933: Add tor-launcher functionality Do not localize errors in TorProcess, but keep it only as a pure backend file. Moreover, the localized errors would be only shown in the console. Instead, report an error code to let them bubble up to the TorProviderBuilder, where we show them in the "Restart Tor" prompt. - - - - - 05bda694 by Pier Angelo Vendrame at 2024-04-15T17:59:38+02:00 fixup! Bug 27476: Implement about:torconnect captive portal within Tor Browser Formatted comments. - - - - - d783693d by Pier Angelo Vendrame at 2024-04-15T17:59:38+02:00 fixup! Bug 40933: Add tor-launcher functionality Bug 42479: Remove localization from the backend files. Also, use an Error object when the bootstrap fail, in preparation to the removal of TorBootstrapRequest. - - - - - 7dc831bf by Pier Angelo Vendrame at 2024-04-15T17:59:38+02:00 fixup! Add TorStrings module for localization Load some torlauncher strings to show in about:torconnect in TorStrings.sys.mjs. We could move them to torConnect.properties, but maybe we are still in time for moving all of them to Fluent, so I preferred reducing the number of migrations. Also, remove unused strings from torlauncher.properties. - - - - - 04fbc3dc by Pier Angelo Vendrame at 2024-04-15T17:59:39+02:00 fixup! Bug 40597: Implement TorSettings module Bug 42479: Improve TorConnect error handling. To do so, also remove localized error messages. Use error codes, and let the frontend use the localized messages. - - - - - 00c53294 by Pier Angelo Vendrame at 2024-04-15T17:59:39+02:00 fixup! Bug 27476: Implement about:torconnect captive portal within Tor Browser Bug 42479: Improve TorConnect error handling. Consume the new error codes and transform them into localized error messages. - - - - - 13ca87b4 by Pier Angelo Vendrame at 2024-04-15T17:59:39+02:00 fixup! Bug 42247: Android helpers for the TorProvider Bug 42479: Improve TorConnect error handling. Pass the new TorConnect error codes to Android. - - - - - 1e2083d7 by Pier Angelo Vendrame at 2024-04-15T17:59:40+02:00 fixup! Bug 40933: Add tor-launcher functionality This partially reverts commit 34f7e83335cde6653d897717741c9c8d41f858b7. It removes the errorCode logic, and throws non-localized error messages. Also, it removes the no-longer used getFormattedLocalizedString. - - - - - 05516a28 by Pier Angelo Vendrame at 2024-04-15T17:59:40+02:00 fixup! Bug 40933: Add tor-launcher functionality Changed a few calls to #stop since this method now expects an Erro object. Do not redefine topics, import them from TorProviderBuilder.sys.mjs. Use lazy imports for our code. - - - - - 01d7d557 by Pier Angelo Vendrame at 2024-04-15T17:59:40+02:00 fixup! Bug 40597: Implement TorSettings module Various fixes to address tor-browser!968. - - - - - 02df5609 by Pier Angelo Vendrame at 2024-04-15T17:59:41+02:00 fixup! Bug 27476: Implement about:torconnect captive portal within Tor Browser Various fixes to address tor-browser!968. - - - - - 76ee8c58 by Pier Angelo Vendrame at 2024-04-15T17:59:41+02:00 fixup! Bug 42247: Android helpers for the TorProvider TorConnectTopics.BootstrapError is now just Error, and it passes directly the Error object. - - - - - 686fb7e7 by henry at 2024-04-15T17:59:41+02:00 fixup! Add TorStrings module for localization Drop unused strings. - - - - - ec7b6520 by henry at 2024-04-15T17:59:42+02:00 fixup! Bug 27476: Implement about:torconnect captive portal within Tor Browser Fix the offline error handler. - - - - - e162a17a by henry at 2024-04-15T17:59:42+02:00 fixup! Bug 40597: Implement TorSettings module Remove an outdated comment. - - - - - 12 changed files: - mobile/android/geckoview/src/main/java/org/mozilla/geckoview/TorIntegrationAndroid.java - toolkit/components/tor-launcher/TorBootstrapRequest.sys.mjs - toolkit/components/tor-launcher/TorControlPort.sys.mjs - toolkit/components/tor-launcher/TorLauncherUtil.sys.mjs - toolkit/components/tor-launcher/TorProcess.sys.mjs - toolkit/components/tor-launcher/TorProvider.sys.mjs - toolkit/components/torconnect/TorConnectParent.sys.mjs - toolkit/components/torconnect/content/aboutTorConnect.js - toolkit/modules/TorAndroidIntegration.sys.mjs - toolkit/modules/TorConnect.sys.mjs - toolkit/modules/TorStrings.sys.mjs - toolkit/torbutton/chrome/locale/en-US/torlauncher.properties Changes: ===================================== mobile/android/geckoview/src/main/java/org/mozilla/geckoview/TorIntegrationAndroid.java ===================================== @@ -42,10 +42,10 @@ public class TorIntegrationAndroid implements BundleEventListener { private static final String EVENT_TOR_STOP = "GeckoView:Tor:StopTor"; private static final String EVENT_MEEK_START = "GeckoView:Tor:StartMeek"; private static final String EVENT_MEEK_STOP = "GeckoView:Tor:StopMeek"; - private static final String EVENT_BOOTSTRAP_STATE_CHANGED = "GeckoView:Tor:BootstrapStateChanged"; + private static final String EVENT_CONNECT_STATE_CHANGED = "GeckoView:Tor:ConnectStateChanged"; + private static final String EVENT_CONNECT_ERROR = "GeckoView:Tor:ConnectError"; private static final String EVENT_BOOTSTRAP_PROGRESS = "GeckoView:Tor:BootstrapProgress"; private static final String EVENT_BOOTSTRAP_COMPLETE = "GeckoView:Tor:BootstrapComplete"; - private static final String EVENT_BOOTSTRAP_ERROR = "GeckoView:Tor:BootstrapError"; private static final String EVENT_TOR_LOGS = "GeckoView:Tor:Logs"; private static final String EVENT_SETTINGS_READY = "GeckoView:Tor:SettingsReady"; private static final String EVENT_SETTINGS_CHANGED = "GeckoView:Tor:SettingsChanged"; @@ -115,10 +115,10 @@ public class TorIntegrationAndroid implements BundleEventListener { EVENT_MEEK_STOP, EVENT_SETTINGS_READY, EVENT_SETTINGS_CHANGED, - EVENT_BOOTSTRAP_STATE_CHANGED, + EVENT_CONNECT_STATE_CHANGED, + EVENT_CONNECT_ERROR, EVENT_BOOTSTRAP_PROGRESS, EVENT_BOOTSTRAP_COMPLETE, - EVENT_BOOTSTRAP_ERROR, EVENT_TOR_LOGS, EVENT_SETTINGS_OPEN); } @@ -148,28 +148,29 @@ public class TorIntegrationAndroid implements BundleEventListener { } else { Log.w(TAG, "Ignoring a settings changed event that did not have the new settings."); } - } else if (EVENT_BOOTSTRAP_STATE_CHANGED.equals(event)) { + } else if (EVENT_CONNECT_STATE_CHANGED.equals(event)) { String state = message.getString("state"); for (BootstrapStateChangeListener listener: mBootstrapStateListeners) { listener.onBootstrapStateChange(state); } + } else if (EVENT_CONNECT_ERROR.equals(event)) { + String code = message.getString("code"); + String msg = message.getString("message"); + String phase = message.getString("phase"); + String reason = message.getString("reason"); + for (BootstrapStateChangeListener listener: mBootstrapStateListeners) { + listener.onBootstrapError(code, msg, phase, reason); + } } else if (EVENT_BOOTSTRAP_PROGRESS.equals(event)) { double progress = message.getDouble("progress"); - String status = message.getString("status"); boolean hasWarnings = message.getBoolean("hasWarnings"); for (BootstrapStateChangeListener listener: mBootstrapStateListeners) { - listener.onBootstrapProgress(progress, status, hasWarnings); + listener.onBootstrapProgress(progress, hasWarnings); } } else if (EVENT_BOOTSTRAP_COMPLETE.equals(event)) { for (BootstrapStateChangeListener listener: mBootstrapStateListeners) { listener.onBootstrapComplete(); } - } else if (EVENT_BOOTSTRAP_ERROR.equals(event)) { - String msg = message.getString("message"); - String details = message.getString("details"); - for (BootstrapStateChangeListener listener: mBootstrapStateListeners) { - listener.onBootstrapError(msg, details); - } } else if (EVENT_TOR_LOGS.equals(event)) { String msg = message.getString("message"); String type = message.getString("logType"); @@ -575,9 +576,9 @@ public class TorIntegrationAndroid implements BundleEventListener { public interface BootstrapStateChangeListener { void onBootstrapStateChange(String state); - void onBootstrapProgress(double progress, String status, boolean hasWarnings); + void onBootstrapProgress(double progress, boolean hasWarnings); void onBootstrapComplete(); - void onBootstrapError(String message, String details); + void onBootstrapError(String code, String message, String phase, String reason); void onSettingsRequested(); } ===================================== toolkit/components/tor-launcher/TorBootstrapRequest.sys.mjs ===================================== @@ -1,17 +1,15 @@ import { setTimeout, clearTimeout } from "resource://gre/modules/Timer.sys.mjs"; -import { TorProviderBuilder } from "resource://gre/modules/TorProviderBuilder.sys.mjs"; -import { TorLauncherUtil } from "resource://gre/modules/TorLauncherUtil.sys.mjs"; - -/* tor-launcher observer topics */ -export const TorTopics = Object.freeze({ - BootstrapStatus: "TorBootstrapStatus", - BootstrapError: "TorBootstrapError", - LogHasWarnOrErr: "TorLogHasWarnOrErr", +const lazy = {}; + +ChromeUtils.defineESModuleGetters(lazy, { + TorProviderBuilder: "resource://gre/modules/TorProviderBuilder.sys.mjs", + TorProviderTopics: "resource://gre/modules/TorProviderBuilder.sys.mjs", }); // modeled after XMLHttpRequest // nicely encapsulates the observer register/unregister logic +// TODO: Remove this class, and move its logic inside the TorProvider. export class TorBootstrapRequest { // number of ms to wait before we abandon the bootstrap attempt // a value of 0 implies we never wait @@ -20,7 +18,7 @@ export class TorBootstrapRequest { // callbacks for bootstrap process status updates onbootstrapstatus = (progress, status) => {}; onbootstrapcomplete = () => {}; - onbootstraperror = (message, details) => {}; + onbootstraperror = error => {}; // internal resolve() method for bootstrap #bootstrapPromiseResolve = null; @@ -30,10 +28,10 @@ export class TorBootstrapRequest { observe(subject, topic, data) { const obj = subject?.wrappedJSObject; switch (topic) { - case TorTopics.BootstrapStatus: { + case lazy.TorProviderTopics.BootstrapStatus: { const progress = obj.PROGRESS; - const status = TorLauncherUtil.getLocalizedBootstrapStatus(obj, "TAG"); if (this.onbootstrapstatus) { + const status = obj.TAG; this.onbootstrapstatus(progress, status); } if (progress === 100) { @@ -47,9 +45,11 @@ export class TorBootstrapRequest { break; } - case TorTopics.BootstrapError: { + case lazy.TorProviderTopics.BootstrapError: { console.info("TorBootstrapRequest: observerd TorBootstrapError", obj); - this.#stop(obj?.message, obj?.details); + const error = new Error(obj.summary); + Object.assign(error, obj); + this.#stop(error); break; } } @@ -65,17 +65,17 @@ export class TorBootstrapRequest { this.#bootstrapPromiseResolve = resolve; // register ourselves to listen for bootstrap events - Services.obs.addObserver(this, TorTopics.BootstrapStatus); - Services.obs.addObserver(this, TorTopics.BootstrapError); + Services.obs.addObserver(this, lazy.TorProviderTopics.BootstrapStatus); + Services.obs.addObserver(this, lazy.TorProviderTopics.BootstrapError); // optionally cancel bootstrap after a given timeout if (this.timeout > 0) { - this.#timeoutID = setTimeout(async () => { + this.#timeoutID = setTimeout(() => { this.#timeoutID = null; - // TODO: Translate, if really used - await this.#stop( - "Tor Bootstrap process timed out", - `Bootstrap attempt abandoned after waiting ${this.timeout} ms` + this.#stop( + new Error( + `Bootstrap attempt abandoned after waiting ${this.timeout} ms` + ) ); }, this.timeout); } @@ -83,15 +83,15 @@ export class TorBootstrapRequest { // Wait for bootstrapping to begin and maybe handle error. // Notice that we do not resolve the promise here in case of success, but // we do it from the BootstrapStatus observer. - TorProviderBuilder.build() + lazy.TorProviderBuilder.build() .then(provider => provider.connect()) .catch(err => { - this.#stop(err.message, err.torMessage); + this.#stop(err); }); }).finally(() => { // and remove ourselves once bootstrap is resolved - Services.obs.removeObserver(this, TorTopics.BootstrapStatus); - Services.obs.removeObserver(this, TorTopics.BootstrapError); + Services.obs.removeObserver(this, lazy.TorProviderTopics.BootstrapStatus); + Services.obs.removeObserver(this, lazy.TorProviderTopics.BootstrapError); this.#bootstrapPromise = null; }); @@ -103,7 +103,7 @@ export class TorBootstrapRequest { } // Internal implementation. Do not use directly, but call cancel, instead. - async #stop(message, details) { + async #stop(error) { // first stop our bootstrap timeout before handling the error if (this.#timeoutID !== null) { clearTimeout(this.#timeoutID); @@ -112,7 +112,7 @@ export class TorBootstrapRequest { let provider; try { - provider = await TorProviderBuilder.build(); + provider = await lazy.TorProviderBuilder.build(); } catch { // This was probably the error that lead to stop in the first place. // No need to continue propagating it. @@ -121,14 +121,13 @@ export class TorBootstrapRequest { await provider?.stopBootstrap(); } catch (e) { console.error("Failed to stop the bootstrap.", e); - if (!message) { - message = e.message; - details = ""; + if (!error) { + error = e; } } - if (this.onbootstraperror && message) { - this.onbootstraperror(message, details); + if (this.onbootstraperror && error) { + this.onbootstraperror(error); } this.#bootstrapPromiseResolve(false); ===================================== toolkit/components/tor-launcher/TorControlPort.sys.mjs ===================================== @@ -311,10 +311,10 @@ class AsyncSocket { * @param {string} message The message to handle */ -class TorError extends Error { +class TorProtocolError extends Error { constructor(command, reply) { super(`${command} -> ${reply}`); - this.name = "TorError"; + this.name = "TorProtocolError"; const info = reply.match(/(?<code>\d{3})(?:\s(?<message>.+))?/); this.torStatusCode = info.groups.code; if (info.groups.message) { @@ -591,7 +591,7 @@ export class TorController { async #sendCommandSimple(command) { const reply = await this.#sendCommand(command); if (!/^250 OK\s*$/i.test(reply)) { - throw new TorError(command, reply); + throw new TorProtocolError(command, reply); } } @@ -672,7 +672,7 @@ export class TorController { reply.match(/^250-([^=]+)=(.*)$/m) || reply.match(/^250\+([^=]+)=\r?\n(.*?)\r?\n^\.\r?\n^250 OK\s*$/ms); if (!match || match[1] !== key) { - throw new TorError(cmd, reply); + throw new TorProtocolError(cmd, reply); } return match[2]; } @@ -784,7 +784,7 @@ export class TorController { TorParsers.unescapeString(m[1]) ); if (!values.length) { - throw new TorError(cmd, reply); + throw new TorProtocolError(cmd, reply); } return values; } @@ -896,7 +896,7 @@ export class TorController { const message = await this.#sendCommand(cmd); // Either `250-CLIENT`, or `250 OK` if no keys are available. if (!message.startsWith("250")) { - throw new TorError(cmd, message); + throw new TorProtocolError(cmd, message); } const re = /^250-CLIENT\s+(?<HSAddress>[A-Za-z2-7]+)\s+(?<KeyType>[^:]+):(?<PrivateKeyBlob>\S+)(?:\s(?<other>.+))?$/gim; @@ -936,7 +936,7 @@ export class TorController { const reply = await this.#sendCommand(cmd); const status = reply.substring(0, 3); if (status !== "250" && status !== "251" && status !== "252") { - throw new TorError(cmd, reply); + throw new TorProtocolError(cmd, reply); } } @@ -952,7 +952,7 @@ export class TorController { const reply = await this.#sendCommand(cmd); const status = reply.substring(0, 3); if (status !== "250" && status !== "251") { - throw new TorError(cmd, reply); + throw new TorProtocolError(cmd, reply); } } @@ -1085,6 +1085,7 @@ export class TorController { ); } const status = { + // Type is actually StatusSeverity in the specifications. TYPE: match[1], ...this.#getKeyValues(match[2]), }; ===================================== toolkit/components/tor-launcher/TorLauncherUtil.sys.mjs ===================================== @@ -417,70 +417,6 @@ export const TorLauncherUtil = Object.freeze({ return aStringName; }, - // "torlauncher." is prepended to aStringName. - getFormattedLocalizedString(aStringName, aArray, aLen) { - if (!aStringName || !aArray) { - return aStringName; - } - try { - const key = kPropNamePrefix + aStringName; - return this._stringBundle.formatStringFromName(key, aArray, aLen); - } catch (e) {} - return aStringName; - }, - - getLocalizedBootstrapStatus(aStatusObj, aKeyword) { - if (!aStatusObj || !aKeyword) { - return ""; - } - - let result; - let fallbackStr; - if (aStatusObj[aKeyword]) { - let val = aStatusObj[aKeyword].toLowerCase(); - let key; - if (aKeyword === "TAG") { - // The bootstrap status tags in tagMap below are used by Tor - // versions prior to 0.4.0.x. We map each one to the tag that will - // produce the localized string that is the best fit. - const tagMap = { - conn_dir: "conn", - handshake_dir: "onehop_create", - conn_or: "enough_dirinfo", - handshake_or: "ap_conn", - }; - if (val in tagMap) { - val = tagMap[val]; - } - - key = "bootstrapStatus." + val; - fallbackStr = aStatusObj.SUMMARY; - } else if (aKeyword === "REASON") { - if (val === "connectreset") { - val = "connectrefused"; - } - - key = "bootstrapWarning." + val; - fallbackStr = aStatusObj.WARNING; - } - - result = TorLauncherUtil.getLocalizedString(key); - if (result === key) { - result = undefined; - } - } - - if (!result) { - result = fallbackStr; - } - - if (aKeyword === "REASON" && aStatusObj.HOSTADDR) { - result += " - " + aStatusObj.HOSTADDR; - } - - return result ? result : ""; - }, - /** * Determine what kind of SOCKS port has been requested for this session or * the browser has been configured for. ===================================== toolkit/components/tor-launcher/TorProcess.sys.mjs ===================================== @@ -193,54 +193,54 @@ export class TorProcess { #makeArgs() { this.#exeFile = lazy.TorLauncherUtil.getTorFile("tor", false); + if (!this.#exeFile) { + throw new Error("Could not find the tor binary."); + } const torrcFile = lazy.TorLauncherUtil.getTorFile("torrc", true); + if (!torrcFile) { + // FIXME: Is this still a fatal error? + throw new Error("Could not find the torrc."); + } // Get the Tor data directory first so it is created before we try to // construct paths to files that will be inside it. this.#dataDir = lazy.TorLauncherUtil.getTorFile("tordatadir", true); + if (!this.#dataDir) { + throw new Error("Could not find the tor data directory."); + } const onionAuthDir = lazy.TorLauncherUtil.getTorFile( "toronionauthdir", true ); - let detailsKey; - if (!this.#exeFile) { - detailsKey = "tor_missing"; - } else if (!torrcFile) { - detailsKey = "torrc_missing"; - } else if (!this.#dataDir) { - detailsKey = "datadir_missing"; - } else if (!onionAuthDir) { - detailsKey = "onionauthdir_missing"; - } - if (detailsKey) { - const details = lazy.TorLauncherUtil.getLocalizedString(detailsKey); - const key = "unable_to_start_tor"; - const err = lazy.TorLauncherUtil.getFormattedLocalizedString( - key, - [details], - 1 - ); - throw new Error(err); + if (!onionAuthDir) { + throw new Error("Could not find the tor onion authentication directory."); } + this.#args = []; + this.#args.push("-f", torrcFile.path); + this.#args.push("DataDirectory", this.#dataDir.path); + this.#args.push("ClientOnionAuthDir", onionAuthDir.path); + + // TODO: Create this starting from pt_config.json (tor-browser#42357). const torrcDefaultsFile = lazy.TorLauncherUtil.getTorFile( "torrc-defaults", false ); - // The geoip and geoip6 files are in the same directory as torrc-defaults. - const geoipFile = torrcDefaultsFile.clone(); - geoipFile.leafName = "geoip"; - const geoip6File = torrcDefaultsFile.clone(); - geoip6File.leafName = "geoip6"; - - this.#args = []; if (torrcDefaultsFile) { this.#args.push("--defaults-torrc", torrcDefaultsFile.path); + // The geoip and geoip6 files are in the same directory as torrc-defaults. + // TODO: Change TorFile to return the generic path to these files to make + // them independent from the torrc-defaults. + const geoipFile = torrcDefaultsFile.clone(); + geoipFile.leafName = "geoip"; + this.#args.push("GeoIPFile", geoipFile.path); + const geoip6File = torrcDefaultsFile.clone(); + geoip6File.leafName = "geoip6"; + this.#args.push("GeoIPv6File", geoip6File.path); + } else { + logger.warn( + "torrc-defaults was not found, some functionalities will be disabled." + ); } - this.#args.push("-f", torrcFile.path); - this.#args.push("DataDirectory", this.#dataDir.path); - this.#args.push("ClientOnionAuthDir", onionAuthDir.path); - this.#args.push("GeoIPFile", geoipFile.path); - this.#args.push("GeoIPv6File", geoip6File.path); } /** ===================================== toolkit/components/tor-launcher/TorProvider.sys.mjs ===================================== @@ -911,6 +911,7 @@ export class TorProvider { * @param {object} status The status object */ onBootstrapStatus(status) { + logger.debug("Received bootstrap status update", status); this.#processBootstrapStatus(status, true); } @@ -939,6 +940,7 @@ export class TorProvider { this.#isBootstrapDone = false; + // Can TYPE ever be ERR for STATUS_CLIENT? if ( isNotification && statusObj.TYPE === "WARN" && @@ -959,20 +961,7 @@ export class TorProvider { } catch (e) { logger.warn(`Cannot set ${Preferences.PromptAtStartup}`, e); } - // TODO: Move l10n to the above layers? - const phase = TorLauncherUtil.getLocalizedBootstrapStatus(statusObj, "TAG"); - const reason = TorLauncherUtil.getLocalizedBootstrapStatus( - statusObj, - "REASON" - ); - const details = TorLauncherUtil.getFormattedLocalizedString( - "tor_bootstrap_failed_details", - [phase, reason], - 2 - ); - logger.error( - `Tor bootstrap error: [${statusObj.TAG}/${statusObj.REASON}] ${details}` - ); + logger.error("Tor bootstrap error", statusObj); if ( statusObj.TAG !== this.#lastWarning.phase || @@ -981,11 +970,19 @@ export class TorProvider { this.#lastWarning.phase = statusObj.TAG; this.#lastWarning.reason = statusObj.REASON; - const message = TorLauncherUtil.getLocalizedString( - "tor_bootstrap_failed" - ); + // FIXME: currently, this is observed only by TorBoostrapRequest. + // We should remove that class, and use an async method to do the + // bootstrap here. + // At that point, the lastWarning mechanism will probably not be necessary + // anymore, since the first error eligible for notification will as a + // matter of fact cancel the bootstrap. Services.obs.notifyObservers( - { message, details }, + { + phase: statusObj.TAG, + reason: statusObj.REASON, + summary: statusObj.SUMMARY, + warning: statusObj.WARNING, + }, TorProviderTopics.BootstrapError ); } ===================================== toolkit/components/torconnect/TorConnectParent.sys.mjs ===================================== @@ -37,10 +37,9 @@ export class TorConnectParent extends JSWindowActorParent { State: TorConnect.state, StateChanged: false, PreviousState: TorConnectState.Initial, - ErrorMessage: TorConnect.errorMessage, + ErrorCode: TorConnect.errorCode, ErrorDetails: TorConnect.errorDetails, BootstrapProgress: TorConnect.bootstrapProgress, - BootstrapStatus: TorConnect.bootstrapStatus, InternetStatus: TorConnect.internetStatus, DetectedLocation: TorConnect.detectedLocation, ShowViewLog: TorConnect.logHasWarningOrError, @@ -59,36 +58,36 @@ export class TorConnectParent extends JSWindowActorParent { this.state.QuickStartEnabled = false; } - // JSWindowActiveParent derived objects cannot observe directly, so create a member - // object to do our observing for us + // JSWindowActiveParent derived objects cannot observe directly, so create a + // member object to do our observing for us. // - // This object converts the various lifecycle events from the TorConnect module, and - // maintains a state object which we pass down to our about:torconnect page, which uses - // the state object to update its UI + // This object converts the various lifecycle events from the TorConnect + // module, and maintains a state object which we pass down to our + // about:torconnect page, which uses the state object to update its UI. this.torConnectObserver = { observe(aSubject, aTopic, aData) { let obj = aSubject?.wrappedJSObject; - // update our state struct based on received torconnect topics and forward on - // to aboutTorConnect.js + // Update our state struct based on received torconnect topics and + // forward on to aboutTorConnect.js. self.state.StateChanged = false; switch (aTopic) { case TorConnectTopics.StateChange: { self.state.PreviousState = self.state.State; self.state.State = obj.state; self.state.StateChanged = true; - - // clear any previous error information if we are bootstrapping + // Clear any previous error information if we are bootstrapping. if (self.state.State === TorConnectState.Bootstrapping) { - self.state.ErrorMessage = null; + self.state.ErrorCode = null; self.state.ErrorDetails = null; } + self.state.BootstrapProgress = TorConnect.bootstrapProgress; + self.state.ShowViewLog = TorConnect.logHasWarningOrError; self.state.HasEverFailed = TorConnect.hasEverFailed; break; } case TorConnectTopics.BootstrapProgress: { self.state.BootstrapProgress = obj.progress; - self.state.BootstrapStatus = obj.status; self.state.ShowViewLog = obj.hasWarnings; break; } @@ -96,9 +95,9 @@ export class TorConnectParent extends JSWindowActorParent { // noop break; } - case TorConnectTopics.BootstrapError: { - self.state.ErrorMessage = obj.message; - self.state.ErrorDetails = obj.details; + case TorConnectTopics.Error: { + self.state.ErrorCode = obj.code; + self.state.ErrorDetails = obj; self.state.InternetStatus = TorConnect.internetStatus; self.state.DetectedLocation = TorConnect.detectedLocation; self.state.ShowViewLog = true; @@ -134,7 +133,7 @@ export class TorConnectParent extends JSWindowActorParent { }, }; - // observe all of the torconnect:.* topics + // Observe all of the torconnect:.* topics. for (const key in TorConnectTopics) { const topic = TorConnectTopics[key]; Services.obs.addObserver(this.torConnectObserver, topic); @@ -158,7 +157,7 @@ export class TorConnectParent extends JSWindowActorParent { } willDestroy() { - // stop observing all of our torconnect:.* topics + // Stop observing all of our torconnect:.* topics. for (const key in TorConnectTopics) { const topic = TorConnectTopics[key]; Services.obs.removeObserver(this.torConnectObserver, topic); @@ -213,7 +212,8 @@ export class TorConnectParent extends JSWindowActorParent { Services.obs.notifyObservers(message.data, BroadcastTopic); break; case "torconnect:get-init-args": - // called on AboutTorConnect.init(), pass down all state data it needs to init + // Called on AboutTorConnect.init(), pass down all state data it needs + // to init. // pretend this is a state transition on init // so we always get fresh UI ===================================== toolkit/components/torconnect/content/aboutTorConnect.js ===================================== @@ -347,6 +347,59 @@ class AboutTorConnect { this.elements.breadcrumbContainer.classList.add("hidden"); } + getLocalizedStatus(status) { + const aliases = { + conn_dir: "conn", + handshake_dir: "onehop_create", + conn_or: "enough_dirinfo", + handshake_or: "ap_conn", + }; + if (status in aliases) { + status = aliases[status]; + } + return TorStrings.torConnect.bootstrapStatus[status] ?? status; + } + + getMaybeLocalizedError(state) { + if (!state?.ErrorCode) { + return ""; + } + switch (state.ErrorCode) { + case "Offline": + return TorStrings.torConnect.offline; + case "BootstrapError": { + const details = state.ErrorDetails?.cause; + if (!details?.phase || !details?.reason) { + return TorStrings.torConnect.torBootstrapFailed; + } + let status = this.getLocalizedStatus(details.phase); + const reason = + TorStrings.torConnect.bootstrapWarning[details.reason] ?? + details.reason; + return TorStrings.torConnect.bootstrapFailedDetails + .replace("%1$S", status) + .replace("%2$S", reason); + } + case "CannotDetermineCountry": + return TorStrings.torConnect.cannotDetermineCountry; + case "NoSettingsForCountry": + return TorStrings.torConnect.noSettingsForCountry; + case "AllSettingsFailed": + return TorStrings.torConnect.autoBootstrappingAllFailed; + case "ExternaError": + // A standard JS error, or something for which we do probably do not + // have a translation. Returning the original message is the best we can + // do. + return state.ErrorDetails.message; + default: + console.warn( + `Unknown error code: ${state.ErrorCode}`, + state.ErrorDetails + ); + return state.ErrorDetails?.message ?? state.ErrorCode; + } + } + /* These methods update the UI based on the current TorConnect state */ @@ -505,10 +558,10 @@ class AboutTorConnect { } } - showOffline(error) { + showOffline(state) { this.setTitle(TorStrings.torConnect.noInternet, "offline"); this.setLongText(TorStrings.torConnect.noInternetDescription); - this.setProgress(error, false); + this.setProgress(this.getMaybeLocalizedError(state), false); this.setBreadcrumbsStatus( BreadcrumbStatus.Default, BreadcrumbStatus.Active, @@ -524,7 +577,7 @@ class AboutTorConnect { showConnectionAssistant(state) { this.setTitle(TorStrings.torConnect.couldNotConnect, "assist"); this.showConfigureConnectionLink(TorStrings.torConnect.assistDescription); - this.setProgress(state?.ErrorDetails, false); + this.setProgress(this.getMaybeLocalizedError(state), false); this.setBreadcrumbsStatus( BreadcrumbStatus.Default, BreadcrumbStatus.Active, @@ -544,7 +597,7 @@ class AboutTorConnect { this.showConfigureConnectionLink( TorStrings.torConnect.errorLocationDescription ); - this.setProgress(state.ErrorMessage, false); + this.setProgress(TorStrings.torConnect.cannotDetermineCountry, false); this.setBreadcrumbsStatus( BreadcrumbStatus.Default, BreadcrumbStatus.Active, @@ -564,7 +617,7 @@ class AboutTorConnect { this.showConfigureConnectionLink( TorStrings.torConnect.isLocationCorrectDescription ); - this.setProgress(state.ErrorMessage, false); + this.setProgress(this.getMaybeLocalizedError(state), false); this.setBreadcrumbsStatus( BreadcrumbStatus.Default, BreadcrumbStatus.Default, @@ -582,7 +635,7 @@ class AboutTorConnect { showFinalError(state) { this.setTitle(TorStrings.torConnect.finalError, "final"); this.setLongText(TorStrings.torConnect.finalErrorDescription); - this.setProgress(state ? state.ErrorDetails : "", false); + this.setProgress(this.getMaybeLocalizedError(state), false); this.setBreadcrumbsStatus( BreadcrumbStatus.Default, BreadcrumbStatus.Default, @@ -666,7 +719,7 @@ class AboutTorConnect { } initElements(direction) { - const isAndroid = navigator.userAgent.indexOf("Android") !== -1; + const isAndroid = navigator.userAgent.includes("Android"); document.body.classList.toggle("android", isAndroid); document.documentElement.setAttribute("dir", direction); ===================================== toolkit/modules/TorAndroidIntegration.sys.mjs ===================================== @@ -29,10 +29,10 @@ const logger = new ConsoleAPI({ const EmittedEvents = Object.freeze({ settingsReady: "GeckoView:Tor:SettingsReady", settingsChanged: "GeckoView:Tor:SettingsChanged", - bootstrapStateChanged: "GeckoView:Tor:BootstrapStateChanged", + connectStateChanged: "GeckoView:Tor:ConnectStateChanged", + connectError: "GeckoView:Tor:ConnectError", bootstrapProgress: "GeckoView:Tor:BootstrapProgress", bootstrapComplete: "GeckoView:Tor:BootstrapComplete", - bootstrapError: "GeckoView:Tor:BootstrapError", torLogs: "GeckoView:Tor:Logs", }); @@ -98,15 +98,14 @@ class TorAndroidIntegrationImpl { break; case lazy.TorConnectTopics.StateChange: lazy.EventDispatcher.instance.sendRequest({ - type: EmittedEvents.bootstrapStateChanged, + type: EmittedEvents.connectStateChanged, state: subj.wrappedJSObject.state ?? "", }); break; case lazy.TorConnectTopics.BootstrapProgress: lazy.EventDispatcher.instance.sendRequest({ type: EmittedEvents.bootstrapProgress, - progress: subj.wrappedJSObject.progress ?? "", - status: subj.wrappedJSObject.status ?? 0, + progress: subj.wrappedJSObject.progress ?? 0, hasWarnings: subj.wrappedJSObject.hasWarnings ?? false, }); break; @@ -115,11 +114,13 @@ class TorAndroidIntegrationImpl { type: EmittedEvents.bootstrapComplete, }); break; - case lazy.TorConnectTopics.BootstrapError: + case lazy.TorConnectTopics.Error: lazy.EventDispatcher.instance.sendRequest({ - type: EmittedEvents.bootstrapError, + type: EmittedEvents.connectError, + code: subj.wrappedJSObject.code ?? "", message: subj.wrappedJSObject.message ?? "", - details: subj.wrappedJSObject.details ?? "", + phase: subj.wrappedJSObject.cause?.phase ?? "", + reason: subj.wrappedJSObject.cause?.reason ?? "", }); break; case lazy.TorProviderTopics.TorLog: ===================================== toolkit/modules/TorConnect.sys.mjs ===================================== @@ -2,7 +2,6 @@ * License, v. 2.0. If a copy of the MPL was not distributed with this * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ -import { XPCOMUtils } from "resource://gre/modules/XPCOMUtils.sys.mjs"; import { setTimeout, clearTimeout } from "resource://gre/modules/Timer.sys.mjs"; const lazy = {}; @@ -13,6 +12,9 @@ ChromeUtils.defineESModuleGetters(lazy, { MoatRPC: "resource://gre/modules/Moat.sys.mjs", TorBootstrapRequest: "resource://gre/modules/TorBootstrapRequest.sys.mjs", TorProviderBuilder: "resource://gre/modules/TorProviderBuilder.sys.mjs", + TorProviderTopics: "resource://gre/modules/TorProviderBuilder.sys.mjs", + TorLauncherUtil: "resource://gre/modules/TorLauncherUtil.sys.mjs", + TorSettings: "resource://gre/modules/TorSettings.sys.mjs", }); // TODO: Should we move this to the about:torconnect actor? @@ -22,16 +24,6 @@ ChromeUtils.defineModuleGetter( "resource:///modules/BrowserWindowTracker.jsm" ); -import { TorLauncherUtil } from "resource://gre/modules/TorLauncherUtil.sys.mjs"; -import { TorSettings } from "resource://gre/modules/TorSettings.sys.mjs"; - -import { TorStrings } from "resource://gre/modules/TorStrings.sys.mjs"; - -const TorTopics = Object.freeze({ - LogHasWarnOrErr: "TorLogHasWarnOrErr", - ProcessExited: "TorProcessExited", -}); - /* Relevant prefs used by tor-launcher */ const TorLauncherPrefs = Object.freeze({ prompt_at_startup: "extensions.torlauncher.prompt_at_startup", @@ -60,7 +52,23 @@ export const TorConnectState = Object.freeze({ Disabled: "Disabled", }); -XPCOMUtils.defineLazyGetter( +export class TorConnectError extends Error { + static Offline = "Offline"; + static BootstrapError = "BootstrapError"; + static CannotDetermineCountry = "CannotDetermineCountry"; + static NoSettingsForCountry = "NoSettingsForCountry"; + static AllSettingsFailed = "AllSettingsFailed"; + static ExternalError = "ExternalError"; + + constructor(code, cause) { + super(cause?.message ?? `TorConnectError: ${code}`, cause ? { cause } : {}); + this.name = "TorConnectError"; + this.code = code; + } +} +Object.freeze(TorConnectError); + +ChromeUtils.defineLazyGetter( lazy, "logger", () => @@ -122,7 +130,7 @@ export const TorConnectTopics = Object.freeze({ StateChange: "torconnect:state-change", BootstrapProgress: "torconnect:bootstrap-progress", BootstrapComplete: "torconnect:bootstrap-complete", - BootstrapError: "torconnect:bootstrap-error", + Error: "torconnect:error", }); // The StateCallback is the base class to implement the various states. @@ -173,7 +181,7 @@ class StateCallback { `${this.#state}'s run threw, transitioning to the Error state.`, err ); - this.changeState(TorConnectState.Error, err?.message, err?.details); + this.changeState(TorConnectState.Error, err); } } @@ -263,14 +271,13 @@ class ConfiguringState extends StateCallback { } run() { - // The configuring state does not do anything. + TorConnect._bootstrapProgress = 0; } } class BootstrappingState extends StateCallback { #bootstrap = null; - #bootstrapError = ""; - #bootstrapErrorDetails = ""; + #bootstrapError = null; #internetTest = null; #cancelled = false; @@ -291,13 +298,13 @@ class BootstrappingState extends StateCallback { this.#bootstrap = new lazy.TorBootstrapRequest(); this.#bootstrap.onbootstrapstatus = (progress, status) => { - TorConnect._updateBootstrapStatus(progress, status); + TorConnect._updateBootstrapProgress(progress, status); }; this.#bootstrap.onbootstrapcomplete = () => { this.#internetTest.cancel(); this.changeState(TorConnectState.Bootstrapped); }; - this.#bootstrap.onbootstraperror = (message, details) => { + this.#bootstrap.onbootstraperror = error => { if (this.#cancelled) { // We ignore this error since it occurred after cancelling (by the // user). We assume the error is just a side effect of the cancelling. @@ -305,13 +312,12 @@ class BootstrappingState extends StateCallback { // "Building circuits: Establishing a Tor circuit failed". // TODO: Maybe move this logic deeper in the process to know when to // filter out such errors triggered by cancelling. - lazy.logger.warn(`Post-cancel error => ${message}; ${details}`); + lazy.logger.warn("Post-cancel error.", error); return; } // We have to wait for the Internet test to finish before sending the // bootstrap error - this.#bootstrapError = message; - this.#bootstrapErrorDetails = details; + this.#bootstrapError = error; this.#maybeTransitionToError(); }; @@ -352,22 +358,23 @@ class BootstrappingState extends StateCallback { // Do not transition to the offline error until we are sure that also the // bootstrap failed, in case Moat is down but the bootstrap can proceed // anyway. - if (this.#bootstrapError === "") { + if (!this.#bootstrapError) { return; } if (this.#internetTest.status === InternetStatus.Offline) { this.changeState( TorConnectState.Error, - TorStrings.torConnect.offline, - "" + new TorConnectError(TorConnectError.Offline) ); } else { // Give priority to the bootstrap error, in case the Internet test fails TorConnect._hasBootstrapEverFailed = true; this.changeState( TorConnectState.Error, - this.#bootstrapError, - this.#bootstrapErrorDetails + new TorConnectError( + TorConnectError.BootstrapError, + this.#bootstrapError + ) ); } } @@ -393,10 +400,12 @@ class BootstrappingState extends StateCallback { TorConnect._detectedLocation = codes[Math.floor(Math.random() * codes.length)]; } + const err = new Error("Censorship simulation"); + err.phase = "conn"; + err.reason = "noroute"; this.changeState( TorConnectState.Error, - "Bootstrap failed (for debugging purposes)", - "Error: Censorship simulation" + new TorConnectError(TorConnectError.BootstrapError, err) ); return true; } @@ -461,8 +470,7 @@ class AutoBootstrappingState extends StateCallback { if (!this.transitioning) { this.changeState( TorConnectState.Error, - "Error: censorship simulation", - "" + new TorConnectError(TorConnectError.AllSettingsFailed) ); } return true; @@ -475,8 +483,7 @@ class AutoBootstrappingState extends StateCallback { if (!this.transitioning) { this.changeState( TorConnectState.Error, - "Error: Severe Censorship simulation", - "" + new TorConnectError(TorConnectError.CannotDetermineCountry) ); } return true; @@ -507,7 +514,7 @@ class AutoBootstrappingState extends StateCallback { // them. const maybeSettings = await Promise.race([ this.#moat.circumvention_settings( - [...TorSettings.builtinBridgeTypes, "vanilla"], + [...lazy.TorSettings.builtinBridgeTypes, "vanilla"], countryCode ), // This might set maybeSettings to undefined. @@ -523,7 +530,7 @@ class AutoBootstrappingState extends StateCallback { // Keep consistency with the other call. this.#settings = await Promise.race([ this.#moat.circumvention_defaults([ - ...TorSettings.builtinBridgeTypes, + ...lazy.TorSettings.builtinBridgeTypes, "vanilla", ]), // This might set this.#settings to undefined. @@ -532,22 +539,12 @@ class AutoBootstrappingState extends StateCallback { } if (!this.#settings?.length && !this.transitioning) { - // Both localized and fallback have, we can just throw to transition to - // the error state (but only if we aren't already transitioning). - // TODO: Let the UI layer localize the strings. - if (!TorConnect._detectedLocation) { // unable to determine country - this.#throwError( - TorStrings.torConnect.autoBootstrappingFailed, - TorStrings.torConnect.cannotDetermineCountry - ); + throw new TorConnectError(TorConnectError.CannotDetermineCountry); } else { // no settings available for country - this.#throwError( - TorStrings.torConnect.autoBootstrappingFailed, - TorStrings.torConnect.noSettingsForCountry - ); + throw new TorConnectError(TorConnectError.NoSettingsForCountry); } } } @@ -585,17 +582,17 @@ class AutoBootstrappingState extends StateCallback { // We need to merge with old settings, in case the user is using a proxy // or is behind a firewall. await provider.writeSettings({ - ...TorSettings.getSettings(), + ...lazy.TorSettings.getSettings(), ...currentSetting, }); // Build out our bootstrap request. const bootstrap = new lazy.TorBootstrapRequest(); bootstrap.onbootstrapstatus = (progress, status) => { - TorConnect._updateBootstrapStatus(progress, status); + TorConnect._updateBootstrapProgress(progress, status); }; - bootstrap.onbootstraperror = (message, details) => { - lazy.logger.error(`Auto-Bootstrap error => ${message}; ${details}`); + bootstrap.onbootstraperror = error => { + lazy.logger.error("Auto-Bootstrap error", error); }; // Begin the bootstrap. @@ -620,14 +617,14 @@ class AutoBootstrappingState extends StateCallback { } if (success) { // Persist the current settings to preferences. - TorSettings.setSettings(currentSetting); - TorSettings.saveToPrefs(); + lazy.TorSettings.setSettings(currentSetting); + lazy.TorSettings.saveToPrefs(); // Do not await `applySettings`. Otherwise this opens up a window of // time where the user can still "Cancel" the bootstrap. // We are calling `applySettings` just to be on the safe side, but the // settings we are passing now should be exactly the same we already // passed earlier. - TorSettings.applySettings().catch(e => + lazy.TorSettings.applySettings().catch(e => lazy.logger.error("TorSettings.applySettings threw unexpectedly.", e) ); this.changeState(TorConnectState.Bootstrapped); @@ -638,19 +635,10 @@ class AutoBootstrappingState extends StateCallback { // Only explicitly change state here if something else has not transitioned // us. if (!this.transitioning) { - this.#throwError( - TorStrings.torConnect.autoBootstrappingFailed, - TorStrings.torConnect.autoBootstrappingAllFailed - ); + throw new TorConnectError(TorConnectError.AllSettingsFailed); } } - #throwError(message, details) { - let err = new Error(message); - err.details = details; - throw err; - } - transitionRequested() { this.#transitionResolve(); } @@ -662,7 +650,7 @@ class AutoBootstrappingState extends StateCallback { if (this.#changedSettings && nextState !== TorConnectState.Bootstrapped) { try { - await TorSettings.applySettings(); + await lazy.TorSettings.applySettings(); } catch (e) { // We cannot do much if the original settings were bad or // if the connection closed, so just report it in the @@ -698,17 +686,15 @@ class ErrorState extends StateCallback { ErrorState.#hasEverHappened = true; } - run(errorMessage, errorDetails) { - TorConnect._errorMessage = errorMessage; - TorConnect._errorDetails = errorDetails; - lazy.logger.error( - `Entering error state (${errorMessage}, ${errorDetails})` - ); + run(error) { + if (!(error instanceof TorConnectError)) { + error = new TorConnectError(TorConnectError.ExternalError, error); + } + TorConnect._errorCode = error.code; + TorConnect._errorDetails = error; + lazy.logger.error(`Entering error state (${error.code})`, error); - Services.obs.notifyObservers( - { message: errorMessage, details: errorDetails }, - TorConnectTopics.BootstrapError - ); + Services.obs.notifyObservers(error, TorConnectTopics.Error); this.changeState(TorConnectState.Configuring); } @@ -835,7 +821,6 @@ class InternetTest { export const TorConnect = { _stateHandler: new InitialState(), _bootstrapProgress: 0, - _bootstrapStatus: null, _internetStatus: InternetStatus.Unknown, // list of country codes Moat has settings for _countryCodes: [], @@ -851,7 +836,7 @@ export const TorConnect = { })() ), _detectedLocation: "", - _errorMessage: null, + _errorCode: null, _errorDetails: null, _logHasWarningOrError: false, _hasBootstrapEverFailed: false, @@ -925,17 +910,15 @@ export const TorConnect = { this._stateHandler.begin(...args); }, - _updateBootstrapStatus(progress, status) { + _updateBootstrapProgress(progress, status) { this._bootstrapProgress = progress; - this._bootstrapStatus = status; lazy.logger.info( - `Bootstrapping ${this._bootstrapProgress}% complete (${this._bootstrapStatus})` + `Bootstrapping ${this._bootstrapProgress}% complete (${status})` ); Services.obs.notifyObservers( { progress: TorConnect._bootstrapProgress, - status: TorConnect._bootstrapStatus, hasWarnings: TorConnect._logHasWarningOrError, }, TorConnectTopics.BootstrapProgress @@ -964,11 +947,13 @@ export const TorConnect = { // So, we prefer initializing TorConnect as soon as possible, so that // the UI will be able to detect it is in the Initializing state and act // consequently. - TorSettings.initializedPromise.then(() => this._settingsInitialized()); + lazy.TorSettings.initializedPromise.then(() => + this._settingsInitialized() + ); // register the Tor topics we always care about - observeTopic(TorTopics.ProcessExited); - observeTopic(TorTopics.LogHasWarnOrErr); + observeTopic(lazy.TorProviderTopics.ProcessExited); + observeTopic(lazy.TorProviderTopics.HasWarnOrErr); } }, @@ -976,11 +961,11 @@ export const TorConnect = { lazy.logger.debug(`Observed ${topic}`); switch (topic) { - case TorTopics.LogHasWarnOrErr: { + case lazy.TorProviderTopics.HasWarnOrErr: { this._logHasWarningOrError = true; break; } - case TorTopics.ProcessExited: { + case lazy.TorProviderTopics.ProcessExited: { // Treat a failure as a possibly broken configuration. // So, prevent quickstart at the next start. Services.prefs.setBoolPref(TorLauncherPrefs.prompt_at_startup, true); @@ -1048,7 +1033,7 @@ export const TorConnect = { // FIXME: This is called before the TorProvider is ready. // As a matter of fact, at the moment it is equivalent to the following // line, but this might become a problem in the future. - return TorLauncherUtil.shouldStartAndOwnTor; + return lazy.TorLauncherUtil.shouldStartAndOwnTor; }, get shouldShowTorConnect() { @@ -1089,7 +1074,7 @@ export const TorConnect = { get shouldQuickStart() { // quickstart must be enabled return ( - TorSettings.quickstart.enabled && + lazy.TorSettings.quickstart.enabled && // and the previous bootstrap attempt must have succeeded !Services.prefs.getBoolPref(TorLauncherPrefs.prompt_at_startup, true) ); @@ -1103,10 +1088,6 @@ export const TorConnect = { return this._bootstrapProgress; }, - get bootstrapStatus() { - return this._bootstrapStatus; - }, - get internetStatus() { return this._internetStatus; }, @@ -1123,8 +1104,8 @@ export const TorConnect = { return this._detectedLocation; }, - get errorMessage() { - return this._errorMessage; + get errorCode() { + return this._errorCode; }, get errorDetails() { @@ -1195,7 +1176,7 @@ export const TorConnect = { Further external commands and helper methods */ openTorPreferences() { - if (TorLauncherUtil.isAndroid) { + if (lazy.TorLauncherUtil.isAndroid) { lazy.EventDispatcher.instance.sendRequest({ type: "GeckoView:Tor:OpenSettings", }); ===================================== toolkit/modules/TorStrings.sys.mjs ===================================== @@ -51,11 +51,11 @@ class TorPropertyStringBundle { return `$(${key})`; } - getStrings(strings) { + getStrings(strings, additionalPrefix = "") { return Object.fromEntries( Object.entries(strings).map(([key, fallback]) => [ key, - this.getString(key, fallback), + this.getString(additionalPrefix + key, fallback), ]) ); } @@ -143,7 +143,7 @@ const Loader = { frequentLocations: "Frequently selected locations", otherLocations: "Other locations", - // TorConnect.jsm error messages + // TorConnect error messages offline: "Internet not reachable", autoBootstrappingFailed: "Automatic configuration failed", autoBootstrappingAllFailed: "None of the configurations we tried worked", @@ -158,11 +158,67 @@ const Loader = { titlebarStatusConnected: "Connected", }; + // Some strings were used through TorLauncherUtils. + // However, we need to use them in about:torconnect, which cannot access + // privileged code. + const bootstrapStatus = { + starting: "Starting", + conn_pt: "Connecting to bridge", + conn_done_pt: "Connected to bridge", + conn_proxy: "Connecting to proxy", + conn_done_proxy: "Connected to proxy", + conn: "Connecting to a Tor relay", + conn_done: "Connected to a Tor relay", + handshake: "Negotiating with a Tor relay", + handshake_done: "Finished negotiating with a Tor relay", + onehop_create: "Establishing an encrypted directory connection", + requesting_status: "Retrieving network status", + loading_status: "Loading network status", + loading_keys: "Loading authority certificates", + requesting_descriptors: "Requesting relay information", + loading_descriptors: "Loading relay information", + enough_dirinfo: "Finished loading relay information", + ap_conn_pt: "Building circuits: Connecting to bridge", + ap_conn_done_pt: "Building circuits: Connected to bridge", + ap_conn_proxy: "Building circuits: Connecting to proxy", + ap_conn_done_proxy: "Building circuits: Connected to proxy", + ap_conn: "Building circuits: Connecting to a Tor relay", + ap_conn_done: "Building circuits: Connected to a Tor relay", + ap_handshake: "Building circuits: Negotiating with a Tor relay", + ap_handshake_done: + "Building circuits: Finished negotiating with a Tor relay", + circuit_create: "Building circuits: Establishing a Tor circuit", + done: "Connected to the Tor network!", + }; + const bootstrapWarning = { + done: "done", + connectrefused: "connection refused", + misc: "miscellaneous", + resourcelimit: "insufficient resources", + identity: "identity mismatch", + timeout: "connection timeout", + noroute: "no route to host", + ioerror: "read/write error", + pt_missing: "missing pluggable transport", + }; + const tsb = new TorPropertyStringBundle( "chrome://torbutton/locale/torConnect.properties", "torConnect." ); - return tsb.getStrings(strings); + const tlsb = new TorPropertyStringBundle( + "chrome://torbutton/locale/torlauncher.properties", + "torlauncher." + ); + return { + ...tsb.getStrings(strings), + bootstrapFailedDetails: tlsb.getString( + "tor_bootstrap_failed_details", + "%1$S failed (%2$S)." + ), + bootstrapStatus: tlsb.getStrings(bootstrapStatus, "bootstrapStatus."), + bootstrapWarning: tlsb.getStrings(bootstrapWarning, "bootstrapWarning."), + }; }, /* ===================================== toolkit/torbutton/chrome/locale/en-US/torlauncher.properties ===================================== @@ -4,20 +4,15 @@ # file, You can obtain one at http://mozilla.org/MPL/2.0/. torlauncher.error_title=Tor Launcher - torlauncher.tor_exited_during_startup=Tor exited during startup. This might be due to an error in your torrc file, a bug in Tor or another program on your system, or faulty hardware. Until you fix the underlying problem and restart Tor, Tor Browser will not start. torlauncher.tor_exited=Tor unexpectedly exited. This might be due to a bug in Tor itself, another program on your system, or faulty hardware. Until you restart Tor, Tor Browser will not be able to reach any websites. If the problem persists, please send a copy of your Tor Log to the support team. torlauncher.tor_exited2=Restarting Tor will not close your browser tabs. torlauncher.restart_tor=Restart Tor -torlauncher.tor_bootstrap_failed=Tor failed to establish a Tor network connection. -torlauncher.tor_bootstrap_failed_details=%1$S failed (%2$S). -torlauncher.unable_to_start_tor=Unable to start Tor.\n\n%S -torlauncher.tor_missing=The Tor executable is missing. -torlauncher.torrc_missing=The torrc file is missing and could not be created. -torlauncher.datadir_missing=The Tor data directory does not exist and could not be created. -torlauncher.onionauthdir_missing=The Tor onion authentication directory does not exist and could not be created. -torlauncher.password_hash_missing=Failed to get hashed password. + +# Translation note: %1$S is a bootstrap phase from torlauncher.bootstrapStatus, +# %2$S is the error from torlauncher.bootstrapWarning +torlauncher.tor_bootstrap_failed_details=%1$S failed (%2$S). torlauncher.bootstrapStatus.starting=Starting torlauncher.bootstrapStatus.conn_pt=Connecting to bridge View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/compare/89a5a7… -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/compare/89a5a7… You're receiving this email because of your account on gitlab.torproject.org.

1 0

[Git][tpo/applications/firefox-android][firefox-android-115.2.1-13.5-1] 2 commits: fixup! Enable the connect assist experiments on alpha
by Pier Angelo Vendrame (＠pierov) 15 Apr '24

15 Apr '24

Pier Angelo Vendrame pushed to branch firefox-android-115.2.1-13.5-1 at The Tor Project / Applications / firefox-android Commits: 4a8875f4 by Pier Angelo Vendrame at 2024-04-09T20:09:45+02:00 fixup! Enable the connect assist experiments on alpha Bug 42479: Improve TorConnect error handling Updated the prototype of onBootstrapError. - - - - - c2db7c96 by Pier Angelo Vendrame at 2024-04-11T10:58:42+02:00 fixup! Add Tor integration and UI Bug 42479: Improve TorConnect error handling Updated the prototype of onBootstrapError. - - - - - 3 changed files: - fenix/app/src/main/java/org/mozilla/fenix/HomeActivity.kt - fenix/app/src/main/java/org/mozilla/fenix/tor/TorController.kt - fenix/app/src/main/java/org/mozilla/fenix/tor/TorControllerGV.kt Changes: ===================================== fenix/app/src/main/java/org/mozilla/fenix/HomeActivity.kt ===================================== @@ -1383,12 +1383,12 @@ open class HomeActivity : LocaleAwareAppCompatActivity(), NavHostActivity, TorIn } override fun onBootstrapStateChange(state: String) = Unit - override fun onBootstrapProgress(progress: Double, status: String, hasWarnings: Boolean) = Unit + override fun onBootstrapProgress(progress: Double, hasWarnings: Boolean) = Unit override fun onBootstrapComplete() { components.useCases.tabsUseCases.removeAllTabs() navHost.navController.navigate(NavGraphDirections.actionStartupHome()) } - override fun onBootstrapError(message: String, details: String) = Unit + override fun onBootstrapError(code: String?, message: String?, phase: String?, reason: String?) = Unit override fun onSettingsRequested() { navHost.navController.navigate(NavGraphDirections.actionGlobalSettingsFragment()) } ===================================== fenix/app/src/main/java/org/mozilla/fenix/tor/TorController.kt ===================================== @@ -14,7 +14,9 @@ interface TorEvents { } class TorError( var message: String, - var details: String + var details: String, + var phase: String, + var reason: String, ) { } interface TorLogs { ===================================== fenix/app/src/main/java/org/mozilla/fenix/tor/TorControllerGV.kt ===================================== @@ -301,8 +301,8 @@ class TorControllerGV( } // TorEventsBootstrapStateChangeListener - override fun onBootstrapProgress(progress: Double, status: String?, hasWarnings: Boolean) { - Log.d(TAG, "onBootstrapProgress($progress, $status, $hasWarnings)") + override fun onBootstrapProgress(progress: Double, hasWarnings: Boolean) { + Log.d(TAG, "onBootstrapProgress($progress, $hasWarnings)") if (progress == 100.0) { lastKnownStatus = TorConnectState.Bootstrapped wasTorBootstrapped = true @@ -312,7 +312,7 @@ class TorControllerGV( onTorConnecting() } - onTorStatusUpdate(status, lastKnownStatus.toTorStatus().status, progress) + onTorStatusUpdate("", lastKnownStatus.toTorStatus().status, progress) } // TorEventsBootstrapStateChangeListener @@ -322,8 +322,8 @@ class TorControllerGV( } // TorEventsBootstrapStateChangeListener - override fun onBootstrapError(message: String?, details: String?) { - lastKnownError = TorError(message ?: "", details ?: "") + override fun onBootstrapError(code: String?, message: String?, phase: String?, reason: String?) { + lastKnownError = TorError(code ?: "", message ?: "", phase ?: "", reason ?: "") onBootstrapStateChange(TorConnectState.Error.state) } View it on GitLab: https://gitlab.torproject.org/tpo/applications/firefox-android/-/compare/7e… -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/firefox-android/-/compare/7e… You're receiving this email because of your account on gitlab.torproject.org.

1 0