From d57103ff392c59d5a59f405a50e8fb9bb32231e8 Mon Sep 17 00:00:00 2001 From: "Jens L." Date: Fri, 31 Oct 2025 01:50:38 +0100 Subject: [PATCH] outpost: revert breaking signals change (#17847) I have no idea why this breaks tests Signed-off-by: Jens Langhammer --- authentik/outposts/signals.py | 7 ++++--- authentik/tenants/tests/test_recovery.py | 5 +++-- 2 files changed, 7 insertions(+), 5 deletions(-) diff --git a/authentik/outposts/signals.py b/authentik/outposts/signals.py index 4adba5ea7c..8b9f0cd15a 100644 --- a/authentik/outposts/signals.py +++ b/authentik/outposts/signals.py @@ -77,9 +77,10 @@ def outpost_m2m_changed(sender, instance: Outpost | Provider, action: str, **_): @receiver(post_save, sender=Outpost) -def outpost_post_save(sender, instance: Outpost, **_): - LOGGER.info("Outpost saved, ensuring token and user are created and permissions are set") - _ = instance.token +def outpost_post_save(sender, instance: Outpost, created: bool, **_): + if created: + LOGGER.info("New outpost saved, ensuring initial token and user are created") + _ = instance.token outpost_controller.send_with_options( args=(instance.pk,), rel_obj=instance.service_connection, diff --git a/authentik/tenants/tests/test_recovery.py b/authentik/tenants/tests/test_recovery.py index e100dbd3c7..a43398f986 100644 --- a/authentik/tenants/tests/test_recovery.py +++ b/authentik/tenants/tests/test_recovery.py @@ -5,7 +5,8 @@ from json import loads from django.urls import reverse from django_tenants.utils import get_public_schema_name -from authentik.core.models import Token, TokenIntents, User +from authentik.core.models import Token, TokenIntents +from authentik.core.tests.utils import create_test_user from authentik.lib.config import CONFIG from authentik.lib.generators import generate_id from authentik.tenants.models import Tenant @@ -21,7 +22,7 @@ class TestRecovery(TenantAPITestCase): def setUp(self): super().setUp() self.tenant = Tenant.objects.get(schema_name=get_public_schema_name()) - self.user: User = User.objects.create_user(username="recovery-test-user") + self.user = create_test_user() @CONFIG.patch("outposts.disable_embedded_outpost", True) @CONFIG.patch("tenants.enabled", True)