Clear entire TinyDB when switching accounts. #578

Closed
opened 2 years ago by opyale · 4 comments
opyale commented 2 years 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 2 years 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 this to the Database-related work project 2 years ago
6543 added the
⚔️ Wontfix
label 9 months ago
Collaborator

-> #1034

we should just drop tinyDB if wa are at that point ...

-> #1034 we should just drop tinyDB if wa are at that point ...
6543 closed this issue 9 months ago
mmarif referenced this issue from a commit 9 months ago
Sign in to join this conversation.
No Milestone
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: gitnex/GitNex#578
Loading…
There is no content yet.