Clear entire TinyDB when switching accounts. #578

Open
opened 12 months ago by opyale · 3 comments
opyale commented 12 months ago
Collaborator

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
Security
labels 12 months ago
Poster
Collaborator

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.
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?
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.
opyale added a new dependency 2 months ago
opyale added this to the Database-related work project 3 days ago
Sign in to join this conversation.
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Loading…
There is no content yet.