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
Unknown Object (File)
Thu, Nov 28, 5:45 AM
Unknown Object (File)
Wed, Nov 27, 5:21 AM
Unknown Object (File)
Tue, Nov 26, 6:23 AM
Unknown Object (File)
Tue, Nov 26, 4:23 AM
Unknown Object (File)
Mon, Nov 25, 3:11 AM
Unknown Object (File)
Sat, Nov 16, 2:34 AM
Unknown Object (File)
Fri, Nov 15, 1:41 PM
Unknown Object (File)
Wed, Nov 13, 7:20 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
Branch
backup-work-6
Lint
No Lint Coverage
Unit
No Test Coverage

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.Fri, Nov 1, 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