[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

[tor-commits] [builders/tor-browser-build] branch main updated: Bug 40631: Stop bundling HTTPS Everywhere on Android (see tor-browser#41160)



This is an automated email from the git hooks/post-receive script.

richard pushed a commit to branch main
in repository builders/tor-browser-build.

The following commit(s) were added to refs/heads/main by this push:
     new af97862  Bug 40631: Stop bundling HTTPS Everywhere on Android (see tor-browser#41160)
af97862 is described below

commit af97862c4b857415eade803c6a417474297fc54b
Author: Giorgio Maone <ma1@xxxxxxxxxxxxxx>
AuthorDate: Thu Sep 29 21:55:02 2022 +0000

    Bug 40631: Stop bundling HTTPS Everywhere on Android (see tor-browser#41160)
---
 projects/browser/build.android            | 16 +++++++-------
 projects/browser/verify_allowed_addons.py |  3 ---
 projects/https-everywhere/build           |  8 -------
 projects/https-everywhere/config          | 36 -------------------------------
 tools/fetch_allowed_addons.py             |  6 ------
 5 files changed, 8 insertions(+), 61 deletions(-)

diff --git a/projects/browser/build.android b/projects/browser/build.android
index a8c5a08..08a4046 100644
--- a/projects/browser/build.android
+++ b/projects/browser/build.android
@@ -10,19 +10,19 @@ test_in_apk=$rootdir/fenix/app-*-androidTest.apk
 
 tar xavf $rootdir/[% c('input_files_by_name/fenix') %]
 
-# Bundle our extensions. We treat HTTPS-Everywhere and NoScript differently as
-# the former should be an internal extension without getting any updates besides
-# from Tor Browser releases. NoScript will be copied over to the profile folder
-# being a "regular" browser extension.
+# Bundle our extensioni(s).
+# NoScript will be copied over to the profile folder
+# as a "regular" browser extension receiving regular AMO updates.
+noscript_path="$ext_dir/{73a6fe31-595d-460b-a920-fcc0f8843232}.xpi"
+
 mkdir -p /var/tmp/build/$ext_dir [% dest_dir _ '/' _ c('filename') %]
 cd /var/tmp/build
-unzip -d $ext_dir/https-everywhere $rootdir/[% c('input_files_by_name/https-everywhere') %]
-mv $rootdir/[% c('input_files_by_name/noscript') %] $ext_dir/{73a6fe31-595d-460b-a920-fcc0f8843232}.xpi
+mv $rootdir/[% c('input_files_by_name/noscript') %] "$noscript_path"
 
 [% IF c("var/verify_allowed_addons") %]
-  # Check that allowed_addons.json contains the right versions of NoScript and HTTPS Everywhere
+  # Check that allowed_addons.json contains the right versions of our bundled extension(s).
   # If so, replace the default allowed_addons.json by ours in the apk assets folder.
-  $rootdir/verify_allowed_addons.py "$rootdir/allowed_addons.json" "$ext_dir/{73a6fe31-595d-460b-a920-fcc0f8843232}.xpi" "$rootdir/[% c('input_files_by_name/https-everywhere') %]"
+  $rootdir/verify_allowed_addons.py "$rootdir/allowed_addons.json" "$noscript_path"
 [% END %]
 
 mv $rootdir/allowed_addons.json $assets_dir/allowed_addons.json
diff --git a/projects/browser/verify_allowed_addons.py b/projects/browser/verify_allowed_addons.py
index b0e89a5..c78e998 100755
--- a/projects/browser/verify_allowed_addons.py
+++ b/projects/browser/verify_allowed_addons.py
@@ -31,7 +31,6 @@ def read_extension_manifest(path):
 def main(argv):
   allowed_addons_path = argv[0]
   noscript_path = argv[1]
-  https_everywhere_path = argv[2]
 
   addons = None
   with open(allowed_addons_path, 'r') as file:
@@ -42,11 +41,9 @@ def main(argv):
     noscript_hash = "sha256:" + hashlib.sha256(file.read()).hexdigest()
 
   noscript_version = read_extension_manifest(noscript_path)["version"]
-  https_everywhere_version = read_extension_manifest(https_everywhere_path)["version"]
 
   verify_extension_hash(addons, '{73a6fe31-595d-460b-a920-fcc0f8843232}', noscript_hash)
   verify_extension_version(addons, '{73a6fe31-595d-460b-a920-fcc0f8843232}', noscript_version)
-  verify_extension_version(addons, 'https-everywhere-eff@xxxxxxx', https_everywhere_version)
 
 if __name__ == "__main__":
    main(sys.argv[1:])
diff --git a/projects/https-everywhere/build b/projects/https-everywhere/build
deleted file mode 100644
index 10dd4bf..0000000
--- a/projects/https-everywhere/build
+++ /dev/null
@@ -1,8 +0,0 @@
-#!/bin/bash
-[% c("var/set_default_env") -%]
-tar xf [% project %]-[% c('version') %].tar.gz
-cd [% project %]-[% c('version') %]
-./make.sh
-# Since 5.0.2 a .xpi for AMO is built, too. We don't need it.
-rm pkg/*-amo.xpi
-mv pkg/*.xpi [% dest_dir _ '/' _ c('filename') %]
diff --git a/projects/https-everywhere/config b/projects/https-everywhere/config
deleted file mode 100644
index e980cf3..0000000
--- a/projects/https-everywhere/config
+++ /dev/null
@@ -1,36 +0,0 @@
-# vim: filetype=yaml sw=2
-version: 2021.7.13
-git_url: https://github.com/EFForg/https-everywhere.git
-git_hash: '[% c("version") %]'
-git_submodule: 1
-gpg_keyring: https-everywhere.gpg
-tag_gpg_id: 1
-filename: "[% project %]-[% c('version') %]-[% c('var/build_id') %].xpi"
-container:
-  use_container: 1
-var:
-  # HTTPS Everywhere is expected to be the same on all platforms. To avoid
-  # building the same thing 4 times, using 4 different container images
-  # (each one with a different suite or architecture), we set the container
-  # to bullseye/amd64 for all platforms. This allows us to create only one
-  # container image, and also build the extension only one time as the
-  # filename does not contain the platform, and var/build_id should be
-  # the same since there is now nothing platform specific in the build
-  # inputs. This allows us to save a little time and disk space.
-  container:
-    suite: bullseye
-    arch: amd64
-  deps:
-    - git
-    - libxslt1.1
-    - libxml2-utils
-    - sqlite3
-    - rsync
-    - zip
-    - unzip
-    - python3
-    - python3-distutils
-  pre_pkginst: ''
-
-input_files:
-  - project: container-image
diff --git a/tools/fetch_allowed_addons.py b/tools/fetch_allowed_addons.py
index ff82537..15d3d7f 100755
--- a/tools/fetch_allowed_addons.py
+++ b/tools/fetch_allowed_addons.py
@@ -24,17 +24,11 @@ def fetch_and_embed_icons(addons):
     icon_data = fetch(addon['icon_url'])
     addon['icon_url'] = 'data:image/png;base64,' + str(base64.b64encode(icon_data), 'utf8')
 
-def patch_https_everywhere(addons):
-  addon = find_addon(addons, 'https-everywhere@xxxxxxx')
-  addon['guid'] = 'https-everywhere-eff@xxxxxxx'
-  addon['url'] = 'https://www.eff.org/https-everywhere'
-
 def main(argv):
   amo_collection = argv[0] if argv else '83a9cccfe6e24a34bd7b155ff9ee32'
   url = 'https://services.addons.mozilla.org/api/v4/accounts/account/mozilla/collections/' + amo_collection + '/addons/'
   data = json.loads(fetch(url))
   fetch_and_embed_icons(data)
-  patch_https_everywhere(data)
   data['results'].sort(key=lambda x: x['addon']['guid'])
   find_addon(data, '{73a6fe31-595d-460b-a920-fcc0f8843232}') # Check that NoScript is present
   print(json.dumps(data, indent=2, ensure_ascii=False))

-- 
To stop receiving notification emails like this one, please contact
the administrator of this repository.
_______________________________________________
tor-commits mailing list
tor-commits@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits