Page MenuHomePhabricator

[backup-service] implement calling `FindUserID` RPC from Backup
ClosedPublic

Authored by kamil on Oct 29 2024, 8:41 AM.
Tags
None
Referenced Files
F3513313: D13806.id45662.diff
Sat, Dec 21, 11:34 PM
F3513312: D13806.id45602.diff
Sat, Dec 21, 11:34 PM
F3513311: D13806.id45427.diff
Sat, Dec 21, 11:34 PM
F3513293: D13806.id.diff
Sat, Dec 21, 11:33 PM
F3513288: D13806.diff
Sat, Dec 21, 11:33 PM
Unknown Object (File)
Wed, Dec 4, 1:22 AM
Unknown Object (File)
Tue, Dec 3, 8:02 PM
Unknown Object (File)
Mon, Dec 2, 6:19 PM
Subscribers

Details

Summary

ENG-6145.

This is similar to other places in the codebase (here or here), there is also alternative where this code could be places in comm-lib but I prefer this, we can move it if needed.

Using is_valid_ethereum_address should be safe, we do the same on the client, and also it's not possible to create username which id valid ETH address.

Depends on D13805

Test Plan

tested later in the stack

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.Oct 30 2024, 9:28 AM
varun added inline comments.
services/backup/src/error.rs
71 ↗(On Diff #45427)

what's this for?

This revision is now accepted and ready to land.Nov 1 2024, 1:40 PM
bartek added inline comments.
services/backup/src/error.rs
71 ↗(On Diff #45427)

Let's name this IdentityClientError to be more precise