Bug #6547
Remplacer le middleware de settings mellon par un adapteur
0%
Description
Pour hobo la possibilité d'adapteur dans le code de mellon a été terminée, c'est ça qui devrait être utilisé pour les SP déployés aussi.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a environ 9 ans
- Fichier 0001-multitenant-use-an-adapter-for-mellon-authentication.patch 0001-multitenant-use-an-adapter-for-mellon-authentication.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Ça permet aussi d'avoir du code exécuté uniquement dans le codepath de mellon.
Les SP qui se basaient sur la présence du backend d'authent mellon pour voir s'il fallait faire du mellon, par contre, doivent être mis à jour, pour se baser sur get_idps; je viens de le faire côté combo :
http://repos.entrouvert.org/combo.git/commit/?id=97403e9ea1cfda3f64c7d5fac4f655d8079e4543
Mis à jour par Frédéric Péters il y a environ 9 ans
Ah oui, ça demande à ce que le StoreRequestsMiddleware soit installé.
Mis à jour par Jérôme Schneider il y a environ 9 ans
Si je comprends bien il faudra modifier les settings de nos paquets multitenant en :
- ajoutant l'option MELLON_ADAPTER :
MELLON_ADAPTER = ('hobo.multitenant.mellon.MellonAdapter',)
- ajoutant le middleware le StoreRequestsMiddleware (comme tu le précises au dessus)
- enlevant l'ancien middleware MellonSettingsMiddleware
Je n'oublie rien ?
A part ca le patch me semble OK mais je n'ai pas testé en vrai.
Mis à jour par Frédéric Péters il y a environ 9 ans
Ouaip, c'est tout à fait ça. (https://dev.entrouvert.org/projects/prod-eo/wiki/HowDoWeDoDjangoTenants/diff?version=2&version_from=1)
commit b31a4d20ecf4f1d832f520f05b44b88935cb6590 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Wed Feb 18 09:26:44 2015 +0100 multitenant: use an adapter for mellon authentication (#6547) This is better than meddling with the settings at runtime.
Mis à jour par Frédéric Péters il y a environ 9 ans
- Statut changé de En cours à Résolu (à déployer)
Mis à jour par Benjamin Dauvergne il y a plus de 8 ans
- Statut changé de Résolu (à déployer) à Fermé
multitenant: use an adapter for mellon authentication (#6547)
This is better than meddling with the settings at runtime.