diff --git a/app/middleware/recherche_cors.py b/app/middleware/recherche_cors.py index 0c6d029..16e20e4 100644 --- a/app/middleware/recherche_cors.py +++ b/app/middleware/recherche_cors.py @@ -10,6 +10,8 @@ Ce module n'autorise donc que : écritures) ; - les origines sœurs : les origines officielles **prod ET dev, listées explicitement** (``PROD_ORIGINS | DEV_ORIGINS``), **plus** + les origines du service d'agents (``AGENT_ORIGINS`` : console + Spark prod ET dev — fédération console, défaut 2), **plus** celles résolues par ``app.services.draco_urls.toutes()`` (explicite > hôte > dérivation > défaut officiel) — jamais ``*``. Correctif 2026-10-28 (fusion #68/#69, matrice constatée par @@ -48,6 +50,14 @@ DEV_ORIGINS = frozenset({ "https://tiktak-dev.dracodev.net", "https://kanbante-dev.dracodev.net", }) +# Origines du service d'agents (console Spark) : la page de +# recherche de la console appelle /api/search depuis le navigateur +# avec le cookie SSO — sans ces deux origines, le navigateur refuse +# la lecture (fédération console, défaut 2). Jamais "*". +AGENT_ORIGINS = frozenset({ + "https://agent.dracodev.net", + "https://agent-dev.dracodev.net", +}) # Clés de draco_urls.toutes() qui sont des applications sœurs. _SOEUR_KEYS = ("obsigate", "flowdeck", "tiktak", "kanbante") @@ -113,7 +123,7 @@ def sister_origins(resolved: dict | None = None, request_host: str | None = None # l'environnement de la requête, désormais inclus d'office). # ``resolved`` n'ajoute que des origines de configuration sans # jamais retirer une origine officielle. - origins |= PROD_ORIGINS | DEV_ORIGINS + origins |= PROD_ORIGINS | DEV_ORIGINS | AGENT_ORIGINS return frozenset(origins) diff --git a/tests/test_recherche.py b/tests/test_recherche.py index bcebe93..c7ffde3 100644 --- a/tests/test_recherche.py +++ b/tests/test_recherche.py @@ -49,6 +49,8 @@ def test_sister_origins_dev(): "https://flowdeck-dev.dracodev.net", "https://tiktak-dev.dracodev.net", "https://kanbante-dev.dracodev.net", + "https://agent.dracodev.net", + "https://agent-dev.dracodev.net", } @@ -86,3 +88,22 @@ def test_preflight_headers(): headers = preflight_headers("https://kanbante-dev.dracodev.net", RESOLVED_DEV) assert headers["Access-Control-Allow-Methods"] == "GET, OPTIONS" assert preflight_headers("https://evil.example", RESOLVED_DEV) is None + + +def test_origines_agent_console(): + # Fédération console (défaut 2) : la page de recherche de la + # console appelle /api/search depuis agent(. -dev) avec le + # cookie SSO — ces deux origines passent, prod ET dev, quelle + # que soit la résolution ; un tiers reste refusé. + assert cors_headers_for( + "https://agent.dracodev.net", RESOLVED_DEV) is not None + assert cors_headers_for( + "https://agent-dev.dracodev.net", {}) is not None + assert cors_headers_for( + "https://agent.dracodev.net", + RESOLVED_DEV)["Access-Control-Allow-Credentials"] == "true" + assert preflight_headers( + "https://agent-dev.dracodev.net", RESOLVED_DEV) is not None + assert "https://agent.dracodev.net" in sister_origins({}) + assert "https://agent-dev.dracodev.net" in sister_origins({}) + assert "https://agent-dev.dracodev.net" in sister_origins()