beemdevelopment / Aegis

A free, secure and open source app for Android to manage your 2-step verification tokens.
https://getaegis.app
GNU General Public License v3.0
8.93k stars 375 forks source link

Regression as of 3.0: Group selection in view does not persist after closing #1329

Closed dimitris-personal closed 4 months ago

dimitris-personal commented 6 months ago

Version

v3.0

Source

Google Play

Vault encryption

Yes (with biometric unlock)

Device

Pixel 7 and Pixel 6A

Android version

Android 14 AP1A.240305.019.A1

ROM

No response

Steps to reproduce

What do you expect to happen?

Group selection was expected to be persistent.

What happens instead?

Group selection is cleared.

Log

No response

michaelschattgen commented 6 months ago

Thanks for your report. Interesting, I can't seem to reproduce this and I can't recall any changes regarding this feature.

dimitris-personal commented 6 months ago

@michaelschattgen please see updated description. Found a workaround, but this still seems to be a new sharp edge maybe worth addressing. Most users may not look at github to find this or think of biometric unlock as related.

Edit: Workaround also seems to have a (not so important) extra side effect.

alexbakker commented 6 months ago

Losing the previously saved group filter is expected behavior when updating to 3.0, but not being able to save a new group filter is not.

We've made some internal changes to groups as preparation to allow entries to be in multiple groups. In short: groups are no longer internally identified by their name, but by a UUID. Upon launching the app, Aegis will detect that your vault file is in the old format, and it will convert it to the new one. During this process, all groups will receive a random UUID. This happens in-memory and these changes are not written to disk until you make some other change (like enabling/disabling biometric unlock, as you discovered) that triggers a save of the vault. Changing the group filter is not an action that triggers a save of the vault, because the filter is stored elsewhere.

This causes trouble because after saving the group filter, it will contain a group UUID that will no longer exist after you relaunch the app.

andbenn commented 6 months ago

When I had this problem, toggling various groups trying to get it to save and stay on a restart, I noticed that at certain times, the back motion would take me back to a list of tokens, usually what I just had up and was trying to escape from. I thought this strange. Maybe some view of the original data. Since toggling biometrics, I'm back to being good. Thanks team for such a nice app!

RavenStryker commented 6 months ago

@michaelschattgen please see updated description. Found a workaround, but this still seems to be a new sharp edge maybe worth addressing. Most users may not look at github to find this or think of biometric unlock as related.

Edit: Workaround also seems to have a (not so important) extra side effect.

Thanks for this, I was wondering what was happening as the app looked different so I assumed it was an update. I can confirm I had the same issue and was also able to get it to work around that by toggling biometrics as well. Cheers!

pedroapero commented 6 months ago

In my case, after upgrading to 3.0, the display was all scrambled. Couldn't see anything on the settings menu. After uninstalling and re-installing (and re-importing my data); it is working fine (can't reproduce and I didn't take screenshot). I'm on Android 8.1.0.