Page MenuHomePhabricator

[identity] Forbid duplicated devices in device lists
ClosedPublic

Authored by bartek on Jul 25 2024, 10:19 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Oct 22, 9:20 AM
Unknown Object (File)
Mon, Oct 21, 7:33 PM
Unknown Object (File)
Wed, Oct 16, 7:36 PM
Unknown Object (File)
Tue, Oct 15, 6:27 PM
Unknown Object (File)
Sat, Oct 12, 4:17 AM
Unknown Object (File)
Sep 25 2024, 5:15 PM
Unknown Object (File)
Sep 25 2024, 5:15 PM
Unknown Object (File)
Sep 25 2024, 5:15 PM
Subscribers

Details

Summary

Addresses ENG-8898.
Added a condition to check if the device list udpate contains duplicates.

Test Plan

Added unit test

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

bartek held this revision as a draft.
bartek published this revision for review.Jul 25 2024, 10:27 PM
bartek added inline comments.
services/identity/src/device_list.rs
331–332

copied() is only needed to convert &&str into &str. It's cheap

This revision is now accepted and ready to land.Jul 26 2024, 8:14 AM
This revision was landed with ongoing or failed builds.Jul 29 2024, 12:16 AM
This revision was automatically updated to reflect the committed changes.