Produits Entr'ouvert » w.c.s.

Les tickets cités manquent de contexte, offrent juste une explication technique de ce que font les patchs, c'est pour répondre à quel problème ?

Frédéric Péters a écrit :

Les tickets cités manquent de contexte, offrent juste une explication technique de ce que font les patchs, c'est pour répondre à quel problème ?

Deux problèmes différents :
1. se trouver sur w.c.s. non connecté alors qu'une session existe sur authentic (ça arrive si la session coté w.c.s. expire avant celle d'authentic, ce qui est normalement garanti même), dans ce cas le cookie PASSIVE_TRIED_COOKIE était toujours présent empêchant de relancer un SSO passif (ce qui rallonge la session coté IdP, etc...),
2. se trouver connecté sur w.c.s avec un utilisateur différent de celui avec lequel on est connecté sur authentic (le SLO a foiré, le cookie sur authentic a expiré avant celui de w.c.s. pour une raison x ou y) ou alors qu'on est plus connecté sur authentic.

Ça corrige les deux problèmes, le code est quasi identique à celui du middleware qui fait ça dans django-melon.

Ce sont des choses remontés par des tickets GLC lors des tests de GLCPRO (ça concernait surtout combo mais le problème est présent sur w.c.s. aussi).

Pour référence le ticket #65431 parle d'une session ouverte sur le portail alors qu'il n'y en a plus sur authentic, mais le même comportement était observé sur w.c.s.

Rebasé.

Il y a cinq patchs attachés mais dans la branche il y a 4 commits ? (ok... c'est 0001 qui s'est perdu ici).

Le sujet du commit 0002 est bien trop long.

Dans ce qui semble un changement d'indentation se glisse au moins une modification (secure=request.scheme == 'https'), ça aiderait la relecture de pointer ce qu'il pourrait y avoir d'autre.

Dans 0003 idéalement les nouveaux attributs sont intégrés dans has_info().

Dans 0004 au-delà de la description donnée dans le sujet il y a un ignore_errors=True qui s'est glissé.

Frédéric Péters a écrit :

Le sujet du commit 0002 est bien trop long.

Ok, je revois ça.

Dans ce qui semble un changement d'indentation se glisse au moins une modification (secure=request.scheme 'https'), ça aiderait la relecture de pointer ce qu'il pourrait y avoir d'autre.

J'ai mergé 0003 dans 0002, les deux ne fonctionnent pas sans l'autre de toute façon et j'ai réduit le titre du commit et mis une description plus exhaustive des changements de comportement.

Le changement secure.request.scheme 'https' c'est pour que ça fonctionne dans les tests: comme webtest simule des requêtes http cookiejar refusait de servir les cookies.

Dans 0003 idéalement les nouveaux attributs sont intégrés dans has_info().

Ok je l'ajoute pour la forme, de fait sur un login il y a tellement de choses qui changent dans la session qu'elle est toujours sauvée (ce mécanisme has_info n'est pas terrible, un jour il faudra voir à faire un truc plus proche de ce que fait Django qui essaye de détecter les modifications).

Dans 0004 au-delà de la description donnée dans le sujet il y a un ignore_errors=True qui s'est glissé.

Ce changement est retiré, ça n'est pas/plus nécessaire, j'ai bataillé avec les cookies de session, c'est devenu #72613 (la branche est rebasée sur #72613).

+            OPENED_SESSION_COOKIE = get_publisher().get_site_option('idp_session_cookie_name')
+            if OPENED_SESSION_COOKIE:
+                if OPENED_SESSION_COOKIE in get_request().cookies:
+                    session.opened_session_value = get_request().cookies[OPENED_SESSION_COOKIE]

Tant qu'à avoir tout ce code qui change (ici et ailleurs), ça pourrait être l'occasion pour écrire en minuscules, comme des variables classiques ? (ça aurait en bonus de sans doute produire un diff plus lisible, qui n'intercalerait pas quelques lignes d'ancien code au milieu du nouveau).

Sur le fond j'ai un peu peur de la partie

  - if user is logged and IDP_OPENED_SESSION cookie value differs from
    the one saved in the Quixote session, user is logged out, if not
    treatment stop here.

mais elle existe dans django-mellon et ça semble passer là, donc ok.

Frédéric Péters (fpeters) a lié une pull request sur Gitea concernant cette demande :

• URL : https://gitea.entrouvert.org/entrouvert/wcs/pulls/14
• Titre : saml: create a new session if expired during passive SSO (#72898)
• Modifications : https://gitea.entrouvert.org/entrouvert/wcs/pulls/14/files

Frédéric Péters (fpeters) a mergé une pull request sur Gitea concernant cette demande :

• URL : https://gitea.entrouvert.org/entrouvert/wcs/pulls/14
• Titre : saml: create a new session if expired during passive SSO (#72898)
• Modifications : https://gitea.entrouvert.org/entrouvert/wcs/pulls/14/files

Automatic expiration