Page MenuHomePhabricator
Differential D11721

[native/web] add SQL transaction to persisting crypto module
ClosedPublic
Actions

Authored by kamil on Apr 22 2024, 4:01 PM.
Tags
None
Referenced Files
F3343579: D11721.id39372.diff
Fri, Nov 22, 3:21 AM
F3340910: D11721.diff
Thu, Nov 21, 10:13 PM
Unknown Object (File)
Fri, Nov 8, 5:12 PM
Unknown Object (File)
Tue, Nov 5, 3:41 PM
Unknown Object (File)
Wed, Oct 23, 7:24 PM
Unknown Object (File)
Wed, Oct 23, 7:24 PM
Unknown Object (File)
Wed, Oct 23, 7:24 PM
Unknown Object (File)
Wed, Oct 23, 7:21 PM
View All 54 Files
Subscribers
ashoat

Details

Reviewers
tomek
marcin
Commits
rCOMMdca857f6ff67: [native/web] add SQL transaction to persisting crypto module
Summary

Not sure why there was no transaction here, but I think it's better to add this. Mostly because it implies more than one operation (persisting two accounts in two separate calls).

Wondering what @marcin thinks about this change.

Depends on D11720

Test Plan

Play a bit with crypto-related API to make sure it still works.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kamil created this revision.Apr 22 2024, 4:01 PM
kamil held this revision as a draft.
Herald added a subscriber: ashoat. · View Herald TranscriptApr 22 2024, 4:01 PM
Harbormaster completed remote builds in B28378: Diff 39372.Apr 22 2024, 4:17 PM
kamil added a child revision: D11727: [CommCoreModule] decrypt and persist plaintext in SQL transaction.Apr 22 2024, 5:18 PM
kamil edited the summary of this revision. (Show Details)Apr 23 2024, 5:31 AM
kamil published this revision for review.Apr 23 2024, 6:38 AM
marcin added a comment.Apr 24 2024, 3:21 AM

I think it is good to add a transaction here as it will force accounts states in SQLite to be in sync.

marcin accepted this revision.Apr 24 2024, 3:21 AM
This revision is now accepted and ready to land.Apr 24 2024, 3:21 AM
kamil updated this revision to Diff 39614.Apr 29 2024, 12:51 AM

rebase before landing

Harbormaster completed remote builds in B28557: Diff 39614.Apr 29 2024, 1:07 AM
Closed by commit rCOMMdca857f6ff67: [native/web] add SQL transaction to persisting crypto module (authored by kamil). · Explain WhyApr 29 2024, 1:11 AM
This revision was automatically updated to reflect the committed changes.
kamil added a commit: rCOMMdca857f6ff67: [native/web] add SQL transaction to persisting crypto module.

Revision Contents
Changeset List

PathSize
native/
cpp/
CommonCpp/
NativeModules/
3 lines
web/
shared-worker/
worker/
3 lines

Diff 39619

View Options

native/cpp/CommonCpp/NativeModules/CommCoreModule.cpp

Loading...
View Options

web/shared-worker/worker/worker-crypto.js

Loading...