From f16e3695404c35871907dd98dcc22c005a4be4f6 Mon Sep 17 00:00:00 2001 From: Ke Sun Date: Fri, 6 Mar 2026 10:37:00 +0800 Subject: [PATCH] fix(celery): remove legacy environment variables to prevent CLI hijacking - Remove BROKER_URL environment variable to prevent Celery CLI override - Remove RESULT_BACKEND environment variable to prevent Celery CLI override - Remove CELERY_BROKER environment variable to prevent Celery CLI override - Remove CELERY_BACKEND environment variable to prevent Celery CLI override - Add clarifying comments explaining the purpose of neutralizing legacy vars - Ensures canonical broker and backend URLs are not accidentally overridden by Celery's CLI/Click integration --- api/app/celery_app.py | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/api/app/celery_app.py b/api/app/celery_app.py index c728abb2..0319e079 100644 --- a/api/app/celery_app.py +++ b/api/app/celery_app.py @@ -29,7 +29,12 @@ _broker_url = f"redis://:{quote(settings.REDIS_PASSWORD)}@{settings.REDIS_HOST}: _backend_url = f"redis://:{quote(settings.REDIS_PASSWORD)}@{settings.REDIS_HOST}:{settings.REDIS_PORT}/{settings.REDIS_DB_CELERY_BACKEND}" os.environ["CELERY_BROKER_URL"] = _broker_url os.environ["CELERY_RESULT_BACKEND"] = _backend_url +# Neutralize legacy Celery env vars that can be hijacked by Celery's CLI/Click +# integration and accidentally override our canonical URLs. os.environ.pop("BROKER_URL", None) +os.environ.pop("RESULT_BACKEND", None) +os.environ.pop("CELERY_BROKER", None) +os.environ.pop("CELERY_BACKEND", None) celery_app = Celery( "redbear_tasks",