Page MenuHomePhabricator

[services-lib] Introduce Blob Client
ClosedPublic

Authored by bartek on Aug 10 2023, 10:16 AM.
Tags
None
Referenced Files
F3384248: D8777.id29822.diff
Thu, Nov 28, 8:07 PM
Unknown Object (File)
Mon, Nov 25, 7:52 AM
Unknown Object (File)
Mon, Nov 25, 2:26 AM
Unknown Object (File)
Sat, Nov 23, 1:58 PM
Unknown Object (File)
Sat, Nov 23, 12:59 PM
Unknown Object (File)
Sat, Nov 23, 12:59 PM
Unknown Object (File)
Fri, Nov 22, 1:13 AM
Unknown Object (File)
Oct 28 2024, 11:29 PM
Subscribers

Details

Summary

Part of ENG-4597.

This diff introduces a foundation for blob client in comm-services-lib. The client is hidden behind a crate feature so services that do not need it do not have to depend on it (e.g. feature-flags).

Test Plan

Cargo build in comm-services-lib, blob and feature-flags.

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

bartek held this revision as a draft.

Fix Cargo.lock dependencies in services

services/backup/Cargo.lock
592 ↗(On Diff #29822)

These files changed because I added the dependency to the lib

services/comm-services-lib/src/blob/client.rs
1 ↗(On Diff #29822)

Will be removed in the next diff

47–50 ↗(On Diff #29822)

This is the place where the authorization token will be stored and managed.

services/comm-services-lib/src/blob/mod.rs
1–2 ↗(On Diff #29822)

This is the first submodule, but there will be more added later

bartek published this revision for review.Aug 11 2023, 2:19 AM

This will be very useful! I would prefer to have multiple packages in comm-services-lib/ (comm-services-lib/database-utils/, comm-services-lib/blob-client), but I don't really have strong argument against using features for this.

This revision is now accepted and ready to land.Aug 11 2023, 4:11 AM
This revision was automatically updated to reflect the committed changes.