Projet

Général

Profil

Development #19763

permettre la surcharge dans le settings des paramètres d'une "application"

Ajouté par Serghei Mihai il y a plus de 6 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Josué Kouka
Catégorie:
-
Version cible:
-
Début:
29 octobre 2017
Echéance:
31 octobre 2017
% réalisé:

100%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

La même application peut avoir des chemins d'authentification, deconnexion, etc, différents par deploiement: par exemple Teamnet à Meyzieu authentifie les usagers sur la page /meyzieu/auth/teamnetauth et à Rochefort sur /portail-famille/auth/teamnetauth.
Tel que c'est fait aujourd'hui il faut créer une nouvelle application pour pouvoir prendre en compte un nouveau portail Teamnet.

Ça a été fait pour Meyzieu:

class TeamnetMeyzieu(Teamnet):
    SITE_LOGIN_PATH_PREFIX = '/meyzieu/'

Il faudrait éviter ça en permettant de redéfinir les configs des applications dans le settings du tenant. Ça nous évitera de devoir écrire du code juste pour:

class TeamnetRochefort(Teamnet):
    SITE_LOGIN_PATH_PREFIX = '/portail-famille/'


Fichiers

Révisions associées

Révision 1f4bd5d8 (diff)
Ajouté par Josué Kouka il y a plus de 6 ans

allow override of app's settings (#19763)

Historique

#2

Mis à jour par Brice Mallet il y a plus de 6 ans

  • Echéance mis à 31 octobre 2017
#3

Mis à jour par Josué Kouka il y a plus de 6 ans

  • Statut changé de Nouveau à En cours
  • Assigné à mis à Josué Kouka
#4

Mis à jour par Josué Kouka il y a plus de 6 ans

Serghei Mihai a écrit :

La même application peut avoir des chemins d'authentification, deconnexion, etc, différents par deploiement: par exemple Teamnet à Meyzieu authentifie les usagers sur la page /meyzieu/auth/teamnetauth et à Rochefort sur /portail-famille/auth/teamnetauth.
Tel que c'est fait aujourd'hui il faut créer une nouvelle application pour pouvoir prendre en compte un nouveau portail Teamnet.

Ça a été fait pour Meyzieu:
[...]

Il faudrait éviter ça en permettant de redéfinir les configs des applications dans le settings du tenant. Ça nous évitera de devoir écrire du code juste pour:
[...]

Tout à fait raison, j'y ai pensé, mais je n'ai malheureusement pas fait de tickets. Je corrige

#5

Mis à jour par Josué Kouka il y a plus de 6 ans

#6

Mis à jour par Serghei Mihai il y a plus de 6 ans

Je voyais globalement la possibilité de redéfinir un paramètre de l'application dans le settings et pas juste SITE_LOGIN_PATH.

#7

Mis à jour par Josué Kouka il y a plus de 6 ans

Serghei Mihai a écrit :

Je voyais globalement la possibilité de redéfinir un paramètre de l'application dans le settings et pas juste SITE_LOGIN_PATH.

Je pense que si permet de surcharger plusieurs parametres autant creer une nouvelle app. Or mis le SITE_LOGIN_PATH (enfin, la encore le path final est identique pour tous les sites), les autres parametres sont très liés aux applications pour lesquelles Mandaye fait reverse-proxy.

#8

Mis à jour par Serghei Mihai il y a plus de 6 ans

Créer une nouvelle app passe par du code: review, release. Or en mettant des settings dans le tenant je deploie une instance en moins d'une heure.

#9

Mis à jour par Josué Kouka il y a plus de 6 ans

Ok. Un patch qui permet de surcharger les parametre d'une application.

#10

Mis à jour par Josué Kouka il y a plus de 6 ans

Nouveau avec le chemin absolu du auth_checker calculé lors de la génération des infos de login.

#11

Mis à jour par Serghei Mihai il y a plus de 6 ans

 def __getattribute__(self, name):
        value = getattr(settings, name, None)
        if value:
            return value

Je mettrais dans la condition plutôt: if value is None pour permettre dans le settings mettre des chaînes vides.

Et plutôt:

return super(AppSettings, self).__getattribute__(name)

#12

Mis à jour par Josué Kouka il y a plus de 6 ans

Serghei Mihai a écrit :

[...]

Je mettrais dans la condition plutôt: if value is None pour permettre dans le settings mettre des chaînes vides.

Done.

Et plutôt:
[...]

Done.

#13

Mis à jour par Serghei Mihai il y a plus de 6 ans

Ack

#14

Mis à jour par Josué Kouka il y a plus de 6 ans

  • Statut changé de En cours à Résolu (à déployer)
  • % réalisé changé de 0 à 100
commit 1f4bd5d85d009048b1d05a8ed8a91f63c7809c35
Author: Josue Kouka <jkouka@entrouvert.com>
Date:   Mon Oct 30 18:38:48 2017 +0100

    allow override of app's settings (#19763)

#15

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

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF