VPN-5680: Complete mobile onboarding without internet (#8328)

mozilla-mobile · Oct 19, 2023 · d913e70 · d913e70
1 parent f6bd991
commit d913e70
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/src/connectionmanager.cpp b/src/connectionmanager.cpp
@@ -938,8 +938,11 @@ bool ConnectionManager::activate(const ServerData& serverData,
 
               // Check if the error propagation has changed the Mozilla VPN
               // state. Continue only if the user is still authenticated and
-              // subscribed.
-              if (App::instance()->state() != App::StateMain) {
+              // subscribed. We can ignore this during onboarding because we are
+              // not actually turning the VPN on (only asking for VPN system
+              // config permissions)
+              if (App::instance()->state() != App::StateMain &&
+                  App::instance()->state() != App::StateOnboarding) {
                 return;
               }