From 7c304e14083bbe8f00036d6c5943bdefbe070547 Mon Sep 17 00:00:00 2001 From: Vitali Yanushchyk Date: Wed, 16 Oct 2024 13:26:01 -0300 Subject: [PATCH] fix ! explicit environment variables --- .../apps/core/management/commands/upgrade.py | 6 +----- src/hope_dedup_engine/apps/faces/admin.py | 1 - src/hope_dedup_engine/config/__init__.py | 4 ++-- 3 files changed, 3 insertions(+), 8 deletions(-) diff --git a/src/hope_dedup_engine/apps/core/management/commands/upgrade.py b/src/hope_dedup_engine/apps/core/management/commands/upgrade.py index c997605d..75ae85d8 100644 --- a/src/hope_dedup_engine/apps/core/management/commands/upgrade.py +++ b/src/hope_dedup_engine/apps/core/management/commands/upgrade.py @@ -174,12 +174,8 @@ def handle(self, *args: Any, **options: Any) -> None: # noqa: C901 verbosity=self.verbosity - 1, interactive=False, ) - - admin = User.objects.get(email=self.admin_email) else: - admin = User.objects.filter(is_superuser=True).first() - if not admin: - raise CommandError("Failure: Error when creating an admin user!") + echo("No admin email provided, skip superuser creation") from hope_dedup_engine.apps.security.constants import DEFAULT_GROUP_NAME diff --git a/src/hope_dedup_engine/apps/faces/admin.py b/src/hope_dedup_engine/apps/faces/admin.py index 6fd74cff..efa34eaf 100644 --- a/src/hope_dedup_engine/apps/faces/admin.py +++ b/src/hope_dedup_engine/apps/faces/admin.py @@ -45,7 +45,6 @@ def sync_dnn_files(self, request): else: worker_count = len(active_workers) if worker_count > 1: - print(f"{worker_count=}") job = group(sync_dnn_files.s(force=True) for _ in range(worker_count)) result = job.apply_async() self.message_user( diff --git a/src/hope_dedup_engine/config/__init__.py b/src/hope_dedup_engine/config/__init__.py index 5f915371..d07de620 100644 --- a/src/hope_dedup_engine/config/__init__.py +++ b/src/hope_dedup_engine/config/__init__.py @@ -29,7 +29,7 @@ class Group(Enum): CONFIG: "Dict[str, ConfigItem]" = { "ADMIN_EMAIL": ( str, - SmartEnv.NOTSET, + "", "admin", True, "Initial user created at first deploy", @@ -100,7 +100,7 @@ class Group(Enum): False, f"{celery_doc}#broker-transport-options", ), - "CSRF_COOKIE_SECURE": (bool, True, False, setting("csrf-cookie-secure")), + "CSRF_COOKIE_SECURE": (bool, True, False, False, setting("csrf-cookie-secure")), "DATABASE_URL": ( str, SmartEnv.NOTSET,