Projet

Général

Profil

Development #17967

debian_config_common.py : ne pas ajouter les trucs de multitenant quand c'est wcs

Ajouté par Frédéric Péters il y a plus de 6 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
11 août 2017
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Parce que wcs conserve sa gestion propre du multitenant (pour le moment au moins).


Fichiers

Révisions associées

Révision fe63e7ae (diff)
Ajouté par Frédéric Péters il y a plus de 6 ans

debian: regroup multitenant settings (#17967)

Révision ec8eda46 (diff)
Ajouté par Frédéric Péters il y a plus de 6 ans

debian: don't configure multitenant stuff for wcs (#17967)

Historique

#1

Mis à jour par Frédéric Péters il y a plus de 6 ans

#2

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.

#3

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',
     )
#4

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.

#5

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

Formats disponibles : Atom PDF