fix: Disallow None id encoding in AsyncNode.to_global_id() (#2898) (

#2955)

Co-authored-by: Gyubong Lee <jopemachine@naver.com>

changes/2898.fix.md

@@ -0,0 +1 @@
+Disallow `None` id encoding in `AsyncNode.to_global_id()`.

src/ai/backend/manager/api/schema.graphql

@@ -1269,9 +1269,7 @@ type ContainerRegistryNode implements Node {
   """The ID of the object"""
   id: ID!
 
-  """
-  Added in 24.09.0. The undecoded UUID type id of DB container_registries row.
-  """
+  """Added in 24.09.0. The UUID type id of DB container_registries row."""
   row_id: UUID
   name: String
 

src/ai/backend/manager/models/container_registry.py

@@ -264,7 +264,7 @@ class Meta:
         description = "Added in 24.09.0."
 
     row_id = graphene.UUID(
-        description="Added in 24.09.0. The undecoded UUID type id of DB container_registries row."
+        description="Added in 24.09.0. The UUID type id of DB container_registries row."
     )
     name = graphene.String()
     url = graphene.String(required=True, description="Added in 24.09.0.")

src/ai/backend/manager/models/gql_relay.py

@@ -143,8 +143,10 @@ async def node_resolver(cls, only_type, root, info, id):
         return await cls.get_node_from_global_id(info, id, only_type=only_type)
 
     @staticmethod
-    def to_global_id(type_, id) -> str:
-        return base64(f"{type_}:{id}")
+    def to_global_id(type_, id_) -> str:
+        if id_ is None:
+            raise Exception("Encoding None value as Global ID is not allowed.")
+        return base64(f"{type_}:{id_}")
 
     @classmethod
     def resolve_global_id(cls, info, global_id: str) -> tuple[str, str]: