Paths

Table of Contentst

[identity] Make set_prekey() update devices table too
ClosedPublic
Actions

Authored by bartek on Dec 12 2023, 12:58 AM.

Details

Reviewers

varun
michal

Commits

rCOMM478280da1600: [identity] Make set_prekey() update devices table too

Summary

Updated DatabaseClient::set_prekey to also update the devices table with the new prekey (double-writing).

Depends on D10301

Test Plan

Called RefreshUserPreKeys RPC and checked with awslocal dynamodb scan that both users and devices table were updated correctly (with the same prekeys)

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

bartek created this revision.Dec 12 2023, 12:58 AM

bartek held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptDec 12 2023, 12:58 AM

bartek added a child revision: D10303: [identity] Augment get_keys_for_user to use devices table.Dec 12 2023, 1:03 AM

Harbormaster completed remote builds in B24973: Diff 34511.Dec 12 2023, 1:46 AM

bartek published this revision for review.Dec 12 2023, 3:20 AM

varun accepted this revision.Dec 19 2023, 8:28 PM

This revision is now accepted and ready to land.Dec 19 2023, 8:28 PM

bartek added a parent revision: D10221: [identity] Add devices to the new list after login and register.Jan 5 2024, 12:55 AM

bartek removed a parent revision: D10301: [identity] Add GetDeviceListForUser RPC.

Closed by commit rCOMM478280da1600: [identity] Make set_prekey() update devices table too (authored by bartek). · Explain WhyJan 15 2024, 6:53 AM

This revision was automatically updated to reflect the committed changes.

bartek added a commit: rCOMM478280da1600: [identity] Make set_prekey() update devices table too.

Revision Contents
Changeset List

Path

Size

services/

identity/

src/

database.rs

12 lines

database/

device_list.rs

46 lines

Diff 34511

View Options

services/identity/src/database.rs