- tbb-commits - lists.torproject.org

[Git][tpo/applications/tor-browser][tor-browser-128.6.0esr-14.5-1] fixup! Add CI for Base Browser
by Pier Angelo Vendrame (＠pierov) 23 Jan '25

23 Jan '25

Pier Angelo Vendrame pushed to branch tor-browser-128.6.0esr-14.5-1 at The Tor Project / Applications / Tor Browser Commits: d8c660d5 by Beatriz Rizental at 2025-01-23T19:18:33+01:00 fixup! Add CI for Base Browser Attempt to download MR branch in case fetching fails due to the pipeline running in the parent project context. - - - - - 1 changed file: - .gitlab/ci/mixins.yml Changes: ===================================== .gitlab/ci/mixins.yml ===================================== @@ -12,7 +12,12 @@ exit 1 fi - echo "Fetching from remote branch ${CI_COMMIT_BRANCH:-$CI_MERGE_REQUEST_SOURCE_BRANCH_NAME}" - - git fetch origin "${CI_COMMIT_BRANCH:-$CI_MERGE_REQUEST_SOURCE_BRANCH_NAME}" + - | + if ! git fetch origin "${CI_COMMIT_BRANCH:-$CI_MERGE_REQUEST_SOURCE_BRANCH_NAME}"; then + echo -e "\e[31mFetching failed for branch ${CI_COMMIT_BRANCH:-$CI_MERGE_REQUEST_SOURCE_BRANCH_NAME} from $CI_REPOSITORY_URL.\e[0m" + echo "Attempting to fetch the merge request branch, assuming this pipeline is not running in a fork." + git fetch origin "merge-requests/${CI_MERGE_REQUEST_IID}/head" + fi - git checkout origin/${CI_COMMIT_BRANCH:-$CI_MERGE_REQUEST_SOURCE_BRANCH_NAME} .with-local-repo-pwsh: @@ -33,5 +38,10 @@ exit 1 } - Write-Output "Fetching from remote branch $branchName" - - git fetch origin $branchName + - | + if (! git fetch origin $branchName) { + Write-Output "Fetching failed for branch $branchName from $env:CI_REPOSITORY_URL." + Write-Output "Attempting to fetch the merge request branch, assuming this pipeline is not running in a fork." + git fetch origin "merge-requests/$env:CI_MERGE_REQUEST_IID/head" + } - git checkout origin/$branchName View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/commit/d8c660d… -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/commit/d8c660d… You're receiving this email because of your account on gitlab.torproject.org.

1 0

[Git][tpo/applications/tor-browser][tor-browser-128.6.0esr-14.5-1] 4 commits: fixup! TB 40933: Add tor-launcher functionality
by Pier Angelo Vendrame (＠pierov) 23 Jan '25

23 Jan '25

