Retry notifyVpnStart to allow value propagation (#3847)

Task/Issue URL: https://app.asana.com/0/488551667048375/1205911542931688/f ### Description See [asana](https://app.asana.com/0/488551667048375/1205911542931688/f) for better explanation ### Steps to test this PR Smoke test AppTP/VPN enable/disable and general functionality
duckduckgo · Nov 11, 2023 · 9b7fda5 · 9b7fda5
1 parent 7866d68
commit 9b7fda5
Showing 1 changed file with 11 additions and 3 deletions.
diff --git a/...impl/src/main/java/com/duckduckgo/mobile/android/vpn/service/TrackerBlockingVpnService.kt b/...impl/src/main/java/com/duckduckgo/mobile/android/vpn/service/TrackerBlockingVpnService.kt
@@ -610,9 +610,17 @@ class TrackerBlockingVpnService : VpnService(), CoroutineScope by MainScope(), V
      */
     private fun notifyVpnStart(): Boolean {
         val emptyNotification = VpnEnabledNotificationContentPlugin.VpnEnabledNotificationContent.EMPTY
-        val vpnNotification: VpnEnabledNotificationContentPlugin.VpnEnabledNotificationContent =
-            vpnEnabledNotificationContentPluginPoint.getHighestPriorityPlugin()?.getInitialContent()
-                ?: emptyNotification
+        var vpnNotification: VpnEnabledNotificationContentPlugin.VpnEnabledNotificationContent = emptyNotification
+        for (retries in 1..20) {
+            vpnNotification =
+                vpnEnabledNotificationContentPluginPoint.getHighestPriorityPlugin()?.getInitialContent()
+                    ?: emptyNotification
+
+            if (vpnNotification != emptyNotification) {
+                logcat { "Notification in retry: $retries" }
+                break
+            }
+        }
 
         startForeground(
             VPN_FOREGROUND_SERVICE_ID,