HomePhabricator
Diffusion Comm dca857f6ff67

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

Description

[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.

Reviewers: tomek, marcin

Reviewed By: marcin

Subscribers: ashoat

Differential Revision: https://phab.comm.dev/D11721

Details

Provenance
kamilAuthored on Apr 22 2024, 4:00 PM
Reviewer
marcin
Differential Revision
D11721: [native/web] add SQL transaction to persisting crypto module
Parents
rCOMM9c2c1d2788de: [lib] update P2P message handler to use sequential decrypt
Branches
Unknown
Tags
Unknown