Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Garde les paramètres GET dans l'URL de redirection après la connexion #6711

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

philippemilink
Copy link
Member

Fix #6700

Il faut encore ajouter les tests.

Contrôle qualité

  1. Sans être connecté, aller sur la page http://127.0.0.1:8000/bibliotheque/?type=tutorial
  2. Cliquer sur le bouton pour accéder à la page de connexion
  3. L'URL contient (de façon encodée) ?next=/bibliotheque/%3Ftype%3Dtutorial
  4. Cliquer à nouveau sur le bouton pour accéder à la page de connexion : l'URL reste la même (il n'y a pas de récursion causée par le paramètre next)
  5. Remplir le formulaire de connexion
  6. On est alors redirigé vers la page http://127.0.0.1:8000/bibliotheque/?type=tutorial, le paramètre type=tutorial est bien conservé

Pour tester un cas potentiellement pathologique, on peut refaire la même séquence d'opération, mais en cherchant à l'aide du formulaire de recherche ?next=.

@coveralls
Copy link

coveralls commented Feb 6, 2025

Coverage Status

coverage: 89.214% (+0.004%) from 89.21%
when pulling eaf5d20 on philippemilink:keep-get-params-login-redirection
into 544dd75 on zestedesavoir:dev.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: En attente de QA
Development

Successfully merging this pull request may close these issues.

La redirection après la connexion oublie les paramètres GET
2 participants