From 7038f2a85de722bd6440b91a89921f240d812280 Mon Sep 17 00:00:00 2001 From: lebaudantoine Date: Tue, 27 May 2025 16:13:42 +0200 Subject: [PATCH] =?UTF-8?q?=F0=9F=90=9B(backend)=20fix=20KeyError=20crash?= =?UTF-8?q?=20when=20role=20is=20undefined=20in=20request=20data?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Use dict.get() instead of direct key access to prevent server crashes when role field is missing. Fix inherited from magnify project codebase. --- src/backend/core/api/serializers.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/backend/core/api/serializers.py b/src/backend/core/api/serializers.py index 4f57ed2d..09a19130 100644 --- a/src/backend/core/api/serializers.py +++ b/src/backend/core/api/serializers.py @@ -37,7 +37,7 @@ class ResourceAccessSerializerMixin: # Update self.instance and ( - data["role"] == models.RoleChoices.OWNER + data.get("role") == models.RoleChoices.OWNER and not self.instance.resource.is_owner(user) or self.instance.role == models.RoleChoices.OWNER and not self.instance.user == user