#578 Clear entire TinyDB when switching accounts.

Open
opened 1 month ago by opyale · 3 comments
opyale commented 1 month ago

This is important so that old and left over values do not possibly affect the functioning of another account.

This is important so that old and left over values do not possibly affect the functioning of another account.
opyale added the
Long-term
label 1 month ago
opyale added the
Security
label 1 month ago
opyale commented 1 month ago
Poster

A method for this is already implemented in TinyDB.

/**
 * Clear SharedPreferences (remove everything)
 */
public void clear() {
    preferences.edit().clear().apply();
}

It only needs to be called in the logout() method.

A method for this is already implemented in ``TinyDB``. ``` /** * Clear SharedPreferences (remove everything) */ public void clear() { preferences.edit().clear().apply(); } ``` It only needs to be called in the ``logout()`` method.
mmarif commented 1 month ago
Owner

We cant at this moment use this until all the data(e.g settings) are migrated to db for example.

Bdw what data would effect the functionality of the app in tinydb if remain there?

We cant at this moment use this until all the data(e.g settings) are migrated to db for example. Bdw what data would effect the functionality of the app in tinydb if remain there?
mmarif commented 1 month ago
Owner

And I do use the app(testing) with multiple instances and multiple accounts and never came across a single issue so far except a token bug(came across recently) which will be handeled later with multiple accounts setup.

And I do use the app(testing) with multiple instances and multiple accounts and never came across a single issue so far except a token bug(came across recently) which will be handeled later with multiple accounts setup.
Sign in to join this conversation.
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.