Development #69301
marquer certaines cellules comme exigeant une session
Début:
20 septembre 2022
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
L'appel à {% make_public_url %} a besoin d'une session active, si ce n'est pas le cas il la créera, mais on peut avoir ça dans un chargement asynchrone de cellules et plusieurs créations parallèles avec comme résultat certains liens cassés (#69187#note-10).
Ajouter dans CellBase un attribut session_required = False, le mettre à True pour les cellules JSON et Fiche(s); et quand une des cellules de la page contient ça, faire la création de la session dans publish_page.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus d'un an
- Fichier 0001-misc-mark-some-cells-as-requiring-a-session-69301.patch 0001-misc-mark-some-cells-as-requiring-a-session-69301.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Assez trivial mais pas de test parce qu'a priori compliqué de simuler cette situation d'appels parallèles. Aussi j'ai préféré conserver le request.session.cycle_key() lors du make_public_url, au cas où.
Mis à jour par Lauréline Guérin il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
Mis à jour par Frédéric Péters il y a plus d'un an
- Statut changé de Solution validée à Résolu (à déployer)
commit 871742a41667d98172a365e3e8d64a5d87043125 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Tue Sep 20 08:02:54 2022 +0200 misc: mark some cells as requiring a session (#69301)
Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
misc: mark some cells as requiring a session (#69301)