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

L'activation de la sauvegarde automatique crash depuis au moins la v.2.8.1 #1125

Closed
NicolasBuquet opened this issue Nov 20, 2024 · 6 comments · Fixed by #1127
Closed

L'activation de la sauvegarde automatique crash depuis au moins la v.2.8.1 #1125

NicolasBuquet opened this issue Nov 20, 2024 · 6 comments · Fixed by #1127
Labels
bug Something isn't working P1

Comments

@NicolasBuquet
Copy link
Contributor

Après création d'un nouveau compte, activer la sauvegarde automatique par le réglage de sécurité ou par la pop-up le prpoposant après relancement de l'application crash.

La signature croisée est marquée comme activée sur iOS.

Sur le même compte sur web, elle est marquéé comme non-active.
simulator_screenshot_48C4733B-D4E8-4966-B3B0-65C83630EA1A
image

Le fait de la réactiver sur iOS semble marcher, mais en fait, ce n'est pas le cas.

Activer la sauvegarde automatique sur iOS alors que la signature croisée n'est pas activée entraîne un crash.

Actver la signature croisée depuis le web entraîne l'affichage de ce message 👍
image

Dans le message suivant, le bouton "Revenir en arrière" devrait être "Continuer" (c'est l'action qu'il a) :
image

Sur web, on présente l'option de saisir une phrase de sécurité !
image

Un crash sour iOS (test de Synapse 1.119 en PreProd) semble indiquer qu'il cherche aussi à activer la sauvegarde automatique en passant pas la passphrase !
La signature croisée s'active alors depuis le web :
image

L'activatiobn de la sauvegarde sécurisée sur iOS est alors possible, bien qu'elle plante à la fin du processus.

Au redémarrage, la signature croisée est activée sur iOS et la sauvegarde automatique aussi.
simulator_screenshot_C1E324D3-DAC8-4A87-8E59-70A7760244B1

Il est possible de la réinitialiser depuis iOS sans plantage.

Sur web, la signature croisée est marquée comme non-activée.
Vérifier la session web n'active pas la signature croisée.

image

@NicolasBuquet NicolasBuquet added bug Something isn't working P1 labels Nov 20, 2024
@NicolasBuquet
Copy link
Contributor Author

NicolasBuquet commented Nov 21, 2024

Testé le lendemain sur le m´me compte avec le web, Android et iOS :

web

image

Android

IMG_0605

iOS

simulator_screenshot_69EC0ED6-E74A-410D-B382-90625BED9E89

Les 3 clients ne formulent pas la même chose.
JJe pense que le web et iOS simplifient en disant que la signature croisée n'est pas exploitable alors qu'Android donne trop de détail.

Par contre, iOS semble donner une information fausse sur la sauvegarde automatique : elle n'est pas activée disent le web et Android, et iOS dit qu'elle est activée.

image

Par contre, il y a une sauvegarde de clés (en version 9) sur le compte. Je pense qu'iOS doit vouloir dire qu'une sauvegarde existe sur le compte.

@NicolasBuquet
Copy link
Contributor Author

NicolasBuquet commented Nov 21, 2024

C'est un crash du à Element.

Element iOS crash si on essaie de régénérer une sauvegarde sécurisée avec Code de récupération.

C'est du à un appel depuis Riot/Modules/Secrets/Setup/RecoveryKey/SecretsSetupRecoveryKeyViewModel.swift createSecureKey qui appelle dans le Matrix-iOS-SDK la méthode de Crypto/SecretStorage/MXSecretStorage :

(MXHTTPOperation*)createKeyWithKeyId:(nullable NSString*)keyId
                               keyName:(nullable NSString*)keyName
                            passphrase:(nullable NSString*)passphrase
                               success:(void (^)(MXSecretStorageKeyCreationInfo *keyCreationInfo))success
                               failure:(void (^)(NSError *error))failure

Hors, l'entête de cette méthode existe toujours dans le SDK, mais son implémentaiton n'existe plus !

https://github.com/matrix-org/matrix-ios-sdk/blob/34c66a0615173fb2d0252f8b9dc5d2650a58a847/MatrixSDK/Crypto/SecretStorage/MXSecretStorage.h#L92

@NicolasBuquet
Copy link
Contributor Author

Problème référencé chez Element : element-hq/element-ios#7871 (comment)

@Delphgateau
Copy link
Collaborator

@NicolasBuquet doublon avec le #1124 ? :)

@NicolasBuquet
Copy link
Contributor Author

NicolasBuquet commented Nov 25, 2024

@Delphgateau
En effet, j'ai vu ça ce matin.
Du coup, je l'ai référencé et fermé.

@NicolasBuquet
Copy link
Contributor Author

Element a merg´la correction dans la branche develop du matrix-ios-sddk que l'on utilise.
Ce sera donc ok pour la prochaine release.

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
Projects
Status: déployé en prod
Development

Successfully merging a pull request may close this issue.

2 participants