Projet

Général

Profil

Development #25696

FranceConnect: pas de logout "complet"

Ajouté par Thomas Noël il y a plus de 5 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
16 août 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Dans root.py la fonction logout gère le SLO pour le cas SAML ; je pense qu'il il manque la même chose pour le cas FC, et donc l'implémentation n'est pas compatible avec les obligations FC.


Fichiers

Révisions associées

Révision 1b9092b1 (diff)
Ajouté par Benjamin Dauvergne il y a plus de 5 ans

implement FranceConnect logout (#25696)

Historique

#2

Mis à jour par Benjamin Dauvergne il y a plus de 5 ans

  • Assigné à mis à Benjamin Dauvergne
#3

Mis à jour par Benjamin Dauvergne il y a plus de 5 ans

Rien de transcendant.

#4

Mis à jour par Frédéric Péters il y a plus de 5 ans

assert resp['Location'].endswith('/ident/fc/logout')

Je comprends 0001 mais je m'en passerais bien, partout ailleurs dans les tests on regarde resp.location.

#5

Mis à jour par Benjamin Dauvergne il y a plus de 5 ans

Je ne connaissais pas resp.location, j'avais pris l'habitude de passer par resp['Location'] sur les projets Django, c'est retiré et les deux assert sont adaptés.

#6

Mis à jour par Thomas Noël il y a plus de 5 ans

Ça serait en réalité inutile mais juste pour la compréhension du truc, je commencerais bien le "if" par un test sur l'activation de la méthode 'fc' :

    if 'fc' in ident_methods and session.extra_user_variables and 'fc_sub' in session.extra_user_variables:
        ....

Ça m'a l'air ok pour le reste ; à pousser après jeudi soir.

#7

Mis à jour par Thomas Noël il y a plus de 5 ans

  • Statut changé de Solution proposée à Solution validée
#8

Mis à jour par Benjamin Dauvergne il y a plus de 5 ans

Ok, pour jeudi.

#9

Mis à jour par Benjamin Dauvergne il y a plus de 5 ans

Benjamin Dauvergne a écrit :

Ok, pour jeudi.

vendredi.

#10

Mis à jour par Thomas Noël il y a plus de 5 ans

  • Statut changé de Solution proposée à Solution validée

Ou samedi.

(ah ah ah)

#11

Mis à jour par Benjamin Dauvergne il y a plus de 5 ans

  • Tracker changé de Bug à Development
  • Statut changé de Solution validée à Résolu (à déployer)
commit 1b9092b1d09fbe99bf9259f7b00dab76e94a3789 (HEAD -> master, tag: v3.18, origin/master, origin/HEAD)
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Fri Oct 5 14:14:35 2018 +0200

    implement FranceConnect logout (#25696)

#12

Mis à jour par Frédéric Péters il y a plus de 5 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF