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)
Mon, Dec 23, 6:57 PM
Unknown Object (File)
Mon, Dec 23, 6:57 PM
Unknown Object (File)
Mon, Dec 23, 6:57 PM
Unknown Object (File)
Mon, Dec 23, 6:57 PM
Unknown Object (File)
Mon, Dec 23, 6:57 PM
Unknown Object (File)
Mon, Dec 23, 6:57 PM
Unknown Object (File)
Mon, Dec 23, 8:39 AM
Unknown Object (File)
Mon, Dec 23, 7:42 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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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.