From 8f0a642175387fb03c84a7f8658667c2167edef6 Mon Sep 17 00:00:00 2001 From: duckduckgrayduck <102841251+duckduckgrayduck@users.noreply.github.com> Date: Tue, 7 Jul 2026 15:41:13 -0500 Subject: [PATCH] Guard against needless celery task enqueues --- documentcloud/addons/views.py | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/documentcloud/addons/views.py b/documentcloud/addons/views.py index 224e730b..4c2ef374 100644 --- a/documentcloud/addons/views.py +++ b/documentcloud/addons/views.py @@ -429,8 +429,12 @@ def perform_update(self, serializer): @action(detail=False, methods=["post"], permission_classes=[AllowAny]) def update_config(self, request): name = request.data.get("repository") - if name: + # If the name doesn't match an existing repo to an Add-On, + # don't needlessly enqueue a celery task only for it to no-op + if name and AddOn.objects.filter(repository=name, removed=False).exists(): update_config.delay(name) + else: + logger.warning("[ADDON] update_config no match for repository=%s", name) return Response(status=status.HTTP_204_NO_CONTENT) class Filter(django_filters.FilterSet):