| 
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:
Changes:
fenix/app/src/main/java/org/mozilla/fenix/HomeActivity.kt
 
| ... | ... | @@ -1383,12 +1383,12 @@ open class HomeActivity : LocaleAwareAppCompatActivity(), NavHostActivity, TorIn |  
| 1383 | 1383 |      }
 |  
| 1384 | 1384 |  
 |  
| 1385 | 1385 |      override fun onBootstrapStateChange(state: String) = Unit
 |  
| 1386 |  | -    override fun onBootstrapProgress(progress: Double, status: String, hasWarnings: Boolean) = Unit
 |  
|  | 1386 | +    override fun onBootstrapProgress(progress: Double, hasWarnings: Boolean) = Unit
 |  
| 1387 | 1387 |      override fun onBootstrapComplete() {
 |  
| 1388 | 1388 |          components.useCases.tabsUseCases.removeAllTabs()
 |  
| 1389 | 1389 |          navHost.navController.navigate(NavGraphDirections.actionStartupHome())
 |  
| 1390 | 1390 |      }
 |  
| 1391 |  | -    override fun onBootstrapError(message: String, details: String) = Unit
 |  
|  | 1391 | +    override fun onBootstrapError(code: String?, message: String?, phase: String?, reason: String?) = Unit
 |  
| 1392 | 1392 |      override fun onSettingsRequested() {
 |  
| 1393 | 1393 |          navHost.navController.navigate(NavGraphDirections.actionGlobalSettingsFragment())
 |  
| 1394 | 1394 |      }
 |  fenix/app/src/main/java/org/mozilla/fenix/tor/TorController.kt
 
 
| ... | ... | @@ -14,7 +14,9 @@ interface TorEvents { |  
| 14 | 14 |  }
 |  
| 15 | 15 |  class TorError(
 |  
| 16 | 16 |      var message: String,
 |  
| 17 |  | -    var details: String
 |  
|  | 17 | +    var details: String,
 |  
|  | 18 | +    var phase: String,
 |  
|  | 19 | +    var reason: String,
 |  
| 18 | 20 |  ) { }
 |  
| 19 | 21 |  
 |  
| 20 | 22 |  interface TorLogs {
 |  fenix/app/src/main/java/org/mozilla/fenix/tor/TorControllerGV.kt
 
 
| ... | ... | @@ -301,8 +301,8 @@ class TorControllerGV( |  
| 301 | 301 |      }
 |  
| 302 | 302 |  
 |  
| 303 | 303 |      // TorEventsBootstrapStateChangeListener
 |  
| 304 |  | -    override fun onBootstrapProgress(progress: Double, status: String?, hasWarnings: Boolean) {
 |  
| 305 |  | -        Log.d(TAG, "onBootstrapProgress($progress, $status, $hasWarnings)")
 |  
|  | 304 | +    override fun onBootstrapProgress(progress: Double, hasWarnings: Boolean) {
 |  
|  | 305 | +        Log.d(TAG, "onBootstrapProgress($progress, $hasWarnings)")
 |  
| 306 | 306 |          if (progress == 100.0) {
 |  
| 307 | 307 |              lastKnownStatus = TorConnectState.Bootstrapped
 |  
| 308 | 308 |              wasTorBootstrapped = true
 |  
| ... | ... | @@ -312,7 +312,7 @@ class TorControllerGV( |  
| 312 | 312 |              onTorConnecting()
 |  
| 313 | 313 |  
 |  
| 314 | 314 |          }
 |  
| 315 |  | -        onTorStatusUpdate(status, lastKnownStatus.toTorStatus().status, progress)
 |  
|  | 315 | +        onTorStatusUpdate("", lastKnownStatus.toTorStatus().status, progress)
 |  
| 316 | 316 |      }
 |  
| 317 | 317 |  
 |  
| 318 | 318 |      // TorEventsBootstrapStateChangeListener
 |  
| ... | ... | @@ -322,8 +322,8 @@ class TorControllerGV( |  
| 322 | 322 |      }
 |  
| 323 | 323 |  
 |  
| 324 | 324 |      // TorEventsBootstrapStateChangeListener
 |  
| 325 |  | -    override fun onBootstrapError(message: String?, details: String?) {
 |  
| 326 |  | -        lastKnownError = TorError(message ?: "", details ?: "")
 |  
|  | 325 | +    override fun onBootstrapError(code: String?, message: String?, phase: String?, reason: String?) {
 |  
|  | 326 | +        lastKnownError = TorError(code ?: "", message ?: "", phase ?: "", reason ?: "")
 |  
| 327 | 327 |          onBootstrapStateChange(TorConnectState.Error.state)
 |  
| 328 | 328 |      }
 |  
| 329 | 329 |  
 |  
 |