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

[BUGFIX] Encoder l'adresse email présente dans l'URL (PIX-17205). #11852

Merged

Conversation

Libouk
Copy link
Member

@Libouk Libouk commented Mar 26, 2025

🌸 Problème

Je reçois mon e-mail de notification de reconnexion, je clique sur le bouton pour réinitialiser mon mot de passe, mon adresse e-mail est bien reportée mais le “+” n’est pas reporté.

Cette PR fait suite à #11753.

🌳 Proposition

Encoder le contenu du query parameter email.

🤧 Pour tester

  • Se créer un compte sur Pix App avec son adresse mail suivi d'un alias (ex: prenom.nom+alias@pix.fr)
  • Se déconnecter de Pix App
  • Se connecter à la base de données de la RA
  • Jouer les requêtes suivantes :
select * from users where "email"='prenom.nom+alias@pix.fr';
select * from "user-logins" where "userId"='votreUserId';
UPDATE "user-logins" SET "lastLoggedAt" = '2020/01/01' where id=votreUserLoginsId;
  • Se connecter à Pix App
  • Aller consulter ses mails pour retrouver pour y trouver le mail envoyé par Pix qui propose à l'utilisateur de réinitialiser son mot de passe
  • Cliquer sur ce lien, modifier https://app.pix.fr en https://app-pr11852.review.pix.fr/
  • Constater que le paramètre email=prenom.nom+alias@pix.fr est présent dans l'url
  • Constater que votre email a été automatiquement reporté dans le champ concerné avec le "+" conservé

@Libouk Libouk self-assigned this Mar 26, 2025
@Libouk Libouk requested a review from a team as a code owner March 26, 2025 11:15
@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@Libouk Libouk force-pushed the pix-17205-fix-email-reset-url-email-encoding branch from 2488a53 to 96f7c3c Compare March 26, 2025 14:08
Copy link
Contributor

@lego-technix lego-technix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Lu et testé fonctionnellement avec succès avec Firefox 🦊

@Libouk Libouk added 👀 Tech Review Needed 👀 Func Review Needed Need PO validation for this functionally and removed Development in progress labels Mar 26, 2025
Copy link
Contributor

@bpetetot bpetetot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Juste un petit commentaire ;)

@Libouk Libouk force-pushed the pix-17205-fix-email-reset-url-email-encoding branch from 96f7c3c to 9e6ca27 Compare March 27, 2025 08:49
@Libouk Libouk added Tech Review OK 🚀 Ready to Merge Func Review OK PO validated functionally the PR and removed 👀 Tech Review Needed 👀 Func Review Needed Need PO validation for this functionally labels Mar 27, 2025
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-17205-fix-email-reset-url-email-encoding branch from 9e6ca27 to 9f8760f Compare March 27, 2025 08:50
@pix-service-auto-merge pix-service-auto-merge merged commit f851118 into dev Mar 27, 2025
10 of 11 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-17205-fix-email-reset-url-email-encoding branch March 27, 2025 09:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants