fix: URLs sœurs dérivées de l'environnement de l'agent — le menu inter-apps ne pointe plus la prod depuis la dev BUG-120
This commit is contained in:
+70
-4
@@ -22,6 +22,16 @@ Variables (voir ``.env.example``) :
|
||||
distincte de ``OBSIGATE_GITEA_URL`` (source connectée de
|
||||
l'assistant, réglable depuis l'UI, pas une URL d'affichage).
|
||||
|
||||
Résolution d'une URL sœur (BUG-120) : la variable explicite prime ;
|
||||
à défaut, l'**environnement est dérivé de l'hôte de l'agent** tel
|
||||
qu'ObsiGate le connaît déjà (:func:`agent_url`) — si cet hôte est de
|
||||
la forme ``agent<SUFFIXE>.dracodev.net`` (``-dev`` en dev, vide en
|
||||
prod), l'URL sœur est ``https://<slug><SUFFIXE>.dracodev.net``. Un
|
||||
hôte hors motif (autre domaine, LAN, IP, port) ne dérive **rien** :
|
||||
on ne fabrique jamais un lien deviné, le défaut officiel s'applique.
|
||||
L'installation pointe donc les bons liens selon son environnement
|
||||
sans déclarer les quatre ``DRACO_<APP>_URL``.
|
||||
|
||||
Ce module ne lit l'environnement qu'à l'appel (jamais à l'import) :
|
||||
les tests et un changement de ``.env`` suivi d'un redémarrage
|
||||
prennent effet sans état figé.
|
||||
@@ -30,6 +40,7 @@ prennent effet sans état figé.
|
||||
from __future__ import annotations
|
||||
|
||||
import os
|
||||
import re
|
||||
from urllib.parse import urlparse
|
||||
|
||||
# Défauts officiels (production future) — jamais de nom en ``-dev`` ici.
|
||||
@@ -51,6 +62,21 @@ _VARIABLES: dict[str, str] = {
|
||||
"gitea": "DRACO_GITEA_URL",
|
||||
}
|
||||
|
||||
# Slugs d'hôte des applications sœurs pour la dérivation
|
||||
# d'environnement (BUG-120) : premier label de ``<slug>.dracodev.net``
|
||||
# (et de ``<slug>-dev.dracodev.net`` en dev). Gitea est **exclue** :
|
||||
# son URL ne se dérive jamais.
|
||||
_SLUGS_HOTE: dict[str, str] = {
|
||||
"obsigate": "og",
|
||||
"flowdeck": "flowdeck",
|
||||
"tiktak": "tiktak",
|
||||
"kanbante": "kanbante",
|
||||
}
|
||||
|
||||
# Premier label de l'hôte de l'agent : ``agent`` suivi d'un suffixe
|
||||
# d'environnement éventuel (``-dev`` en dev, vide en prod).
|
||||
_MOTIF_AGENT = re.compile(r"^agent(-[a-z0-9-]*)?$")
|
||||
|
||||
|
||||
def _nettoyer(valeur: str) -> str:
|
||||
return valeur.strip().rstrip("/")
|
||||
@@ -87,12 +113,52 @@ def agent_origine() -> str:
|
||||
return origine(agent_url()) or DEFAUTS["agent"]
|
||||
|
||||
|
||||
def _suffixe_environnement() -> str | None:
|
||||
"""Suffixe d'environnement déduit de l'hôte de l'agent (BUG-120).
|
||||
|
||||
Lit l'hôte de :func:`agent_url` — l'origine de l'agent telle
|
||||
qu'ObsiGate la connaît déjà (``DRACO_AGENT_URL``, à défaut
|
||||
``AGENT_EVENEMENTS_URL``). Si cet hôte est exactement de la forme
|
||||
``agent<SUFFIXE>.dracodev.net`` sans port, retourne ``SUFFIXE``
|
||||
(``-dev`` en dev, ``""`` en prod). Sinon (autre domaine, LAN, IP,
|
||||
port exotique) retourne ``None`` : aucune dérivation, on ne
|
||||
fabrique jamais un lien deviné.
|
||||
"""
|
||||
u = urlparse(agent_url())
|
||||
hote = (u.hostname or "").lower()
|
||||
labels = hote.split(".")
|
||||
if len(labels) != 3 or labels[1:] != ["dracodev", "net"]:
|
||||
return None
|
||||
try:
|
||||
if u.port is not None:
|
||||
return None
|
||||
except ValueError:
|
||||
return None
|
||||
motif = _MOTIF_AGENT.match(labels[0])
|
||||
if motif is None:
|
||||
return None
|
||||
return motif.group(1) or ""
|
||||
|
||||
|
||||
def app_url(app: str) -> str:
|
||||
"""Base publique d'une application de l'écosystème (sans ``/`` final)."""
|
||||
"""Base publique d'une application de l'écosystème (sans ``/`` final).
|
||||
|
||||
Résolution d'une URL sœur (BUG-120) : ``DRACO_<APP>_URL``
|
||||
explicite d'abord ; à défaut, dérivation d'environnement depuis
|
||||
l'hôte de l'agent (:func:`_suffixe_environnement`) ; à défaut,
|
||||
le défaut officiel. L'URL Gitea ne se dérive jamais.
|
||||
"""
|
||||
if app == "agent":
|
||||
return agent_url()
|
||||
variable = _VARIABLES[app]
|
||||
return _nettoyer(os.environ.get(variable, "")) or DEFAUTS[app]
|
||||
explicite = _nettoyer(os.environ.get(variable, ""))
|
||||
if explicite:
|
||||
return explicite
|
||||
if app in _SLUGS_HOTE:
|
||||
suffixe = _suffixe_environnement()
|
||||
if suffixe is not None:
|
||||
return f"https://{_SLUGS_HOTE[app]}{suffixe}.dracodev.net"
|
||||
return DEFAUTS[app]
|
||||
|
||||
|
||||
def app_hote(app: str) -> str:
|
||||
@@ -105,8 +171,8 @@ def hotes_reconnus() -> dict[str, str]:
|
||||
|
||||
Contient **toujours** les hôtes officiels (un lien collé depuis la
|
||||
production reste reconnaissable en dev, et réciproquement) **plus**
|
||||
l'hôte configuré de chaque application (les noms en ``-dev`` du
|
||||
poste quand ils sont déclarés dans son ``.env``).
|
||||
l'hôte résolu de chaque application (les noms en ``-dev`` du poste,
|
||||
déclarés dans son ``.env`` ou dérivés de l'hôte de l'agent).
|
||||
"""
|
||||
hotes: dict[str, str] = {}
|
||||
for app, defaut in DEFAUTS.items():
|
||||
|
||||
Reference in New Issue
Block a user