Add missing FirewallPolicyError class

mullvad · Nov 17, 2023 · efeac6d · efeac6d
1 parent 0a82036
commit efeac6d
Show file tree

Hide file tree

Showing 5 changed files with 44 additions and 11 deletions.
diff --git a/.../kotlin/net/mullvad/mullvadvpn/compose/component/notificationbanner/NotificationBanner.kt b/.../kotlin/net/mullvad/mullvadvpn/compose/component/notificationbanner/NotificationBanner.kt
@@ -37,6 +37,7 @@ import net.mullvad.mullvadvpn.ui.VersionInfo
 import net.mullvad.mullvadvpn.ui.notification.StatusLevel
 import net.mullvad.talpid.tunnel.ErrorState
 import net.mullvad.talpid.tunnel.ErrorStateCause
+import net.mullvad.talpid.tunnel.FirewallPolicyError
 import org.joda.time.DateTime
 
 @Preview
@@ -61,7 +62,13 @@ private fun PreviewNotificationBanner() {
                         InAppNotification.TunnelStateBlocked,
                         InAppNotification.NewDevice("Courageous Turtle"),
                         InAppNotification.TunnelStateError(
-                            error = ErrorState(ErrorStateCause.SetFirewallPolicyError, true)
+                            error =
+                                ErrorState(
+                                    ErrorStateCause.SetFirewallPolicyError(
+                                        FirewallPolicyError.Generic
+                                    ),
+                                    true
+                                )
                         )
                     )
                     .map { it.toNotificationData({}, {}, {}) }

diff --git a/android/lib/model/src/main/kotlin/net/mullvad/mullvadvpn/model/TunnelState.kt b/android/lib/model/src/main/kotlin/net/mullvad/mullvadvpn/model/TunnelState.kt
@@ -6,6 +6,7 @@ import net.mullvad.talpid.net.TunnelEndpoint
 import net.mullvad.talpid.tunnel.ActionAfterDisconnect
 import net.mullvad.talpid.tunnel.ErrorState
 import net.mullvad.talpid.tunnel.ErrorStateCause
+import net.mullvad.talpid.tunnel.FirewallPolicyError
 
 sealed class TunnelState : Parcelable {
     @Parcelize object Disconnected : TunnelState(), Parcelable
@@ -73,9 +74,20 @@ sealed class TunnelState : Parcelable {
                 DISCONNECTING -> Disconnecting(ActionAfterDisconnect.Nothing)
                 BLOCKING -> Error(ErrorState(ErrorStateCause.StartTunnelError, true))
                 ERROR -> {
-                    Error(ErrorState(ErrorStateCause.SetFirewallPolicyError, false))
+                    Error(
+                        ErrorState(
+                            ErrorStateCause.SetFirewallPolicyError(FirewallPolicyError.Generic),
+                            false
+                        )
+                    )
                 }
-                else -> Error(ErrorState(ErrorStateCause.SetFirewallPolicyError, false))
+                else ->
+                    Error(
+                        ErrorState(
+                            ErrorStateCause.SetFirewallPolicyError(FirewallPolicyError.Generic),
+                            false
+                        )
+                    )
             }
         }
     }

diff --git a/android/lib/talpid/src/main/kotlin/net/mullvad/talpid/tunnel/ErrorStateCause.kt b/android/lib/talpid/src/main/kotlin/net/mullvad/talpid/tunnel/ErrorStateCause.kt
@@ -14,19 +14,23 @@ sealed class ErrorStateCause : Parcelable {
         }
     }
 
-    @Parcelize object Ipv6Unavailable : ErrorStateCause()
+    @Parcelize data object Ipv6Unavailable : ErrorStateCause()
 
-    @Parcelize object SetFirewallPolicyError : ErrorStateCause()
+    @Parcelize
+    data class SetFirewallPolicyError(val firewallPolicyError: FirewallPolicyError) :
+        ErrorStateCause()
 
-    @Parcelize object SetDnsError : ErrorStateCause()
+    @Parcelize data object SetDnsError : ErrorStateCause()
 
-    @Parcelize class InvalidDnsServers(val addresses: ArrayList<InetAddress>) : ErrorStateCause()
+    @Parcelize
+    data class InvalidDnsServers(val addresses: ArrayList<InetAddress>) : ErrorStateCause()
 
-    @Parcelize object StartTunnelError : ErrorStateCause()
+    @Parcelize data object StartTunnelError : ErrorStateCause()
 
-    @Parcelize class TunnelParameterError(val error: ParameterGenerationError) : ErrorStateCause()
+    @Parcelize
+    data class TunnelParameterError(val error: ParameterGenerationError) : ErrorStateCause()
 
-    @Parcelize object IsOffline : ErrorStateCause()
+    @Parcelize data object IsOffline : ErrorStateCause()
 
-    @Parcelize object VpnPermissionDenied : ErrorStateCause()
+    @Parcelize data object VpnPermissionDenied : ErrorStateCause()
 }
diff --git a/android/lib/talpid/src/main/kotlin/net/mullvad/talpid/tunnel/FirewallPolicyError.kt b/android/lib/talpid/src/main/kotlin/net/mullvad/talpid/tunnel/FirewallPolicyError.kt
@@ -0,0 +1,9 @@
+package net.mullvad.talpid.tunnel
+
+import android.os.Parcelable
+import kotlinx.parcelize.Parcelize
+
+@Parcelize
+enum class FirewallPolicyError : Parcelable {
+    Generic
+}
diff --git a/mullvad-jni/src/classes.rs b/mullvad-jni/src/classes.rs
@@ -95,5 +95,6 @@ pub const CLASSES: &[&str] = &[
     "net/mullvad/talpid/CreateTunResult$InvalidDnsServers",
     "net/mullvad/talpid/CreateTunResult$PermissionDenied",
     "net/mullvad/talpid/CreateTunResult$TunnelDeviceError",
+    "net/mullvad/talpid/tunnel/FirewallPolicyError",
     "net/mullvad/talpid/TalpidVpnService",
 ];