added more validations to load utils

client/ayon_core/pipeline/load/utils.py

@@ -1,9 +1,11 @@
 import os
+import uuid
 import platform
 import logging
 import inspect
 import collections
 import numbers
+from typing import Any
 
 import ayon_api
 
@@ -464,8 +466,13 @@ def update_container(container, version=-1):
 
  # Compute the different version from 'representation'
  project_name = get_current_project_name()
+ repre_id = container["representation"]
+ if not _is_valid_representation_id(repre_id):
+ raise ValueError(
+ f"Got container with invalid representation id '{repre_id}'"
+ )
  current_representation = ayon_api.get_representation_by_id(
- project_name, container["representation"]
+ project_name, repre_id
  )
 
  assert current_representation is not None, "This is a bug"
@@ -817,6 +824,16 @@ def get_outdated_containers(host=None, project_name=None):
  return filter_containers(containers, project_name).outdated
 
 
+def _is_valid_representation_id(repre_id: Any) -> bool:
+ if not repre_id:
+ return False
+ try:
+ uuid.UUID(repre_id)
+ except (ValueError, TypeError, AttributeError):
+ return False
+ return True
+
+
 def filter_containers(containers, project_name):
  """Filter containers and split them into 4 categories.
 
@@ -852,7 +869,7 @@ def filter_containers(containers, project_name):
  repre_ids = {
  container["representation"]
  for container in containers
- if container["representation"]
+ if _is_valid_representation_id(container["representation"])
  }
  if not repre_ids:
  if containers:
@@ -917,7 +934,7 @@ def filter_containers(containers, project_name):
  for container in containers:
  container_name = container["objectName"]
  repre_id = container["representation"]
- if not repre_id:
+ if not _is_valid_representation_id(repre_id):
  invalid_containers.append(container)
  continue