Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement key backup APIs for rust and create backup in bootstrapSecretStorage #3690

Merged
merged 7 commits into from
Sep 5, 2023

Conversation

BillCarsonFr
Copy link
Member

@BillCarsonFr BillCarsonFr commented Sep 1, 2023

Fixes https://github.com/vector-im/crypto-internal/issues/107
Based on #3689

Implements missing key backup API in rust and create backups as part of the bootstrap security process

Checklist

  • Tests written for new code (and old code if feasible)
  • Linter and other CI checks pass
  • Sign-off given on the changes (see CONTRIBUTING.md)

This change is marked as an internal change (Task), so will not be included in the changelog.

@BillCarsonFr BillCarsonFr added the T-Task Tasks for the team like planning label Sep 1, 2023
@BillCarsonFr BillCarsonFr changed the title Valere/element r/keybackup reset api rust Add missing KeyBackup API for rust and create backpup in bootstrap Sep 1, 2023
@BillCarsonFr BillCarsonFr force-pushed the valere/element-r/keybackup_reset_api_rust branch from 4d7b7b5 to dcdbbee Compare September 1, 2023 15:29
@BillCarsonFr BillCarsonFr marked this pull request as ready for review September 1, 2023 15:38
@BillCarsonFr BillCarsonFr requested a review from a team as a code owner September 1, 2023 15:38
@BillCarsonFr BillCarsonFr requested review from richvdh and weeman1337 and removed request for weeman1337 September 1, 2023 15:38
@BillCarsonFr BillCarsonFr force-pushed the valere/element-r/keybackup_reset_api_rust branch from dcdbbee to dc7a029 Compare September 4, 2023 08:14
@BillCarsonFr BillCarsonFr changed the base branch from develop to valere/element-r/keybackup_reset_api September 4, 2023 08:35
@richvdh richvdh changed the title Add missing KeyBackup API for rust and create backpup in bootstrap Add missing KeyBackup API for rust and create backup in bootstrap Sep 4, 2023
@richvdh richvdh changed the title Add missing KeyBackup API for rust and create backup in bootstrap Implement key backup APIs for rust and create backup in bootstrap Sep 4, 2023
src/rust-crypto/rust-crypto.ts Outdated Show resolved Hide resolved
src/rust-crypto/backup.ts Outdated Show resolved Hide resolved
src/rust-crypto/backup.ts Outdated Show resolved Hide resolved
spec/integ/crypto/crypto.spec.ts Outdated Show resolved Hide resolved
Comment on lines 2205 to 2209

if (content.key) {
resolve(content.key);
}

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not convinced this is an improvement, but ymmv

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not sure I touched that, I am a bit confused

spec/integ/crypto/crypto.spec.ts Outdated Show resolved Hide resolved
Base automatically changed from valere/element-r/keybackup_reset_api to develop September 4, 2023 20:13
@BillCarsonFr BillCarsonFr requested a review from richvdh September 5, 2023 12:14
@richvdh richvdh changed the title Implement key backup APIs for rust and create backup in bootstrap Implement key backup APIs for rust and create backup in bootstrapSecretStorage Sep 5, 2023
Copy link
Member

@richvdh richvdh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@BillCarsonFr BillCarsonFr added this pull request to the merge queue Sep 5, 2023
Merged via the queue into develop with commit d7831f9 Sep 5, 2023
28 checks passed
@BillCarsonFr BillCarsonFr deleted the valere/element-r/keybackup_reset_api_rust branch September 5, 2023 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T-Task Tasks for the team like planning
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants