Page MenuHomePhabricator

[lib] handle `DEVICE_LIST_UPDATED` message
ClosedPublic

Authored by kamil on Jun 10 2024, 8:52 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Nov 20, 2:40 PM
Unknown Object (File)
Wed, Nov 20, 2:40 PM
Unknown Object (File)
Sat, Nov 2, 12:01 AM
Unknown Object (File)
Sat, Nov 2, 12:01 AM
Unknown Object (File)
Sat, Nov 2, 12:00 AM
Unknown Object (File)
Fri, Nov 1, 11:40 PM
Unknown Object (File)
Oct 22 2024, 1:16 PM
Unknown Object (File)
Oct 22 2024, 9:04 AM
Subscribers

Details

Summary

We want to call getAndUpdateDeviceListsForUsers on each request, leaving logs as in D12347

Depends on D12378

Test Plan

Send this message to device and make sure it's processed

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.Jun 10 2024, 9:52 AM
lib/tunnelbroker/use-peer-to-peer-message-handler.js
192 ↗(On Diff #41167)

Noticed that you decided to call this even if !result.valid. Can you share some context around that decision, and what it means if !result.valid?

lib/tunnelbroker/use-peer-to-peer-message-handler.js
192 ↗(On Diff #41167)

This is to leave logs to check if the device list was properly signed by the primary device.

However, currently, there is no full device list support and we have to support legacy Identity auth (that's the case where result.valid = false). I can add a code comment to make it more clear.

This revision is now accepted and ready to land.Jun 13 2024, 3:15 AM
This revision was automatically updated to reflect the committed changes.