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

Nouveau code de récupération (juste généré sur Android) non accepté par le client web #884

Closed
NicolasBuquet opened this issue Feb 5, 2024 · 13 comments
Assignees
Labels
bug Something isn't working P1 Priority 1 web

Comments

@NicolasBuquet
Copy link

NicolasBuquet commented Feb 5, 2024

pour reproduire

  • connecter et vérifier un appareil Android
  • connecter et vérifier un appareil Web
  • se déconnecter de Web
  • se connecter à Web
  • renouveller le code de recup Android
  • rentrer ce code sur Web

Description

Ce matin (05/02/2024) Delphine était déconnectée de son Tchap web sur Chrome (hypothèse : nettoyage des local data par manque de place sur l'ordinateur ?).

Au login, le client web lui demande son code de récupération pour accéder à ses messages.

Elle le régénère sur Android et le copie sur web.

Ce dernier accepte le code mais le redemande ensuite en boucle.

Il semble en fait que, bien qu'il le redemande, la première saisie ait été acceptée, car les messages finissent par se déverrouiller malgré le message d'erreur.

Screen.Recording.2024-02-05.at.10.11.09.11.mp4
Screenshot 2024-02-05 at 10 16 04 Screenshot 2024-02-05 at 10 16 58
@NicolasBuquet NicolasBuquet added P1 Priority 1 bug Something isn't working web labels Feb 5, 2024
@odelcroi
Copy link
Member

odelcroi commented Feb 5, 2024

lié avec tchapgouv/tchap-ios#951 ?

@odelcroi
Copy link
Member

odelcroi commented Feb 5, 2024

Merci pour le ticket,
est ce que cette FAQ a résolu le problème : https://aide.tchap.beta.gouv.fr/fr/article/mon-code-de-recuperation-est-refuse-b7suxq/

@Delphgateau
Copy link

Delphgateau commented Feb 6, 2024

@odelcroi non, on a tenté en live de déconnecter la session ayant besoin du Code de Récupération mais cela n'a rien changé. Pour rappel ma session mobile est sur Android

@odelcroi odelcroi moved this to Backlog in Roadmap Produit Feb 6, 2024
@odelcroi odelcroi moved this from Backlog to prêt pour le sprint in Roadmap Produit Feb 6, 2024
@yostyle
Copy link

yostyle commented Feb 6, 2024

Ajouter les infos de debug

@odelcroi
Copy link
Member

odelcroi commented Feb 8, 2024

@Delphgateau tu utilises quel URL de tchap web et quelle version de l'appli Android?

@Delphgateau
Copy link

web : https://app-v4.prod.lab.tchap.incubateur.net/
mobile : 2.10.0

@Delphgateau
Copy link

@estellecomment
Copy link
Contributor

Dans notre dernière upgrade massive (14 versions youhou), ya un bug similaire (mais pas clairement pareil) qui a été fixé :
element-hq/element-web#26321

Je vais tester si je peux le reproduire post-mega-upgrade.

@estellecomment estellecomment moved this from prêt pour le sprint to sprint en cours - WIP in Roadmap Produit Feb 20, 2024
@estellecomment estellecomment moved this from sprint en cours - WIP to prêt pour le sprint in Roadmap Produit Feb 20, 2024
@estellecomment
Copy link
Contributor

estellecomment commented Mar 11, 2024

Login avec email et mot de passe. On arrive ici (SetupEncryptionBody):

Screen Shot 2024-03-11 at 3 36 03 PM

A ce stade, deux flows possibles :

FLOW 1 qui fail 🟥

  • a. click "Verifier avec un code de recuperation" : ca ouvre le dialogue suivant pour input le code (AccessSecretStorageDialog)
Screen Shot 2024-03-11 at 3 38 15 PM
  • b. changer le code de recup sur android
  • c. input le code

On obtient :
Screen Shot 2024-03-11 at 3 39 16 PM

On est coincé, "Continuer" est bloqué.

FLOW 2 : presque pareil mais dans le desordre, et qui marche 🟩

  • b. changer le code de recup sur android
  • a. click "Verifier avec un code de recuperation" pour ouvrir AccessSecretStorageDialog
  • c. input le code

On obtient :
Screen Shot 2024-03-11 at 3 42 17 PM

Et on peut continuer, les messages sont bien decryptes.

Note : apres flow 1, on peut annuler, et du coup on se retrouve dans flow 2 : le code est deja créé, on ouvre AccessSecretStorageDialog, on input le code, et c'est bon.

--> Donc il y a un workaround simple : cliquer "Revenir en arriere" et recommencer : "Verifier avec un code de recuperation"

@estellecomment
Copy link
Contributor

Il y a sans doute qq chose qui est initialisé quand on clique "Vérifier avec un Code de Récupération" ("encryption|verification|verify_using_key"), et qui n'est pas refreshed quand on input le code, donc qui n'a pas le nouveau code.
Je vais voir si on peut reproduire sur element-web, pour jeter cette balle de l'autre coté de la barrière :)

@estellecomment
Copy link
Contributor

Filed in element-web : element-hq/element-web#27155

@estellecomment estellecomment moved this from prêt pour le sprint to sprint en cours - WIP in Roadmap Produit Mar 12, 2024
@odelcroi
Copy link
Member

odelcroi commented Mar 18, 2024

@estellecomment @MarcWadai De notre coté, mentionner ce workaround dans nos FAQ clot cette tache ?

@estellecomment estellecomment self-assigned this Mar 19, 2024
@estellecomment
Copy link
Contributor

Trop difficile de suivre le fil d'execution du code sans documentation pour comprendre comment il est organisé. J'abandonne.

Du coup, pour compenser, FAQ mise à jour : https://aide.tchap.beta.gouv.fr/fr/article/mon-code-de-recuperation-est-refuse-b7suxq/

@github-project-automation github-project-automation bot moved this from sprint en cours - WIP to déployé en prod in Roadmap Produit Mar 21, 2024
@MarcWadai MarcWadai moved this from déployé en prod to done in Roadmap Produit May 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working P1 Priority 1 web
Projects
Archived in project
Development

No branches or pull requests

5 participants