separate account from other data for backup/restore #13

Merged
anomalist merged 5 commits from separate-dbs into master 2 years ago
Owner

Separating the account from other data for a few reasons:

  • Long-term, the signing activity will be done by a separate function, even a different app.

  • When backing-up & restoring, dexie-export-import exports the account unencrypted, and I feel the sensitive identifier data needs to be stored externally more securely than the rest of the data. (It's possible to filter out the identity & seed info, but then the table export has 0 elements so an import will erase the data.) This is one of the main reasons I took this action today.

  • For internal signing, the private data should be exposed as little as possible; this will allow us to open and close the DB separately for the signing activities.

Separating the account from other data for a few reasons: * Long-term, the signing activity will be done by a separate function, even a different app. * When backing-up & restoring, dexie-export-import exports the account unencrypted, and I feel the sensitive identifier data needs to be stored externally more securely than the rest of the data. (It's possible to filter out the identity & seed info, but then the table export has 0 elements so an import will erase the data.) This is one of the main reasons I took this action today. * For internal signing, the private data should be exposed as little as possible; this will allow us to open and close the DB separately for the signing activities.
trentlarson added 1 commit 2 years ago
trentlarson requested review from anomalist 2 years ago
trentlarson added 1 commit 2 years ago
trentlarson added 1 commit 2 years ago
trentlarson added 1 commit 2 years ago
trentlarson added 1 commit 2 years ago
anomalist merged commit c47d6d8ae4 into master 2 years ago
Poster
Owner

Hm... I didn't see a notification that this was merged.

Hm... I didn't see a notification that this was merged.

Reviewers

anomalist was requested for review 2 years ago
The pull request has been merged as c47d6d8ae4.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
1 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.