Merge pull request #1964 from fractal-analytics-platform/1959-review-…

…assumption-on-single-group-in-data-migration-script

Fix `where` in create-first-group function

CHANGELOG.md

@@ -90,6 +90,8 @@
     * Introduce `FractalSSH.fetch_file` and `FractalSSH.read_remote_json_file` (\#1949).
     * Use `paramiko.sftp_client.SFTPClient` methods directly rathen than `fabric` wrappers (\#1949).
     * Disable prefetching for `SFTPClient.get` (\#1949).
+* Internal:
+    * Update `_create_first_group` so that it only searches for `UserGroups` with a given name (\#1964).
 * Dependencies:
     * Bump fastapi to `0.115` (\#1942).
 * Testing:

fractal_server/app/security/__init__.py

@@ -315,13 +315,20 @@ async def _create_first_user(
 
 
 def _create_first_group():
+    """
+    Create a `UserGroup` with `name=FRACTAL_DEFAULT_GROUP_NAME`, if missing.
+    """
     function_logger = set_logger("fractal_server.create_first_group")
 
     function_logger.info(
         f"START _create_first_group, with name '{FRACTAL_DEFAULT_GROUP_NAME}'"
     )
     with next(get_sync_db()) as db:
-        group_all = db.execute(select(UserGroup))
+        group_all = db.execute(
+            select(UserGroup).where(
+                UserGroup.name == FRACTAL_DEFAULT_GROUP_NAME
+            )
+        )
         if group_all.scalars().one_or_none() is None:
             first_group = UserGroup(name=FRACTAL_DEFAULT_GROUP_NAME)
             db.add(first_group)