Page MenuHomePhabricator

[SQLite] add query to mark outbound P2P message as sent
ClosedPublic

Authored by kamil on May 16 2024, 5:28 AM.
Tags
None
Referenced Files
F3370516: D12071.id40593.diff
Tue, Nov 26, 3:05 AM
F3367945: D12071.diff
Mon, Nov 25, 5:26 PM
Unknown Object (File)
Fri, Nov 1, 1:45 PM
Unknown Object (File)
Oct 14 2024, 4:58 PM
Unknown Object (File)
Oct 14 2024, 4:58 PM
Unknown Object (File)
Oct 14 2024, 4:57 PM
Unknown Object (File)
Sep 23 2024, 10:11 AM
Unknown Object (File)
Sep 23 2024, 6:24 AM
Subscribers

Details

Summary

We want to mark message as sent to avoid sending this again. Attempt to send message should happen only for encrypted messages, and even if marking as sent fails this is not a problem - this message will be sent but message will be discarded due to not being able to decrypt the same message twice.

Using hardcoded constants in code - for later usage on the JS side I'll introduce type to make it safer.

Depends on D12070

Test Plan

Tests

Diff Detail

Repository
rCOMM Comm
Branch
publish-broadcasting
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

kamil held this revision as a draft.
kamil published this revision for review.May 17 2024, 3:22 AM
native/cpp/CommonCpp/DatabaseManagers/SQLiteQueryExecutor.cpp
2222–2224

Please use SQLiteDataConverters.h

2227–2232

You can remove those lines - SQLiteStatementWrapper destructor does it automatically.

This revision is now accepted and ready to land.May 23 2024, 8:42 AM