Authenticator icon indicating copy to clipboard operation
Authenticator copied to clipboard

Preferences in private mode doesn't get saved on Firefox.

Open syrupderg opened this issue 2 years ago • 7 comments

Describe the issue

Authenticator doesn't applies the changes made to private mode, this wasn't a issue on Chromium based or other browsers.

Browser

Firefox

Browser Version

122.0

Extension Version

6.3.5

syrupderg avatar Jan 28 '24 18:01 syrupderg

See https://github.com/Authenticator-Extension/Authenticator/issues/1085#issuecomment-1731911650

Sneezry avatar Jan 29 '24 15:01 Sneezry

See #1085 (comment)

i forgot that i made that issue before but it is definitely a bug, other addons doesnt loses preferences but authenticator loses theme and and some of the other preferences, while still keeping the auth code and password that has been set. it works normal on normal tabs but on private tabs it uses the default preferences.

syrupderg avatar Jan 31 '24 21:01 syrupderg

Could you share some examples of other add-ons that manage to retain user preferences effectively? It would be really interesting to understand how they handle the storage of configuration settings, especially if they're not using local storage. This could give us some great insights.

Sneezry avatar Feb 01 '24 03:02 Sneezry

@Sneezry they likely use chrome.storage.*. We use localstorage for user preferences and chrome.storage for accounts.

mymindstorm avatar Feb 01 '24 04:02 mymindstorm

@mymindstorm We are transitioning from localStorage to chrome.storage.local in MV3, which will effectively resolve the issue.

Sneezry avatar Feb 01 '24 04:02 Sneezry

@Sneezry Could we add a version field to the storage to make future migrations easier? If that isn't already there.

mymindstorm avatar Feb 01 '24 05:02 mymindstorm

@mymindstorm Indeed, that's a logical approach. Let's design a schema for the data stored in the storage system, incorporating a version element and flexible attributes to accommodate future expansions.

Sneezry avatar Feb 01 '24 05:02 Sneezry