Page MenuHomePhabricator

[SQLite] implement C++ keyserver store ops
ClosedPublic

Authored by kamil on Jan 22 2024, 4:32 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Nov 21, 1:24 AM
Unknown Object (File)
Wed, Nov 6, 6:48 PM
Unknown Object (File)
Tue, Nov 5, 9:58 PM
Unknown Object (File)
Tue, Nov 5, 9:53 PM
Unknown Object (File)
Tue, Nov 5, 9:50 PM
Unknown Object (File)
Tue, Nov 5, 8:31 PM
Unknown Object (File)
Tue, Nov 5, 7:16 PM
Unknown Object (File)
Oct 24 2024, 1:43 AM
Subscribers

Details

Summary

Code for native ops

Depends on D10775

Test Plan

Call this code:

await commCoreModule.processKeyserverStoreOperations([
    {
      type: 'replace_keyserver',
      payload: {
        id: '1',
        keyserverInfo: 'fdfd',
      },
    },
    {
      type: 'replace_keyserver',
      payload: {
        id: '2',
        keyserverInfo: 'fdfd',
      },
    },
  ]);

  await commCoreModule.processKeyserverStoreOperations([
    {
      type: 'replace_keyserver',
      payload: {
        id: '2',
        keyserverInfo: 'fdfd45454545',
      },
    },
  ]);

  await commCoreModule.processKeyserverStoreOperations([
    {
      type: 'remove_keyserver',
      payload: {
        id: '1',
      },
    },
  ]);

  await commCoreModule.processKeyserverStoreOperations([
    {
      type: 'remove_all_keyservers',
    },
  ]);

Diff Detail

Repository
rCOMM Comm
Branch
land-ks-sql
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

kamil held this revision as a draft.
kamil published this revision for review.Jan 22 2024, 6:39 AM
ashoat added a subscriber: will.

Might be good for @will to review this stack in preparation for his projects next month

This revision is now accepted and ready to land.Jan 24 2024, 9:25 AM

update to allow removing array of keyservers

native/cpp/CommonCpp/DatabaseManagers/SQLiteQueryExecutor.cpp
1340 ↗(On Diff #36219)

this formatting is a rebase issue

This revision was automatically updated to reflect the committed changes.