feat(NET-584): session mgmt for RAC

gravitl · Sep 19, 2023 · a1a0eca · a1a0eca
1 parent c69b4f3
commit a1a0eca
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/controllers/user.go b/controllers/user.go
@@ -120,15 +120,13 @@ func authenticateUser(response http.ResponseWriter, request *http.Request) {
 					slog.Info(fmt.Sprintf("enabling ext client %s for user %s due to RAC autodisabling feature", client.ClientID, client.OwnerID))
 					if newClient, err := logic.ToggleExtClientConnectivity(&client, true); err != nil {
 						slog.Error("error disabling ext client in RAC autodisable hook", "error", err)
-						return
+						continue // dont return but try for other clients
 					} else {
 						// publish peer update to ingress gateway
 						if ingressNode, err := logic.GetNodeByID(newClient.IngressGatewayID); err == nil {
 							if err = mq.PublishPeerUpdate(); err != nil {
 								slog.Error("error updating ext clients on", "ingress", ingressNode.ID.String(), "err", err.Error())
 							}
-						} else {
-							return
 						}
 					}
 				}

diff --git a/pro/remote_access_client.go b/pro/remote_access_client.go
@@ -48,7 +48,7 @@ func racAutoDisableHook() error {
 				slog.Info(fmt.Sprintf("disabling ext client %s for user %s due to RAC autodisabling", client.ClientID, client.OwnerID))
 				if err := disableExtClient(&client); err != nil {
 					slog.Error("error disabling ext client in RAC autodisable hook", "error", err)
-					return err
+					continue // dont return but try for other clients
 				}
 			}
 		}