Page MenuHomePhabricator
Differential D12350

[identity] Forbid registering multiple devices with the same ID
ClosedPublic
Actions

Authored by bartek on Fri, Jun 7, 3:59 AM.
Tags
None
Referenced Files
F2080526: D12350.diff
Sat, Jun 22, 7:44 PM
Unknown Object (File)
Fri, Jun 21, 6:23 AM
Unknown Object (File)
Fri, Jun 21, 6:23 AM
Unknown Object (File)
Fri, Jun 21, 6:23 AM
Unknown Object (File)
Sun, Jun 16, 10:40 PM
Unknown Object (File)
Fri, Jun 14, 11:19 AM
Unknown Object (File)
Fri, Jun 14, 8:01 AM
Unknown Object (File)
Fri, Jun 14, 7:29 AM
View All 13 Files
Subscribers
ashoat
tomek

Details

Reviewers
varun
will
Commits
rCOMM19818e9af187: [identity] Forbid registering multiple devices with the same ID
Summary

Addresses ENG-8383.
We don't want to allow registering multiple users with the same device ID.
However, if given device ID belongs to the same user, we can allow that (details in Linear task).

Depends on D12349

Test Plan

Commtest: called register_user_device() twice with the same device keys. Second call failed due to Status::AlreadyExists

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bartek created this revision.Fri, Jun 7, 3:59 AM
bartek held this revision as a draft.
Herald added subscribers: tomek, ashoat. · View Herald TranscriptFri, Jun 7, 3:59 AM
Harbormaster failed remote builds in B29524: Diff 41097!Fri, Jun 7, 4:04 AM
Harbormaster completed remote builds in B29524: Diff 41097.Sun, Jun 9, 11:29 PM
bartek published this revision for review.Sun, Jun 9, 11:52 PM
bartek added 1 blocking reviewer(s): varun.
varun accepted this revision.Wed, Jun 12, 8:56 AM
This revision is now accepted and ready to land.Wed, Jun 12, 8:56 AM
varun added a comment.Wed, Jun 12, 9:04 AM

in the login RPCs, don't we need to check that the device ID doesn't belong to a different user if it already exists?

varun requested changes to this revision.Wed, Jun 12, 9:05 AM
This revision now requires changes to proceed.Wed, Jun 12, 9:05 AM
bartek updated this revision to Diff 41366.Mon, Jun 17, 2:51 AM

Verify if device ID belongs to the same user

Harbormaster completed remote builds in B29706: Diff 41366.Mon, Jun 17, 3:08 AM
bartek added inline comments.Mon, Jun 17, 7:48 AM
services/identity/src/client_service.rs
1107 ↗(On Diff #41366)

I'm going to rebase on @varun's stack (when it's landed) before landing this and add a constant for this message

varun added inline comments.Mon, Jun 17, 7:56 AM
services/identity/src/client_service.rs
1107 ↗(On Diff #41366)

thanks, will land my stack ASAP

varun accepted this revision.Mon, Jun 17, 10:26 AM
This revision is now accepted and ready to land.Mon, Jun 17, 10:26 AM
bartek updated this revision to Diff 41432.Mon, Jun 17, 11:56 PM

Add constant for Tonic status code

Harbormaster completed remote builds in B29737: Diff 41432.Tue, Jun 18, 12:11 AM
Closed by commit rCOMM19818e9af187: [identity] Forbid registering multiple devices with the same ID (authored by bartek). · Explain WhyTue, Jun 18, 2:18 AM
This revision was automatically updated to reflect the committed changes.
bartek added a commit: rCOMM19818e9af187: [identity] Forbid registering multiple devices with the same ID.

Revision Contents
Changeset List

PathSize
services/
identity/
src/
63 lines
1 line
database/
22 lines

Diff 41434

View Options

services/identity/src/client_service.rs

Loading...
View Options

services/identity/src/constants.rs

Loading...
View Options

services/identity/src/database/device_list.rs

Loading...