Pier Angelo Vendrame pushed to branch tor-browser-128.6.0esr-14.5-1 at The Tor Project / Applications / Tor Browser Commits: 90f60943 by clairehurst at 2025-01-23T18:07:07+00:00 fixup! TB 40933: Add tor-launcher functionality Bug 43222: Fix tor logs timestamps - - - - - a1c22a73 by clairehurst at 2025-01-23T18:07:07+00:00 fixup! TB 42247: Android helpers for the TorProvider Bug 43222: Fix tor logs timestamps - - - - - db1d2b30 by clairehurst at 2025-01-23T18:07:07+00:00 fixup! TB 41878: [android] Add standalone Tor Bootstrap Bug 43222: Fix tor logs timestamps - - - - - b270a085 by clairehurst at 2025-01-23T18:07:07+00:00 fixup! TB 40041 [android]: Implement Tor Network Settings Bug 43222: Fix tor logs timestamps - - - - - 8 changed files: - mobile/android/fenix/app/src/main/java/org/mozilla/fenix/tor/TorController.kt - mobile/android/fenix/app/src/main/java/org/mozilla/fenix/tor/TorControllerGV.kt - + mobile/android/fenix/app/src/main/java/org/mozilla/fenix/tor/TorLog.kt - mobile/android/fenix/app/src/main/java/org/mozilla/fenix/tor/TorLogsComposeFragment.kt - mobile/android/fenix/app/src/main/java/org/mozilla/fenix/tor/TorLogsViewModel.kt - mobile/android/geckoview/src/main/java/org/mozilla/geckoview/TorIntegrationAndroid.java - toolkit/components/tor-launcher/TorProvider.sys.mjs - toolkit/modules/TorAndroidIntegration.sys.mjs Changes: ===================================== mobile/android/fenix/app/src/main/java/org/mozilla/fenix/tor/TorController.kt ===================================== @@ -20,7 +20,7 @@ class TorError( ) { } interface TorLogs { - fun onLog(type: String?, message: String?) + fun onLog(type: String?, message: String?, timestamp: String?) } internal enum class TorStatus(val status: String) { @@ -32,7 +32,7 @@ internal enum class TorStatus(val status: String) { } interface TorController: TorEvents { - val logEntries: MutableList<Pair<String?, String?>> + val logEntries: MutableList<TorLog> val isStarting: Boolean val isRestarting: Boolean val isBootstrapped: Boolean ===================================== mobile/android/fenix/app/src/main/java/org/mozilla/fenix/tor/TorControllerGV.kt ===================================== @@ -66,7 +66,7 @@ class TorControllerGV( private var isTorBootstrapped = false get() = ((_lastKnownStatus.value.isStarted()) && wasTorBootstrapped) - private val entries = mutableListOf<Pair<String?, String?>>() + private val entries = mutableListOf<TorLog>() override val logEntries get() = entries override val isStarting get() = _lastKnownStatus.value.isStarting() override val isRestarting get() = isTorRestarting @@ -217,10 +217,10 @@ class TorControllerGV( } } - override fun onLog(type: String?, message: String?) { + override fun onLog(type: String?, message: String?, timestamp: String?) { synchronized(torLogListeners) { - entries.add(Pair(type, message)) - torLogListeners.toList().forEach { it.onLog(type, message) } + entries.add(TorLog(type ?: "null", message ?: "null", timestamp ?: "null")) + torLogListeners.toList().forEach { it.onLog(type ?: "null", message ?: "null", timestamp) } } } ===================================== mobile/android/fenix/app/src/main/java/org/mozilla/fenix/tor/TorLog.kt ===================================== @@ -0,0 +1,10 @@ +package org.mozilla.fenix.tor + +import androidx.compose.runtime.Stable + +@Stable +data class TorLog( + val type: String, + val text: String, + val timestamp: String, +) ===================================== mobile/android/fenix/app/src/main/java/org/mozilla/fenix/tor/TorLogsComposeFragment.kt ===================================== @@ -108,16 +108,14 @@ class TorLogsComposeFragment : Fragment() { bottom = 16.dp, ), ) { - DisableSelection { - Text( - text = log.timestamp.toString(), - color = PhotonColors.LightGrey40, - modifier = modifier - .padding(bottom = 4.dp), - ) - } Text( - text = log.text, + text = log.timestamp, + color = PhotonColors.LightGrey40, + modifier = modifier + .padding(bottom = 4.dp), + ) + Text( + text = "[${log.type}] " + log.text, color = PhotonColors.LightGrey05, ) } ===================================== mobile/android/fenix/app/src/main/java/org/mozilla/fenix/tor/TorLogsViewModel.kt ===================================== @@ -10,22 +10,18 @@ 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 androidx.lifecycle.LiveData import androidx.lifecycle.MutableLiveData 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 - private val _torLogs: MutableLiveData<List<TorLog>> = MutableLiveData( - mutableListOf(TorLog("---------------" + application.getString(R.string.tor_initializing_log) + "---------------")), - ) + private val _torLogs: MutableLiveData<List<TorLog>> = MutableLiveData(mutableListOf()) fun torLogs(): LiveData<List<TorLog>> { return _torLogs @@ -38,20 +34,14 @@ class TorLogsViewModel(application: Application) : AndroidViewModel(application) 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}")) } + val currentEntries = torController.logEntries for (log in currentEntries) { addLog(log) } } - override fun onLog(type: String?, message: String?) { - if (message == null || type == null) return - if (type == "ON" && type.startsWith("Circuit")) return - - addLog(TorLog("[$type] $message")) + override fun onLog(type: String?, message: String?, timestamp: String?) { + addLog(TorLog(type ?: "null", message ?: "null", timestamp ?: "null")) } override fun onCleared() { @@ -86,14 +76,8 @@ class TorLogsViewModel(application: Application) : AndroidViewModel(application) var ret = "" for (log in torLogs().value ?: return getApplication<Application>().getString(R.string.default_error_msg)) { - ret += log.text + '\n' + ret += "${log.timestamp} [${log.type}] ${log.text}\n" } return ret } } - -@Stable -data class TorLog( - val text: String, - val timestamp: Timestamp = Timestamp(System.currentTimeMillis()), -) ===================================== mobile/android/geckoview/src/main/java/org/mozilla/geckoview/TorIntegrationAndroid.java ===================================== @@ -169,8 +169,9 @@ public class TorIntegrationAndroid implements BundleEventListener { } else if (EVENT_TOR_LOGS.equals(event)) { String msg = message.getString("message"); String type = message.getString("logType"); + String timestamp = message.getString("timestamp"); for (TorLogListener listener : mLogListeners) { - listener.onLog(type, msg); + listener.onLog(type, msg, timestamp); } } } @@ -636,7 +637,7 @@ public class TorIntegrationAndroid implements BundleEventListener { } public interface TorLogListener { - void onLog(String logType, String message); + void onLog(String logType, String message, String timestamp); } private @NonNull void reloadSettings() { ===================================== toolkit/components/tor-launcher/TorProvider.sys.mjs ===================================== @@ -47,7 +47,7 @@ const logger = console.createInstance({ */ /** * @typedef {object} LogEntry An object with a log message - * @property {Date} date The date at which we received the message + * @property {string} timestamp The local date-time stamp at which we received the message * @property {string} type The message level * @property {string} msg The message */ @@ -518,13 +518,7 @@ export class TorProvider { */ getLog() { return this.#logs - .map(logObj => { - const timeStr = logObj.date - .toISOString() - .replace("T", " ") - .replace("Z", ""); - return `${timeStr} [${logObj.type}] ${logObj.msg}`; - }) + .map(logObj => `${logObj.timestamp} [${logObj.type}] ${logObj.msg}`) .join(TorLauncherUtil.isWindows ? "\r\n" : "\n"); } @@ -1031,9 +1025,16 @@ export class TorProvider { Services.obs.notifyObservers(null, TorProviderTopics.HasWarnOrErr); } - Services.obs.notifyObservers({ type, msg }, TorProviderTopics.TorLog); + const timestamp = new Date() + .toISOString() + .replace("T", " ") + .replace("Z", ""); + + Services.obs.notifyObservers( + { type, msg, timestamp }, + TorProviderTopics.TorLog + ); - const date = new Date(); const maxEntries = Services.prefs.getIntPref( Preferences.MaxLogEntries, 1000 @@ -1042,7 +1043,7 @@ export class TorProvider { this.#logs.splice(0, 1); } - this.#logs.push({ date, type, msg }); + this.#logs.push({ type, msg, timestamp }); switch (type) { case "ERR": logger.error(`[Tor error] ${msg}`); ===================================== toolkit/modules/TorAndroidIntegration.sys.mjs ===================================== @@ -136,6 +136,7 @@ class TorAndroidIntegrationImpl { type: EmittedEvents.torLogs, logType: subj.wrappedJSObject.type ?? "", message: subj.wrappedJSObject.msg ?? "", + timestamp: subj.wrappedJSObject.timestamp ?? "", }); break; case lazy.TorSettingsTopics.Ready: View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/compare/ed4f39… -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/compare/ed4f39… You're receiving this email because of your account on gitlab.torproject.org.

1 0

[Git][tpo/applications/tor-browser] Pushed new branch tor-browser-129.0a1-15.0-1
by Pier Angelo Vendrame (＠pierov) 23 Jan '25

23 Jan '25

Pier Angelo Vendrame pushed new branch tor-browser-129.0a1-15.0-1 at The Tor Project / Applications / Tor Browser -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/tree/tor-brows… You're receiving this email because of your account on gitlab.torproject.org.

1 0

[Git][tpo/applications/mullvad-browser][mullvad-browser-128.6.0esr-14.5-1] fixup! BB 42305: Add script to combine translation files across versions.
by Pier Angelo Vendrame (＠pierov) 22 Jan '25

22 Jan '25

Pier Angelo Vendrame pushed to branch mullvad-browser-128.6.0esr-14.5-1 at The Tor Project / Applications / Mullvad Browser Commits: 69adf7a7 by Henry Wilkes at 2025-01-22T18:27:21+01:00 fixup! BB 42305: Add script to combine translation files across versions. MB 324: Make the added comments also work for Mullvad Browser. - - - - - 1 changed file: - tools/base-browser/l10n/combine-translation-versions.py Changes: ===================================== tools/base-browser/l10n/combine-translation-versions.py ===================================== @@ -98,6 +98,10 @@ class BrowserBranch: self.name = branch_name self.prefix = version_match.group("prefix") self.browser_version = version_match.group("browser") + # Convert tor-browser to "Tor Browser", and similar. + browser_name = self.prefix.replace("-", " ").title() + self.browser_version_name = f"{browser_name} {self.browser_version}" + self._is_head = is_head self._ref = "HEAD" if is_head else f"origin/{branch_name}" @@ -306,7 +310,7 @@ for file_dict in json.loads(args.files): name, None if current_file is None else current_file.content, None if stable_file is None else stable_file.content, - f"Will be unused in Tor Browser {current_branch.browser_version}!", + f"Will be unused in {current_branch.browser_version_name}!", ) if legacy_branch and not file_dict.get("exclude-legacy", False): @@ -330,7 +334,7 @@ for file_dict in json.loads(args.files): name, content, legacy_file.content, - f"Unused in Tor Browser {stable_branch.browser_version}!", + f"Unused in {stable_branch.browser_version_name}!", ) elif legacy_branch: logger.info(f"Excluding legacy branch for {name}") View it on GitLab: https://gitlab.torproject.org/tpo/applications/mullvad-browser/-/commit/69a… -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/mullvad-browser/-/commit/69a… You're receiving this email because of your account on gitlab.torproject.org.

1 0

[Git][tpo/applications/tor-browser][base-browser-128.6.0esr-14.5-1] fixup! BB 42305: Add script to combine translation files across versions.
by Pier Angelo Vendrame (＠pierov) 22 Jan '25

22 Jan '25

Pier Angelo Vendrame pushed to branch base-browser-128.6.0esr-14.5-1 at The Tor Project / Applications / Tor Browser Commits: f1e7068c by Henry Wilkes at 2025-01-22T18:26:42+01:00 fixup! BB 42305: Add script to combine translation files across versions. MB 324: Make the added comments also work for Mullvad Browser. - - - - - 1 changed file: - tools/base-browser/l10n/combine-translation-versions.py Changes: ===================================== tools/base-browser/l10n/combine-translation-versions.py ===================================== @@ -98,6 +98,10 @@ class BrowserBranch: self.name = branch_name self.prefix = version_match.group("prefix") self.browser_version = version_match.group("browser") + # Convert tor-browser to "Tor Browser", and similar. + browser_name = self.prefix.replace("-", " ").title() + self.browser_version_name = f"{browser_name} {self.browser_version}" + self._is_head = is_head self._ref = "HEAD" if is_head else f"origin/{branch_name}" @@ -306,7 +310,7 @@ for file_dict in json.loads(args.files): name, None if current_file is None else current_file.content, None if stable_file is None else stable_file.content, - f"Will be unused in Tor Browser {current_branch.browser_version}!", + f"Will be unused in {current_branch.browser_version_name}!", ) if legacy_branch and not file_dict.get("exclude-legacy", False): @@ -330,7 +334,7 @@ for file_dict in json.loads(args.files): name, content, legacy_file.content, - f"Unused in Tor Browser {stable_branch.browser_version}!", + f"Unused in {stable_branch.browser_version_name}!", ) elif legacy_branch: logger.info(f"Excluding legacy branch for {name}") View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/commit/f1e7068… -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/commit/f1e7068… You're receiving this email because of your account on gitlab.torproject.org.

1 0

[Git][tpo/applications/tor-browser][tor-browser-128.6.0esr-14.5-1] fixup! BB 42305: Add script to combine translation files across versions.
by Pier Angelo Vendrame (＠pierov) 22 Jan '25

22 Jan '25

Pier Angelo Vendrame pushed to branch tor-browser-128.6.0esr-14.5-1 at The Tor Project / Applications / Tor Browser Commits: ed4f395b by Henry Wilkes at 2025-01-22T17:18:39+00:00 fixup! BB 42305: Add script to combine translation files across versions. MB 324: Make the added comments also work for Mullvad Browser. - - - - - 1 changed file: - tools/base-browser/l10n/combine-translation-versions.py Changes: ===================================== tools/base-browser/l10n/combine-translation-versions.py ===================================== @@ -98,6 +98,10 @@ class BrowserBranch: self.name = branch_name self.prefix = version_match.group("prefix") self.browser_version = version_match.group("browser") + # Convert tor-browser to "Tor Browser", and similar. + browser_name = self.prefix.replace("-", " ").title() + self.browser_version_name = f"{browser_name} {self.browser_version}" + self._is_head = is_head self._ref = "HEAD" if is_head else f"origin/{branch_name}" @@ -306,7 +310,7 @@ for file_dict in json.loads(args.files): name, None if current_file is None else current_file.content, None if stable_file is None else stable_file.content, - f"Will be unused in Tor Browser {current_branch.browser_version}!", + f"Will be unused in {current_branch.browser_version_name}!", ) if legacy_branch and not file_dict.get("exclude-legacy", False): @@ -330,7 +334,7 @@ for file_dict in json.loads(args.files): name, content, legacy_file.content, - f"Unused in Tor Browser {stable_branch.browser_version}!", + f"Unused in {stable_branch.browser_version_name}!", ) elif legacy_branch: logger.info(f"Excluding legacy branch for {name}") View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/commit/ed4f395… -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/commit/ed4f395… You're receiving this email because of your account on gitlab.torproject.org.

1 0

[Git][tpo/applications/mullvad-browser][mullvad-browser-128.6.0esr-14.5-1] fixup! Add CI for Mullvad Browser
by Pier Angelo Vendrame (＠pierov) 22 Jan '25

22 Jan '25

Pier Angelo Vendrame pushed to branch mullvad-browser-128.6.0esr-14.5-1 at The Tor Project / Applications / Mullvad Browser Commits: 98f5c475 by Henry Wilkes at 2025-01-22T16:26:59+00:00 fixup! Add CI for Mullvad Browser MB 324: Remove trailing comma from translation JSON. - - - - - 1 changed file: - .gitlab/ci/jobs/update-translations.yml Changes: ===================================== .gitlab/ci/jobs/update-translations.yml ===================================== @@ -54,7 +54,7 @@ "name": "mullvad-browser.ftl", "branch": "mullvad-browser", "directory": "toolkit/toolkit/global" - }, + } ]' View it on GitLab: https://gitlab.torproject.org/tpo/applications/mullvad-browser/-/commit/98f… -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/mullvad-browser/-/commit/98f… You're receiving this email because of your account on gitlab.torproject.org.

1 0

[Git][tpo/applications/tor-browser] Pushed new branch tor-browser-135.0a1-15.0-1
by morgan (＠morgan) 22 Jan '25

22 Jan '25

morgan pushed new branch tor-browser-135.0a1-15.0-1 at The Tor Project / Applications / Tor Browser -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/tree/tor-brows… You're receiving this email because of your account on gitlab.torproject.org.

1 0

[Git][tpo/applications/tor-browser] Pushed new branch tor-browser-134.0a1-15.0-1
by morgan (＠morgan) 22 Jan '25

22 Jan '25

morgan pushed new branch tor-browser-134.0a1-15.0-1 at The Tor Project / Applications / Tor Browser -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/tree/tor-brows… You're receiving this email because of your account on gitlab.torproject.org.

1 0

[Git][tpo/applications/tor-browser] Pushed new branch tor-browser-133.0a1-15.0-1
by morgan (＠morgan) 22 Jan '25

22 Jan '25

morgan pushed new branch tor-browser-133.0a1-15.0-1 at The Tor Project / Applications / Tor Browser -- View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/tree/tor-brows… You're receiving this email because of your account on gitlab.torproject.org.

1 0