fix(model-server): RestWebModelClient couldn't request a clientId

modelix · Nov 25, 2024 · 47a12bf · 47a12bf
1 parent 5b647b4
commit 47a12bf
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/model-server/src/main/kotlin/org/modelix/model/server/handlers/KeyValueLikeModelServer.kt b/model-server/src/main/kotlin/org/modelix/model/server/handlers/KeyValueLikeModelServer.kt
@@ -294,6 +294,9 @@ class KeyValueLikeModelServer(
                 serverId = {
                     throw NoPermissionException("'$key' is read-only.")
                 },
+                legacyClientId = {
+                    throw NoPermissionException("Directly writing to 'clientId' is not allowed")
+                },
                 unknown = {
                     userDefinedEntries[key] = value
                 },
@@ -349,6 +352,7 @@ class KeyValueLikeModelServer(
             serverId = {
                 if (isWrite) throw NoPermissionException("'$key' is read-only.")
             },
+            legacyClientId = {},
             unknown = {
                 call.checkPermission(ModelServerPermissionSchema.legacyUserDefinedObjects.run { if (isWrite) write else read })
             },
@@ -360,6 +364,7 @@ class KeyValueLikeModelServer(
         immutableObject: () -> R,
         branch: (branch: BranchReference) -> R,
         serverId: () -> R,
+        legacyClientId: () -> R,
         unknown: () -> R,
     ): R {
         return when {
@@ -368,6 +373,7 @@ class KeyValueLikeModelServer(
             key.startsWith(PROTECTED_PREFIX) -> throw NoPermissionException("Access to keys starting with '$PROTECTED_PREFIX' is only permitted to the model server itself.")
             key.startsWith(RepositoriesManager.KEY_PREFIX) -> throw NoPermissionException("Access to keys starting with '${RepositoriesManager.KEY_PREFIX}' is only permitted to the model server itself.")
             key == RepositoriesManager.LEGACY_SERVER_ID_KEY || key == RepositoriesManager.LEGACY_SERVER_ID_KEY2 -> serverId()
+            key == "clientId" -> legacyClientId()
             else -> unknown()
         }
     }