Page MenuHomePhabricator

[services-lib] Let blob client use services token
ClosedPublic

Authored by bartek on Sep 25 2023, 5:36 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Nov 19, 5:31 PM
Unknown Object (File)
Mon, Oct 28, 11:44 AM
Unknown Object (File)
Sun, Oct 27, 4:14 AM
Unknown Object (File)
Oct 13 2024, 7:54 PM
Unknown Object (File)
Oct 13 2024, 7:54 PM
Unknown Object (File)
Oct 13 2024, 7:54 PM
Unknown Object (File)
Oct 13 2024, 7:48 PM
Unknown Object (File)
Oct 13 2024, 10:13 AM
Subscribers

Details

Summary

Changed blob client logic to use the new AuthorizationCredential enum instead of the more specific UserIdentity

Depends on D9243

Test Plan

Blob client works as before

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bartek held this revision as a draft.
bartek published this revision for review.Sep 25 2023, 5:56 AM
bartek added inline comments.
services/comm-services-lib/src/blob/client.rs
101–108 ↗(On Diff #31384)

Maybe we should deprecate this function

michal added inline comments.
services/comm-services-lib/src/blob/client.rs
101–108 ↗(On Diff #31384)

I don't have strong feelings about this, makes sense

This revision is now accepted and ready to land.Sep 25 2023, 6:21 AM