Redirection des agents qui se déconnectent vers la page de connexion #4832
Conversation
| @@ -17,7 +17,7 @@ def destroy | |||
|
|
|||
| agent_connect_client = AgentConnectOpenIdClient::Logout.new(session.delete(:agent_connect_id_token)) | |||
|
|
|||
| redirect_to agent_connect_client.agent_connect_logout_url(root_url), allow_other_host: true | |||
| redirect_to agent_connect_client.agent_connect_logout_url(new_agent_session_url), allow_other_host: true | |||
There was a problem hiding this comment.
malheureusement, les url de redirections après la déconnexion pro connect (encore appelé agent connect dans le code) sont whitelisté dans notre configuration pro connect faite avec leur équipe. Avec ce changement on casse ce fonctionnement, et on va rediriger l'agent vers une page d'erreur.
J'ai eu le même soucis dans la pr sur l'oauth, et j'ai mis en place une solution alternative.
Je pense qu'on pourrait à minima refactorer #agent_connect_logout_url pour que la root_url soit hardcodée dans la méthode avec un commentaire explicatif pour éviter que quelqu'un d'autre essaye de faire la même modif dans 6 mois. Je peux m'en occuper si tu veux
There was a problem hiding this comment.
Merci @victormours pour ton retour
Je n’avais pas cet élément effectivement
Est-ce qu’on peut passer des params dans l’URL ? Parce qu’une solution alternative serait de rediriger vers le root_url avec un param pour rediriger vers le login
Pas certain que ça soit hyper propre, et il faudrait qu’on regarde si ça vaut vraiment le coup niveau produit.
Dans tous les cas, je suis preneur de ta proposition pour éviter ça effectivement.
There was a problem hiding this comment.
je suis chaud qu'on en parle de vive voix, peut-être que le param ça pourrait être utile effectivement
There was a problem hiding this comment.
discuté ensemble : je vais préparer un nouvel endpoint post_redirection pour éviter d'avoir à faire des bricolages, et on pourra demander à proconnect de changer l'url une fois que ça sera en prod.
A ce moment, on pourra rouvrir une version plus simple de cette pr
|
Voir #4832 (comment) |
Contexte
Durant la discussion sur le bug du calendrier avec Mehdi et Teo, nous nous sommes rendu compte que cela n’avait pas beaucoup de sens de rediriger les agents vers la page d’accueil.
Nous changeons donc cette logique pour rediriger vers la page de connexion après déconnexion.