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

modelix · Dec 11, 2024 · ff08972 · ff08972
1 parent 693f3bd
commit ff08972
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
@@ -280,6 +280,9 @@ class KeyValueLikeModelServer(
                 serverId = {
                     throw NoPermissionException("'$key' is read-only.")
                 },
+                legacyClientId = {
+                    throw NoPermissionException("Directly writing to 'clientId' is not allowed")
+                },
                 unknown = {
                     userDefinedEntries[key] = value
                 },
@@ -335,6 +338,7 @@ class KeyValueLikeModelServer(
             serverId = {
                 if (isWrite) throw NoPermissionException("'$key' is read-only.")
             },
+            legacyClientId = {},
             unknown = {
                 call.checkPermission(ModelServerPermissionSchema.legacyUserDefinedObjects.run { if (isWrite) write else read })
             },
@@ -346,6 +350,7 @@ class KeyValueLikeModelServer(
         immutableObject: () -> R,
         branch: (branch: BranchReference) -> R,
         serverId: () -> R,
+        legacyClientId: () -> R,
         unknown: () -> R,
     ): R {
         return when {
@@ -354,6 +359,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()
         }
     }