Bug #16429
Utilisation de KNOWN_SERVICES dans le cadre de multi-publik
0%
Description
Noté à l'occasion de #16393, il y a divers endroits dans Combo qui utilisent KNOWN_SERVICES pour tirer le premier service d'un type donné et font ça sans prêter attention au côté multipublik (préférer un service qui ne soit pas marqué "secondary").
Au moins :
- combo/apps/fargo,
remote_service=settings.KNOWN_SERVICES['fargo'].values()[0] - combo/apps/family,
settings.KNOWN_SERVICES['passerelle'].values()[0].
Fichiers
Révisions associées
Historique
Mis à jour par Thomas Noël il y a presque 7 ans
Est-ce qu'on pourrait, dans une autre vision, faire que KNOWN_SERVICES ne contient que des primary ? Qu'est-ce que ça bloquerait ?
Mis à jour par Frédéric Péters il y a presque 7 ans
Par exemple si un porte-document est déployé globalement au niveau de l'interco, on en perdrait l'accès.
Dans ce patch, maintenant que le cas Fargo va être corrigé dans #16853, il resterait juste à ajouter une sélection du service famille dans combo/apps/family.
Mis à jour par Frédéric Péters il y a environ 4 ans
- Fichier 0001-family-ignore-non-primary-services-16429.patch 0001-family-ignore-non-primary-services-16429.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Version minimale pour fermer ce ticket : modification cellules famille pour uniquement considérer le passerelle primaire; si jamais se trouve un moment un environnement où les familles sont à tirer d'un passerelle secondaire, les cellules pourront être modifiées pour gagner une sélection du bon passerelle, comme ça a été fait pour Fargo.
Mis à jour par Frédéric Péters il y a environ 4 ans
Mis à jour par Thomas Noël il y a environ 4 ans
- Statut changé de Solution proposée à Solution validée
J'ai jamais été bien fan du [0] mais bon allez.
(ie je trouverais plus joli de faire comme dans get_fargo_site :
for site in settings.KNOWN_SERVICES['passerelle'].values():
if not site.get('secondary'):
return site
)Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
Primaire il n'y en aura jamais qu'un, [0] l'attrape. Faire comme dans fargo, c'est gérer correctement la possibilité de plusieurs services, je l'ai fait pour wcs, je l'ai fait pour fargo, personne ne l'a fait ici.
commit 0e445cf6b9f0bb46c03fe212d1aef36692c83fa1
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Wed Mar 4 08:48:55 2020 +0100
family: ignore non-primary services (#16429)
Mis à jour par Thomas Noël il y a environ 4 ans
Frédéric Péters a écrit :
Primaire il n'y en aura jamais qu'un, [0] l'attrape.
Oui bien sûr c'est juste qu'on continue à tout parcourir alors qu'on l'a déjà attrapé, mais vraiment, pas grave du tout du tout.
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
family: ignore non-primary services (#16429)