Page MenuHomePhabricator

[Identity] Refactor get user_id and device_id
ClosedPublic

Authored by jon on Jul 24 2023, 6:52 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Dec 14, 7:06 PM
Unknown Object (File)
Sat, Dec 14, 7:06 PM
Unknown Object (File)
Sat, Dec 14, 7:06 PM
Unknown Object (File)
Sat, Dec 14, 7:06 PM
Unknown Object (File)
Sat, Dec 14, 6:59 PM
Unknown Object (File)
Fri, Dec 13, 12:52 AM
Unknown Object (File)
Fri, Dec 6, 11:51 AM
Unknown Object (File)
Mon, Nov 25, 11:18 AM
Subscribers

Details

Summary

This is useful for all authenticated requests, refactor.

Part of: https://linear.app/comm/issue/ENG-4445

Depends on D8605

Test Plan
cd identity && cargo build

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Jul 24 2023, 6:58 AM
Harbormaster failed remote builds in B21149: Diff 28971!
bartek added inline comments.
services/identity/src/grpc_services/authenticated.rs
73 ↗(On Diff #29028)

If they're a common pair, we can go further and define a struct type for them

This revision is now accepted and ready to land.Jul 26 2023, 3:04 AM
jon added inline comments.
services/identity/src/grpc_services/authenticated.rs
73 ↗(On Diff #29028)

A lot of these changes are just small improvements, it's hard to know when I should make something more formalized.

76 ↗(On Diff #29028)

Would probably be nice for this to be defined in some more formal way. However, that would probably involve moving a bunch of this to something in shared/

77 ↗(On Diff #29028)

this should probably be related to missing a header instead of unauthenticated, although this value is also used for authentication.