From c8893c32f3d197e96166bdc109f4da2febde5607 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Sebastian=20K=C3=BCrten?= <sebastian@mobanisto.de>
Date: Thu, 4 Nov 2021 15:39:25 +0100
Subject: [PATCH] Use updated briar master; use new style for defining tor
 ports

---
 briar                                         |  2 +-
 .../briar/desktop/DesktopModule.kt            | 22 ++++++++++---------
 .../briar/desktop/DesktopTestModule.kt        | 22 ++++++++++---------
 3 files changed, 25 insertions(+), 21 deletions(-)

diff --git a/briar b/briar
index bf091ef854..8a4fe7ca49 160000
--- a/briar
+++ b/briar
@@ -1 +1 @@
-Subproject commit bf091ef8547c25b5065eea9114931ebf88562c78
+Subproject commit 8a4fe7ca493285a136127f08e60303347d849cd2
diff --git a/src/main/kotlin/org/briarproject/briar/desktop/DesktopModule.kt b/src/main/kotlin/org/briarproject/briar/desktop/DesktopModule.kt
index 73c16fd05f..5d6a320667 100644
--- a/src/main/kotlin/org/briarproject/briar/desktop/DesktopModule.kt
+++ b/src/main/kotlin/org/briarproject/briar/desktop/DesktopModule.kt
@@ -9,15 +9,15 @@ import org.briarproject.bramble.api.db.DatabaseConfig
 import org.briarproject.bramble.api.plugin.PluginConfig
 import org.briarproject.bramble.api.plugin.TorConstants.DEFAULT_CONTROL_PORT
 import org.briarproject.bramble.api.plugin.TorConstants.DEFAULT_SOCKS_PORT
+import org.briarproject.bramble.api.plugin.TorControlPort
 import org.briarproject.bramble.api.plugin.TorDirectory
+import org.briarproject.bramble.api.plugin.TorSocksPort
 import org.briarproject.bramble.api.plugin.TransportId
 import org.briarproject.bramble.api.plugin.duplex.DuplexPluginFactory
 import org.briarproject.bramble.api.plugin.simplex.SimplexPluginFactory
 import org.briarproject.bramble.battery.DefaultBatteryManagerModule
 import org.briarproject.bramble.event.DefaultEventExecutorModule
 import org.briarproject.bramble.network.JavaNetworkModule
-import org.briarproject.bramble.plugin.TorPorts
-import org.briarproject.bramble.plugin.TorPortsImpl
 import org.briarproject.bramble.plugin.tor.CircumventionModule
 import org.briarproject.bramble.plugin.tor.UnixTorPluginFactory
 import org.briarproject.bramble.socks.SocksModule
@@ -68,18 +68,22 @@ internal class DesktopModule(
         return DesktopDatabaseConfig(dbDir, keyDir)
     }
 
-    @Provides
-    @Singleton
-    fun provideTorPorts(): TorPorts {
-        return TorPortsImpl(socksPort, controlPort)
-    }
-
     @Provides
     @TorDirectory
     internal fun provideTorDirectory(): File {
         return appDir.resolve("tor").toFile()
     }
 
+    @Provides
+    @Singleton
+    @TorSocksPort
+    internal fun provideTorSocksPort() = socksPort
+
+    @Provides
+    @Singleton
+    @TorControlPort
+    internal fun provideTorControlPort() = controlPort
+
     @Provides
     internal fun providePluginConfig(tor: UnixTorPluginFactory): PluginConfig {
         val duplex: List<DuplexPluginFactory> =
@@ -101,7 +105,5 @@ internal class DesktopModule(
         override fun shouldEnableImageAttachments() = false
         override fun shouldEnableProfilePictures() = false
         override fun shouldEnableDisappearingMessages() = false
-        override fun shouldEnableTransferData() = false
-        override fun shouldEnableShareAppViaOfflineHotspot() = false
     }
 }
diff --git a/src/test/kotlin/org/briarproject/briar/desktop/DesktopTestModule.kt b/src/test/kotlin/org/briarproject/briar/desktop/DesktopTestModule.kt
index 562d13bba9..d555e054aa 100644
--- a/src/test/kotlin/org/briarproject/briar/desktop/DesktopTestModule.kt
+++ b/src/test/kotlin/org/briarproject/briar/desktop/DesktopTestModule.kt
@@ -9,15 +9,15 @@ import org.briarproject.bramble.api.db.DatabaseConfig
 import org.briarproject.bramble.api.plugin.PluginConfig
 import org.briarproject.bramble.api.plugin.TorConstants.DEFAULT_CONTROL_PORT
 import org.briarproject.bramble.api.plugin.TorConstants.DEFAULT_SOCKS_PORT
+import org.briarproject.bramble.api.plugin.TorControlPort
 import org.briarproject.bramble.api.plugin.TorDirectory
+import org.briarproject.bramble.api.plugin.TorSocksPort
 import org.briarproject.bramble.api.plugin.TransportId
 import org.briarproject.bramble.api.plugin.duplex.DuplexPluginFactory
 import org.briarproject.bramble.api.plugin.simplex.SimplexPluginFactory
 import org.briarproject.bramble.battery.DefaultBatteryManagerModule
 import org.briarproject.bramble.event.DefaultEventExecutorModule
 import org.briarproject.bramble.network.JavaNetworkModule
-import org.briarproject.bramble.plugin.TorPorts
-import org.briarproject.bramble.plugin.TorPortsImpl
 import org.briarproject.bramble.plugin.tor.CircumventionModule
 import org.briarproject.bramble.plugin.tor.UnixTorPluginFactory
 import org.briarproject.bramble.socks.SocksModule
@@ -73,18 +73,22 @@ internal class DesktopTestModule(
         return DesktopDatabaseConfig(dbDir, keyDir)
     }
 
-    @Provides
-    @Singleton
-    fun provideTorPorts(): TorPorts {
-        return TorPortsImpl(socksPort, controlPort)
-    }
-
     @Provides
     @TorDirectory
     internal fun provideTorDirectory(): File {
         return appDir.resolve("tor").toFile()
     }
 
+    @Provides
+    @Singleton
+    @TorSocksPort
+    internal fun provideTorSocksPort() = socksPort
+
+    @Provides
+    @Singleton
+    @TorControlPort
+    internal fun provideTorControlPort() = controlPort
+
     @Provides
     internal fun providePluginConfig(tor: UnixTorPluginFactory): PluginConfig {
         val duplex: List<DuplexPluginFactory> =
@@ -106,8 +110,6 @@ internal class DesktopTestModule(
         override fun shouldEnableImageAttachments() = false
         override fun shouldEnableProfilePictures() = false
         override fun shouldEnableDisappearingMessages() = false
-        override fun shouldEnableTransferData() = false
-        override fun shouldEnableShareAppViaOfflineHotspot() = false
     }
 
     @Provides
-- 
GitLab