Fixes #412 - Not able to add secure internet servers

eduvpn · Feb 5, 2024 · 665dcd5 · 665dcd5
1 parent f237ea7
commit 665dcd5
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/app/src/main/java/nl/eduvpn/app/service/HistoryService.java b/app/src/main/java/nl/eduvpn/app/service/HistoryService.java
@@ -29,6 +29,7 @@
 import nl.eduvpn.app.entity.CertExpiryTimes;
 import nl.eduvpn.app.entity.CurrentServer;
 import nl.eduvpn.app.entity.Instance;
+import nl.eduvpn.app.entity.Organization;
 import nl.eduvpn.app.entity.OrganizationList;
 import nl.eduvpn.app.entity.exception.CommonException;
 import nl.eduvpn.app.utils.Listener;
@@ -104,6 +105,10 @@ public CurrentServer getCurrentServer() {
         return _memoryCachedOrganizationList;
     }
 
+    public boolean hasSecureInternetServer() {
+        return _addedServers.getSecureInternetServer() != null;
+    }
+
     public void setOrganizationList(@Nullable OrganizationList organizationList) {
         _memoryCachedOrganizationList = organizationList;
     }

diff --git a/app/src/main/java/nl/eduvpn/app/viewmodel/OrganizationSelectionViewModel.kt b/app/src/main/java/nl/eduvpn/app/viewmodel/OrganizationSelectionViewModel.kt
@@ -78,15 +78,15 @@ class OrganizationSelectionViewModel @Inject constructor(
             // We want to be able to handle async failures, so use supervisorScope
             // https://kotlinlang.org/docs/reference/coroutines/exception-handling.html#supervision
             supervisorScope {
-                val organizationListDeferred = if (historyService.organizationList == null) {
+                val organizationListDeferred = if (!historyService.hasSecureInternetServer()) {
                     connectionState.postValue(ConnectionState.FetchingOrganizations)
                     async {
                         val organizationList = organizationService.fetchOrganizations()
                         historyService.organizationList = organizationList
                         organizationList
                     }
                 } else {
-                    // We can't show any organization servers, user needs to reset to switch.
+                    // We can't show any organization servers (secure internet), user needs to reset to switch.
                     connectionState.postValue(ConnectionState.FetchingServerList)
                     CompletableDeferred(OrganizationList(-1L, emptyList()))
                 }