Add exta delay to TTL at redirection

Author: pcrespov

services/web/server/src/simcore_service_webserver/resource_manager/websocket_manager.py

@@ -45,7 +45,7 @@ def _resource_key(self) -> Dict[str, str]:
             else "*",
         }
 
-    async def set_socket_id(self, socket_id: str) -> None:
+    async def set_socket_id(self, socket_id: str, *, extra_tll: int = 0) -> None:
         log.debug(
             "user %s/tab %s adding socket %s in registry...",
             self.user_id,
@@ -54,9 +54,9 @@ async def set_socket_id(self, socket_id: str) -> None:
         )
         registry = get_registry(self.app)
         await registry.set_resource(self._resource_key(), (SOCKET_ID_KEY, socket_id))
-        # hearthbeat is not emulated in tests, make sure that with very small GC intervals
-        # the resources do not result as timeout; this value is usually in the order of minutes
-        timeout = max(3, get_service_deletion_timeout(self.app))
+        # NOTE: hearthbeat is not emulated in tests, make sure that with very small GC intervals
+        # the resources do not expire; this value is usually in the order of minutes
+        timeout = max(3, get_service_deletion_timeout(self.app)) + abs(extra_tll)
         await registry.set_key_alive(self._resource_key(), timeout)
 
     async def get_socket_id(self) -> Optional[str]:

services/web/server/src/simcore_service_webserver/studies_access.py

@@ -91,14 +91,21 @@ async def activate_as_guest_user(user: Dict, app: web.Application):
     from .login.handlers import GUEST, ANONYMOUS
     from .resource_manager.websocket_manager import WebsocketRegistry
 
+    # Save time to allow user to be redirected back and not being deleted by GC
+    # SEE https://github.com/ITISFoundation/osparc-simcore/pull/1928#discussion_r517176479
+    EXTRA_TIME_TO_COMPLETE_REDIRECT = 3
+
     if user.get("role") == ANONYMOUS:
         db = get_storage(app)
         username = user["name"]
 
         # creates an entry in the socket's registry to avoid garbage collector (GC) deleting it
-        # See https://github.com/ITISFoundation/osparc-simcore/issues/1853
+        # SEE https://github.com/ITISFoundation/osparc-simcore/issues/1853
+        #
         registry = WebsocketRegistry(user["id"], f"{username}-guest-session", app)
-        await registry.set_socket_id(f"{username}-guest-socket-id")
+        await registry.set_socket_id(
+            f"{username}-guest-socket-id", extra_tll=EXTRA_TIME_TO_COMPLETE_REDIRECT
+        )
 
         # Now that we know the ID, we set the user as GUEST
         # This extra step is to prevent the possibility that the GC is cleaning while