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

[tor-commits] [tor-browser-build/master] Bug 40336: Delete snowflake config option



commit 098fc5e1057a2eedc0297e0bd47e83992d2da62d
Author: Nicolas Vigier <boklm@xxxxxxxxxxxxxx>
Date:   Thu Sep 2 12:20:02 2021 +0200

    Bug 40336: Delete snowflake config option
    
    Since snowflake is now enabled everywhere, we don't need the config
    option anymore.
---
 ...01-Revert-Bug-30318-Add-snowflake-support.patch | 78 ----------------------
 projects/tor-android-service/build                 |  3 -
 projects/tor-android-service/config                |  2 -
 projects/tor-browser/build                         | 11 +--
 projects/tor-browser/config                        |  2 +-
 projects/tor-onion-proxy-library/build             | 28 +++-----
 projects/tor-onion-proxy-library/config            |  5 --
 rbm.conf                                           | 12 ----
 8 files changed, 11 insertions(+), 130 deletions(-)

diff --git a/projects/tor-android-service/0001-Revert-Bug-30318-Add-snowflake-support.patch b/projects/tor-android-service/0001-Revert-Bug-30318-Add-snowflake-support.patch
deleted file mode 100644
index b7be2ae..0000000
--- a/projects/tor-android-service/0001-Revert-Bug-30318-Add-snowflake-support.patch
+++ /dev/null
@@ -1,78 +0,0 @@
-From f3cf622e537a1283b821b3f3ce8e396a01bcb84b Mon Sep 17 00:00:00 2001
-From: Georg Koppen <gk@xxxxxxxxxxxxxx>
-Date: Sat, 16 Jan 2021 20:56:30 +0000
-Subject: [PATCH] Revert "Bug 30318: Add snowflake support"
-
-This reverts commit ecc251d0a73f7e2034a271efd28036a0108b8688.
-
-diff --git a/service/src/main/assets/common/bridges.txt b/service/src/main/assets/common/bridges.txt
-index 1525eba..fe37735 100644
---- a/service/src/main/assets/common/bridges.txt
-+++ b/service/src/main/assets/common/bridges.txt
-@@ -13,5 +13,3 @@ obfs4 45.145.95.6:27015 C5B7CD6946FF10C5B3E89691A7D3F2C122D2117C cert=TD7PbUO0/0
- obfs4 [2a0c:4d80:42:702::1]:27015 C5B7CD6946FF10C5B3E89691A7D3F2C122D2117C cert=TD7PbUO0/0k6xYHMPW3vJxICfkMZNdkRrb63Zhl5j9dW3iRGiCx0A7mPhe5T2EDzQ35+Zw iat-mode=0
- obfs4 51.222.13.177:80 5EDAC3B810E12B01F6FD8050D2FD3E277B289A08 cert=2uplIpLQ0q9+0qMFrK5pkaYRDOe460LL9WHBvatgkuRr/SL31wBOEupaMMJ6koRE6Ld0ew iat-mode=0
- meek_lite 0.0.2.0:2 97700DFE9F483596DDA6264C4D7DF7641E1E39CE url=https://meek.azureedge.net/ front=ajax.aspnetcdn.com
--
--snowflake 0.0.3.0:1 2B280B23E1107BB62ABFC40DDCC8824814F80A72
-diff --git a/service/src/main/java/org/torproject/android/service/CustomTorInstaller.java b/service/src/main/java/org/torproject/android/service/CustomTorInstaller.java
-index 52bc4fe..198c981 100644
---- a/service/src/main/java/org/torproject/android/service/CustomTorInstaller.java
-+++ b/service/src/main/java/org/torproject/android/service/CustomTorInstaller.java
-@@ -40,17 +40,14 @@ public class CustomTorInstaller extends TorInstaller {
-     @Override
-     public InputStream openBridgesStream() throws IOException {
-         /*
--            BridgesList is an overloaded field, which can cause some confusion.
--            The list can be:
--              1) a filter like obfs4, meek, or snowflake OR
--              2) it can be a custom bridge
--            For (1), we just pass back all bridges, the filter will occur
--              elsewhere in the library.
--            For (2) we return the bridge list as a raw stream.
--            If length is greater than 9, then we know this is a custom bridge
-+            BridgesList is an overloaded field, which can cause some confusion. The list can be:
-+            1) a filter like obfs4 or meek OR 2) it can be a custom bridge
-+            For (1), we just pass back all bridges, the filter will occur elsewhere in the library.
-+            For (2) we return the bridge list as a raw stream
-+            If length is greater than 5, then we know this is a custom bridge
-          */
-         String userDefinedBridgeList = Prefs.getBridgesList();
--        byte bridgeType = (byte) (userDefinedBridgeList.length() > 9 ? 1 : 0);
-+        byte bridgeType = (byte) (userDefinedBridgeList.length() > 5 ? 1 : 0);
-         // Terrible hack. Must keep in sync with topl::addBridgesFromResources.
-         if (bridgeType == 0) {
-             switch (userDefinedBridgeList) {
-@@ -60,9 +57,6 @@ public class CustomTorInstaller extends TorInstaller {
-                 case "meek":
-                     bridgeType = 3;
-                     break;
--                case "snowflake":
--                    bridgeType = 4;
--                    break;
-             }
-         }
- 
-diff --git a/service/src/main/java/org/torproject/android/service/TorService.java b/service/src/main/java/org/torproject/android/service/TorService.java
-index f1ddcda..32d7f30 100644
---- a/service/src/main/java/org/torproject/android/service/TorService.java
-+++ b/service/src/main/java/org/torproject/android/service/TorService.java
-@@ -365,12 +365,10 @@ public final class TorService extends Service implements TorServiceConstants, Or
- 
-             //Check bridges to see if we need this
-             File nativeDir = new File(getApplicationInfo().nativeLibraryDir);
--            File pluggableTransportObfs = new File(nativeDir, "libObfs4proxy.so");
--            if(!pluggableTransportObfs.canExecute()) pluggableTransportObfs.setExecutable(true);
--            File pluggableTransportSnow = new File(nativeDir, "libSnowflake.so");
--            if(!pluggableTransportSnow.canExecute()) pluggableTransportSnow.setExecutable(true);
-+            File pluggableTransport = new File(nativeDir, "libObfs4proxy.so");
-+            if(!pluggableTransport.canExecute()) pluggableTransport.setExecutable(true);
- 
--            builder.configurePluggableTransportsFromSettings(pluggableTransportObfs, pluggableTransportSnow);
-+            builder.configurePluggableTransportsFromSettings(pluggableTransport);
-             mDataService.updateConfigBuilder(builder);
-             onionProxyManager.getTorInstaller().updateTorConfigCustom
-                     (builder.asString());
--- 
-2.30.0
-
diff --git a/projects/tor-android-service/build b/projects/tor-android-service/build
index 68d63cd..797d5c2 100644
--- a/projects/tor-android-service/build
+++ b/projects/tor-android-service/build
@@ -14,9 +14,6 @@ tar -C $GRADLE_HOME -xf $rootdir/[% c('input_files_by_name/gradle') %]
 tar -C /var/tmp/build -xf [% project %]-[% c('version') %].tar.gz
 
 cd /var/tmp/build/[% project %]-[% c('version') %]
-[% IF ! c("var/snowflake") %]
-  patch -p1 < $rootdir/0001-Revert-Bug-30318-Add-snowflake-support.patch
-[% END %]
 # Replace pre-packaged tor library with the latest build
 rm -fR service/src/main/jniLibs/*
 [% FOREACH arch = ['armv7', 'aarch64', 'x86', 'x86_64'] -%]
diff --git a/projects/tor-android-service/config b/projects/tor-android-service/config
index 171f4a9..c1ccb7d 100644
--- a/projects/tor-android-service/config
+++ b/projects/tor-android-service/config
@@ -25,8 +25,6 @@ input_files:
     exec: '[% INCLUDE "fetch-gradle-dependencies" %]'
   - project: tor-onion-proxy-library
     name: topl
-  - filename: 0001-Revert-Bug-30318-Add-snowflake-support.patch
-    enable: '[% ! c("var/snowflake") %]'
   - project: tor
     name: tor-armv7
     target_prepend:
diff --git a/projects/tor-browser/build b/projects/tor-browser/build
index 1d2e35b..d2b8482 100644
--- a/projects/tor-browser/build
+++ b/projects/tor-browser/build
@@ -48,9 +48,7 @@ MARTOOLS=$rootdir/mar-tools
 mv [% c('input_files_by_name/noscript') %] "$TBDIR/$EXTSPATH/{73a6fe31-595d-460b-a920-fcc0f8843232}.xpi"
 
 tar -C "$TBDIR" -xf [% c('input_files_by_name/obfs4') %]
-[% IF c("var/snowflake") %]
-  tar -C "$TBDIR" -xf [% c('input_files_by_name/snowflake') -%]
-[% END -%]
+tar -C "$TBDIR" -xf [% c('input_files_by_name/snowflake') -%]
 
 tar -C "$TBDIR[% IF c("var/osx") %]/Contents/Resources[% END %]" -xf [% c('input_files_by_name/fonts') %]
 
@@ -135,12 +133,7 @@ cp -a Bundle-Data/Docs/* "$TBDIR/$DOCSPATH"
 tar -C Bundle-Data/[% bundledata_osname %] [% IF ! c("var/namecoin") %]--exclude=*Electrum-NMC* --exclude=*ncprop279*[% END %] -c . | tar -C "$TBDIR[% IF ! c("var/osx") %]/TorBrowser[% END %]" -x
 
 cat Bundle-Data/PTConfigs/[% bundledata_osname %]/torrc-defaults-appendix >> "$TBDIR/$TORCONFIGPATH/torrc-defaults"
-[% IF ! c("var/snowflake") %]
-  grep -v 'default_bridge\.snowflake' Bundle-Data/PTConfigs/bridge_prefs.js \
-    >> "$GENERATEDPREFSPATH"
-[% ELSE %]
-  cat Bundle-Data/PTConfigs/bridge_prefs.js >> "$GENERATEDPREFSPATH"
-[% END %]
+cat Bundle-Data/PTConfigs/bridge_prefs.js >> "$GENERATEDPREFSPATH"
 
 [% IF ! c("var/multi_lingual") %]
   echo 'pref("extensions.torlauncher.prompt_for_locale", false);' >> "$GENERATEDPREFSPATH"
diff --git a/projects/tor-browser/config b/projects/tor-browser/config
index 09f77b2..4091361 100644
--- a/projects/tor-browser/config
+++ b/projects/tor-browser/config
@@ -75,7 +75,7 @@ input_files:
     enable: '[% ! c("var/android") %]'
   - project: snowflake
     name: snowflake
-    enable: '[% c("var/snowflake") && ! c("var/android") %]'
+    enable: '[% ! c("var/android") %]'
   - filename: Bundle-Data
     enable: '[% ! c("var/android") %]'
   - URL: https://addons.cdn.mozilla.net/user-media/addons/722/noscript_security_suite-11.2.11-an+fx.xpi
diff --git a/projects/tor-onion-proxy-library/build b/projects/tor-onion-proxy-library/build
index d4950f6..452d7e4 100644
--- a/projects/tor-onion-proxy-library/build
+++ b/projects/tor-onion-proxy-library/build
@@ -21,42 +21,30 @@ cd /var/tmp/build/[% project %]-[% c('version') %]
 # build will pull down the correct android tool versions
 patch -p1 < $rootdir/gradle.patch
 patch -p1 < $rootdir/0001-Bug-33931-Filter-bridges-in-stream-by-type.patch
-[% IF c("var/snowflake") %]
-  patch -p1 < $rootdir/0001-Bug-30318-Add-snowflake-support.patch
-[% END %]
+patch -p1 < $rootdir/0001-Bug-30318-Add-snowflake-support.patch
 
 [% FOREACH arch = ['armv7', 'aarch64', 'x86', 'x86_64'] -%]
   # Extract obfs4proxy from TorBrowser/Tor/PluggableTransports/obfs4proxy
   tar --strip-components=4 -xf $rootdir/[% c('input_files_by_name/obfs4-' _ arch) %]
 
-  [% IF c("var/snowflake") %]
-    # Extract snowflake from TorBrowser/Tor/PluggableTransports/snowflake
-    tar --strip-components=4 -xf $rootdir/[% c('input_files_by_name/snowflake-' _ arch) %]
-  [% END %]
+  # Extract snowflake from TorBrowser/Tor/PluggableTransports/snowflake
+  tar --strip-components=4 -xf $rootdir/[% c('input_files_by_name/snowflake-' _ arch) %]
   # Overwrite the obfs4proxy binary provided by Pluto and add Snowflake
   [% IF arch == "armv7" -%]
     cp obfs4proxy external/pluto/bin/armeabi-v7a/
     cp obfs4proxy external/pluto/bin/armeabi/
-    [% IF c("var/snowflake") %]
-      cp snowflake-client external/pluto/bin/armeabi-v7a/
-      cp snowflake-client external/pluto/bin/armeabi/
-    [% END %]
+    cp snowflake-client external/pluto/bin/armeabi-v7a/
+    cp snowflake-client external/pluto/bin/armeabi/
   [% ELSIF arch == "aarch64" -%]
     cp obfs4proxy external/pluto/bin/arm64-v8a/
-    [% IF c("var/snowflake") %]
-      cp snowflake-client external/pluto/bin/arm64-v8a/
-    [% END %]
+    cp snowflake-client external/pluto/bin/arm64-v8a/
   [% ELSE -%]
     cp obfs4proxy external/pluto/bin/[% arch %]/
-    [% IF c("var/snowflake") %]
-      cp snowflake-client external/pluto/bin/[% arch %]/
-    [% END %]
+    cp snowflake-client external/pluto/bin/[% arch %]/
   [% END -%]
 
   rm obfs4proxy
-  [% IF c("var/snowflake") %]
-    rm snowflake-client
-  [% END %]
+  rm snowflake-client
 [% END -%]
 
 # Build Android Libraries and Apps
diff --git a/projects/tor-onion-proxy-library/config b/projects/tor-onion-proxy-library/config
index f4a5c24..09f1bc7 100644
--- a/projects/tor-onion-proxy-library/config
+++ b/projects/tor-onion-proxy-library/config
@@ -42,26 +42,21 @@ input_files:
     name: snowflake-armv7
     target_prepend:
       - android-armv7
-    enable: '[% c("var/snowflake") %]'
   - project: snowflake
     name: snowflake-aarch64
     target_prepend:
       - android-aarch64
-    enable: '[% c("var/snowflake") %]'
   - project: snowflake
     name: snowflake-x86
     target_prepend:
       - android-x86
-    enable: '[% c("var/snowflake") %]'
   - project: snowflake
     name: snowflake-x86_64
     target_prepend:
       - android-x86_64
-    enable: '[% c("var/snowflake") %]'
   - filename: 'gradle-dependencies-[% c("var/gradle_dependencies_version") %]'
     name: gradle-dependencies
     exec: '[% INCLUDE "fetch-gradle-dependencies" %]'
   - filename: gradle.patch
   - filename: 0001-Bug-33931-Filter-bridges-in-stream-by-type.patch
   - filename: 0001-Bug-30318-Add-snowflake-support.patch
-    enable: '[% c("var/snowflake") %]'
diff --git a/rbm.conf b/rbm.conf
index d231de1..63105ad 100644
--- a/rbm.conf
+++ b/rbm.conf
@@ -335,9 +335,6 @@ targets:
       android_min_api_aarch64: 21
       # This is needed to get the offline build part for Glean right.
       glean_parser: 1.29.0
-      # We only build snowflake on the alpha and nightly
-      # channels for now.
-      snowflake: '[% c("var/alpha") || c("var/nightly") %]'
       container:
         suite: buster
         arch: amd64
@@ -394,9 +391,6 @@ targets:
       linux: 1
       compiler: gcc
       configure_opt: '[% c("var/configure_opt_project") %]'
-      # We only build snowflake on the alpha and nightly
-      # channels for now.
-      snowflake: '[% c("var/alpha") || c("var/nightly") %]'
       # Only build Namecoin for linux on nightly
       namecoin: '[% c("var/nightly") %]'
       container:
@@ -456,9 +450,6 @@ targets:
       flag_mwindows: '-mwindows'
       compiler: mingw-w64
       faketime_path: /usr/lib/x86_64-linux-gnu/faketime/libfaketime.so.1
-      # We only build snowflake on the alpha and nightly
-      # channels for now.
-      snowflake: '[% c("var/alpha") || c("var/nightly") %]'
       deps:
         - build-essential
         - python
@@ -486,9 +477,6 @@ targets:
       locale_ja: ja-JP-mac
       # We only support RLBox on the nightly channel for now
       rlbox: '[% c("var/nightly") %]'
-      # We only build snowflake on the alpha and nightly
-      # channels for now.
-      snowflake: '[% c("var/alpha") || c("var/nightly") %]'
       deps:
         - build-essential
         - python



_______________________________________________
tor-commits mailing list
tor-commits@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits