Development #17967
debian_config_common.py : ne pas ajouter les trucs de multitenant quand c'est wcs
0%
Description
Parce que wcs conserve sa gestion propre du multitenant (pour le moment au moins).
Fichiers
Révisions associées
debian: don't configure multitenant stuff for wcs (#17967)
Historique
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Fichier 0001-debian-don-t-configure-multitenant-stuff-for-wcs-179.patch 0001-debian-don-t-configure-multitenant-stuff-for-wcs-179.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
Ack, si ça peut être découpé en deux patchs un qui nettoye (genre le if 'mellon' in INSTALLED_APPS), l'autre qui ajoute la condition, c'est cool, sinon c'est pas bien grave.
Mis à jour par Thomas Noël il y a plus de 6 ans
Attention deux fois MIDDLEWARE_CLASSES ajouté avant et après ligne 210 dans :
MIDDLEWARE_CLASSES = MIDDLEWARE_CLASSES + ( 'hobo.middleware.VersionMiddleware', # /__version__ 'hobo.middleware.cors.CORSMiddleware', ) + MIDDLEWARE_CLASSES
et même soucis ligne 231
Sur cette partie :
# SAML2: search IdP in <tenant> if 'authentic2' not in INSTALLED_APPS: MELLON_ADAPTER = ('hobo.multitenant.mellon.MellonAdapter',) MELLON_DEFAULT_ASSERTION_CONSUMER_BINDING = 'artifact' MELLON_OPENED_SESSION_COOKIE_NAME = 'A2_OPENED_SESSION'
je laisserais les deux dernières lignes en dehors du if PROJECT_NAME != 'wcs':
Je ne toucherais pas à cette partie, parce que pas sûr de tout bien voir :
-if 'authentic2' not in INSTALLED_APPS: +if 'mellon' in INSTALLED_APPS: MIDDLEWARE_CLASSES = MIDDLEWARE_CLASSES + ( 'mellon.middleware.PassiveAuthenticationMiddleware', ) -else: + +if 'authentic2' in INSTALLED_APPS: MIDDLEWARE_CLASSES = MIDDLEWARE_CLASSES + ( 'hobo.agent.authentic2.middleware.ProvisionningMiddleware', )
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Statut changé de En cours à Résolu (à déployer)
Voilà, poussé en deux commits, le premier pour regrouper le paramétrage de tenants, le second pour mettre un "if wcs" (en fait trois, parce que j'ai laissé ce qui relève des templates ensemble).
Et j'ai limité les nettoyages genre "mellon" in INSTALLED_APPS, que ça vienne plus tard.
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
debian: regroup multitenant settings (#17967)