Skip to content

Conversation

@P-Jeremy
Copy link
Contributor

@P-Jeremy P-Jeremy commented Jun 23, 2025

🔆 Problème

La PR #12387 permet d’afficher à l’utilisateur le nombre de tentatives restantes avant que son compte soit bloqué définitivement. Ce message s’affiche après le blocage temporaire de 4 minutes qui se produit à la 20ème tentatives. Or le mécanisme ne fonctionne pas.

⛱️ Proposition

Réparer l'affichage

🏄 Pour tester

Tentatives de connexion avec email sur Pix App, utiliser un email et un mauvais mot de passe:

  • 10 premières tentatives : L'adresse e-mail ou l'identifiant et/ou le mot de passe saisis sont incorrects.
  • À la 11ᵉ tentative : Vous avez effectué trop de tentatives de connexion, votre compte Pix est bloqué temporairement pendant 2 minutes. Réessayez plus tard ou cliquez sur mot de passe oublié pour le réinitialiser.
  • 10 tentatives suivantes : L'adresse e-mail ou l'identifiant et/ou le mot de passe saisis sont incorrects.
  • À la 21ᵉ tentative : Vous avez effectué trop de tentatives de connexion, votre compte Pix est bloqué temporairement pendant 4 minutes. Réessayez plus tard ou cliquez sur mot de passe oublié pour le réinitialiser.
  • 10 tentatives suivantes : Attention : il vous reste tentatives avant le blocage définitif de votre compte Pix.
  • À la 31ᵉ tentative : Votre compte est bloqué car vous avez effectué trop de tentatives de connexion. Pour le débloquer, contactez-nous.

Tentatives de connexion avec email sur Pix App, utiliser un username et un mauvais mot de passe:

  • 10 premières tentatives : L'identifiant et/ou le mot de passe saisis sont incorrects. En cas d’oubli, contactez votre enseignant pour récupérer votre identifiant et/ou réinitialiser votre mot de passe.
  • À la 11ᵉ tentative : Vous avez effectué trop de tentatives de connexion, votre compte Pix est bloqué temporairement pendant 2 minutes. En cas d’oubli, contactez votre enseignant pour récupérer votre identifiant et/ou réinitialiser votre mot de passe.
  • 10 tentatives suivantes :L'identifiant et/ou le mot de passe saisis sont incorrects. En cas d’oubli, contactez votre enseignant pour récupérer votre identifiant et/ou réinitialiser votre mot de passe.
  • À la 21ᵉ tentative : Vous avez effectué trop de tentatives de connexion, votre compte Pix est bloqué temporairement pendant 4 minutes. En cas d’oubli, contactez votre enseignant pour récupérer votre identifiant et/ou réinitialiser votre mot de passe.
  • 10 tentatives suivantes : Attention : il vous reste tentatives avant le blocage définitif de votre compte Pix. En cas d’oubli, contactez votre enseignant pour récupérer votre identifiant et/ou réinitialiser votre mot de passe.
  • À la 31ᵉ tentative : Votre compte est bloqué car vous avez effectué trop de tentatives de connexion. Pour le débloquer, contactez-nous.

Tester de la même manière pour le cas d’une authentification OIDC et pour le cas d’une authentification SAML

Se connecter avec un email et mot de passe corrects pour vérifier la non régression.

Se connecter avec un username et mot de passe corrects pour vérifier la non régression.

@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 :

@P-Jeremy P-Jeremy force-pushed the pix-18405-remaining-login-attemps-bug-fix branch from 90554b3 to e9e0dc9 Compare June 23, 2025 14:20
@P-Jeremy P-Jeremy added 👀 Tech Review Needed 👀 Func Review Needed Need PO validation for this functionally and removed Development in progress labels Jun 23, 2025
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 🦊

@P-Jeremy P-Jeremy force-pushed the pix-18405-remaining-login-attemps-bug-fix branch from e9e0dc9 to 90d1231 Compare June 24, 2025 07:47
@P-Jeremy P-Jeremy added Tech Review OK Func Review OK PO validated functionally the PR 🚀 Ready to Merge and removed 👀 Tech Review Needed 👀 Func Review Needed Need PO validation for this functionally labels Jun 24, 2025
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-18405-remaining-login-attemps-bug-fix branch from 90d1231 to 536f054 Compare June 24, 2025 12:51
@pix-service-auto-merge pix-service-auto-merge merged commit 4f73df4 into dev Jun 24, 2025
10 of 11 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-18405-remaining-login-attemps-bug-fix branch June 24, 2025 12:57
pix-service pushed a commit that referenced this pull request Jun 25, 2025
# [5.143.0](v5.142.0...v5.143.0) (2025-06-25)

### 🚀 Amélioration

- [#12604](#12604) Afficher une modale avant de quitter la page de fin de parcours (PIX-18023)
- [#12625](#12625) Ajouter un filtre sur l'api maddo pour remonter les dernières participations mise à jour (PIX-18288)
- [#12626](#12626) ajouter une route PATCH users/{id} pour enrichir un utilisateur anonyme avec des données d'authentification (Pix-18022)
- [#12639](#12639) Détacher les profile-rewards de l'organisation lors de la supression de learner (PIX-18113).
- [#12640](#12640) Lister par défaut les feature-toggles.
- [#12619](#12619) Repenser l'affichage des niveaux sur la page d'analyse de Pix Orga (PIX-18337)
- [#12635](#12635) v2 du module tmp-ia-fonctionnement-debut pour revue PYO

### 🐛 Correction

- [#12632](#12632) Corriger une typo dans le message de MER
- [#12528](#12528) Mettre à jour les informations d'une invitation à rejoindre une organisation lors d'un renvoie avec de nouveaux paramètres (PIX-18065)
- [#12621](#12621) Réparer l'affichage du nombre de tentatives de connexions restantes (PIX-18405)

### 🏗️ Tech

- [#12627](#12627) Bump audit-logger dependencies
- [#12568](#12568) Montée de version EmberJS pour PixAdmin

### ☕ Autre

- [#12638](#12638) Corriger l'exemple de titre de PR
@pix-service
Copy link
Contributor

🎉 This PR is included in version 5.143.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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.

7 participants