Ajout d'une page de gestion des sessions #6021

Situphen · 2021-02-04T19:51:51Z

Ajout d'une page de gestion des sessions avec la liste des sessions de l'utilisateur et la possibilité de supprimer les sessions autres que la session actuelle.

Ajout d'un intergiciel qui garde en mémoire pour chaque session l'adresse IP, le user agent et le timestamp actuels. Comme tout intergiciel, ce code s'exécute à chaque requête donc il est le plus minimaliste possible mais il faudra s'assurer que les performances des requêtes ne sont pas dégradées.
Utilisation d'un modèle de session personnalisé, hérité du modèle de session actuel (le modèle cached_db de Django), tel que proposé dans la documentation de Django. Cela ne devrait pas poser de soucis, mis à part le fait que tous les utilisateurs devront se reconnecter après la mise en production.

QA :

Vérifier le bon fonctionnement du site web
Se connecter depuis plusieurs appareils pour vérifier le bon fonctionnement de la page de gestion des sessions (accessible depuis les paramètres), notamment le fait de pouvoir déconnecter les autres sessions

coveralls · 2021-02-04T20:01:39Z

coverage: 88.674% (+0.02%) from 88.659%
when pulling 1845fd2 on Situphen:sessions
into db60cd7 on zestedesavoir:dev.

philippemilink

Il faut faire un petit rebase.

Est-ce que tu pourrais ajouter des tests, stp ? (ne serait-ce que pour s'assurer que les URLs que tu rajoutes peuvent être appelées correctement, pas forcément besoin de vérifier que ce qui est affiché est correct)

zds/middlewares/managesessionsmiddleware.py

Situphen

Modifications :

if user is not None and user.is_authenticated
if session_key and session_key != self.request.session.session_key
ajout des tests

philippemilink

Je viens de faire une revue de code un peu plus poussée (j'ai même été un peu tatillon). Une fois que les points que j'ai soulevés seront corrigés, je testerai, mais ça me semble déjà pas mal.

requirements.txt

templates/member/settings/sessions.html

zds/member/views/sessions.py

zds/middlewares/managesessionsmiddleware.py

AmauryCarrade · 2024-01-17T10:58:09Z

Hello! Est-ce que cette PR est toujours d'actualité, et est-ce que de l'aide serait souhaitée en plus de la future QA ?

philippemilink · 2024-01-17T19:10:11Z

Pour moi (mais je ne suis pas l'auteur), cette PR est toujours d'actualité, il faut seulement prendre en compte mes remarques :)

Situphen · 2024-01-17T20:01:53Z

Je confirme que cette PR est encore d'actualité mais je n'ai pas la motivation pour me remettre dedans en ce moment. @AmauryCarrade Si tu as envie de la reprendre, n'hésites pas, je peux te donner les droits sur ma branche ou tu peux créer une nouvelle PR. Il faut simplement me prévenir histoire que ça ne coincide pas avec une reprise de la motivation de mon côté :D

Situphen · 2024-03-09T23:58:04Z

Bon, j'ai eu un élan de motivation donc j'ai pu mettre à jour cette PR et prendre en compte les différentes remarques !

philippemilink

QA OK ✔️

Ce paramètre possède la même valeur que sa définition dans zds/settings/abstract_base/django.py, quelque soit l'environnement d'exécution. Fait suite à la PR zestedesavoir#6021

Ce paramètre possède la même valeur que sa définition dans zds/settings/abstract_base/django.py, quelque soit l'environnement d'exécution. Fait suite à la PR #6021

Bug introduit par 2b2215e (PR zestedesavoir#6021), où le backend pour gérer les sessions passe de django.contrib.sessions.backends.cached_db à zds.utils.custom_cached_db_backend, ce qui change la table utilisée pour stocker les sessions et le modèle à utiliser pour les manipuler. La vue qui compte le nombre de sessions pour Munin n'a pas été mise à jour utiliser le nouveau modèle. - Utilise les classes de sessions définies dans les paramètres, plutôt que forcément django.contrib.sessions - Ajoute un test pour vérifier que le nombre de sessions rapporté par Munin est correct - Renomme l'import DBStore en CachedDBStore pour éviter une confusion sur quel backend est utilisé pour stocker les sessions

Le changement dans les sessions introduit par 2b2215e (PR zestedesavoir#6021) utilise une nouvelle table utils_customsession pour stocker les sessions, tout en conservant la table django_session.

Bug introduit par 2b2215e (PR zestedesavoir#6021), où le backend pour gérer les sessions passe de django.contrib.sessions.backends.cached_db à zds.utils.custom_cached_db_backend, ce qui change la table utilisée pour stocker les sessions et le modèle à utiliser pour les manipuler. La vue qui compte le nombre de sessions pour Munin n'a pas été mise à jour utiliser le nouveau modèle. - Utilise les classes de sessions définies dans les paramètres, plutôt que forcément django.contrib.sessions - Ajoute un test pour vérifier que le nombre de sessions rapporté par Munin est correct - Renomme l'import DBStore en CachedDBStore pour éviter une confusion sur quel backend est utilisé pour stocker les sessions

Bug introduit par 2b2215e (PR #6021), où le backend pour gérer les sessions passe de django.contrib.sessions.backends.cached_db à zds.utils.custom_cached_db_backend, ce qui change la table utilisée pour stocker les sessions et le modèle à utiliser pour les manipuler. La vue qui compte le nombre de sessions pour Munin n'a pas été mise à jour pour utiliser le nouveau modèle. - Utilise les classes de sessions définies dans les paramètres, plutôt que forcément django.contrib.sessions - Ajoute un test pour vérifier que le nombre de sessions rapporté par Munin est correct - Renomme l'import DBStore en CachedDBStore pour éviter une confusion sur quel backend est utilisé pour stocker les sessions

Situphen force-pushed the sessions branch from ed8e48a to f788411 Compare February 5, 2021 18:52

AmauryCarrade added C-Back Concerne le back-end Django S-Évolution Ajoute de nouvelles fonctionnalités labels Feb 8, 2021

AmauryCarrade mentioned this pull request Feb 19, 2021

Amélioration des statistiques des contenus en les traitant côté serveur #6030

Closed

Situphen force-pushed the sessions branch from f788411 to 1d4465a Compare September 14, 2021 22:09

Situphen force-pushed the sessions branch 2 times, most recently from cf1919a to 119c3d8 Compare October 12, 2022 11:46

Situphen force-pushed the sessions branch 2 times, most recently from 43b94c3 to a665806 Compare May 23, 2023 21:23

Situphen changed the title ~~Amélioration du fonctionnement des sessions~~ Ajout d'une page de gestion des sessions May 23, 2023

Situphen force-pushed the sessions branch from a665806 to daa56fa Compare May 23, 2023 22:04

Situphen marked this pull request as ready for review May 23, 2023 22:04

Situphen force-pushed the sessions branch from daa56fa to 2e0ca8d Compare May 24, 2023 18:28

philippemilink requested changes May 24, 2023

View reviewed changes

zds/middlewares/managesessionsmiddleware.py Outdated Show resolved Hide resolved

Situphen force-pushed the sessions branch 2 times, most recently from fe4f79d to f4de4cd Compare June 5, 2023 18:20

Situphen commented Jun 5, 2023

View reviewed changes

Situphen requested a review from philippemilink June 5, 2023 18:22

philippemilink requested changes Jun 8, 2023

View reviewed changes

Situphen force-pushed the sessions branch from f4de4cd to 05a8d2c Compare September 7, 2023 19:25

Situphen added the S-Zombie Ticket ou PR oubliée label Mar 9, 2024

Situphen force-pushed the sessions branch 3 times, most recently from 4ebfa0c to 5cd96a9 Compare March 9, 2024 23:29

Situphen removed the S-Zombie Ticket ou PR oubliée label Mar 9, 2024

Situphen force-pushed the sessions branch from 5cd96a9 to abc0238 Compare March 9, 2024 23:39

Situphen requested a review from philippemilink March 9, 2024 23:47

Ajout d'une page de gestion des sessions

1845fd2

Situphen force-pushed the sessions branch from abc0238 to 1845fd2 Compare March 9, 2024 23:48

philippemilink approved these changes Mar 10, 2024

View reviewed changes

philippemilink merged commit 2b2215e into zestedesavoir:dev Mar 10, 2024
12 checks passed

Situphen deleted the sessions branch March 10, 2024 13:55

philippemilink mentioned this pull request Mar 10, 2024

Supprime SESSION_ENGINE de zds/settings/prod.py #6595

Merged

philippemilink mentioned this pull request Aug 25, 2024

Corrige le nombre de sessions rapporté par Munin #6636

Merged

philippemilink mentioned this pull request Aug 25, 2024

Ajoute un commentaire décrivant les tables utilisées pour les sessions #6637

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Ajout d'une page de gestion des sessions #6021

Ajout d'une page de gestion des sessions #6021

Situphen commented Feb 4, 2021 •

edited

Loading

coveralls commented Feb 4, 2021 •

edited

Loading

philippemilink left a comment

Situphen left a comment

philippemilink left a comment

AmauryCarrade commented Jan 17, 2024

philippemilink commented Jan 17, 2024

Situphen commented Jan 17, 2024

Situphen commented Mar 9, 2024

philippemilink left a comment

Ajout d'une page de gestion des sessions #6021

Ajout d'une page de gestion des sessions #6021

Conversation

Situphen commented Feb 4, 2021 • edited Loading

coveralls commented Feb 4, 2021 • edited Loading

philippemilink left a comment

Choose a reason for hiding this comment

Situphen left a comment

Choose a reason for hiding this comment

philippemilink left a comment

Choose a reason for hiding this comment

AmauryCarrade commented Jan 17, 2024

philippemilink commented Jan 17, 2024

Situphen commented Jan 17, 2024

Situphen commented Mar 9, 2024

philippemilink left a comment

Choose a reason for hiding this comment

Situphen commented Feb 4, 2021 •

edited

Loading

coveralls commented Feb 4, 2021 •

edited

Loading