Development #19763
permettre la surcharge dans le settings des paramètres d'une "application"
100%
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
Historique
Mis à jour par Josué Kouka il y a plus de 6 ans
- Statut changé de Nouveau à En cours
- Assigné à mis à Josué Kouka
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
Mis à jour par Josué Kouka il y a plus de 6 ans
- Fichier 0001-add-parameter-to-override-default-login-path-19763.patch 0001-add-parameter-to-override-default-login-path-19763.patch ajouté
- Patch proposed changé de Non à Oui
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
.
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.
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.
Mis à jour par Josué Kouka il y a plus de 6 ans
- Fichier 0001-allow-override-of-app-s-settings-19763.patch 0001-allow-override-of-app-s-settings-19763.patch ajouté
Ok. Un patch qui permet de surcharger les parametre d'une application.
Mis à jour par Josué Kouka il y a plus de 6 ans
- Fichier 0001-allow-override-of-app-s-settings-19763.patch 0001-allow-override-of-app-s-settings-19763.patch ajouté
Nouveau avec le chemin absolu du auth_checker
calculé lors de la génération des infos de login.
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)
Mis à jour par Josué Kouka il y a plus de 6 ans
- Fichier 0001-allow-override-of-app-s-settings-19763.patch 0001-allow-override-of-app-s-settings-19763.patch ajouté
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.
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)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Fermé
allow override of app's settings (#19763)