Page MenuHomePhabricator

[services-lib] Introduce Blob Client
ClosedPublic

Authored by bartek on Aug 10 2023, 10:16 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, May 19, 3:04 PM
Unknown Object (File)
Mon, May 6, 4:52 PM
Unknown Object (File)
Mon, May 6, 12:29 PM
Unknown Object (File)
Sun, May 5, 5:56 AM
Unknown Object (File)
Sun, May 5, 3:02 AM
Unknown Object (File)
Sat, May 4, 4:51 PM
Unknown Object (File)
Sat, May 4, 1:12 PM
Unknown Object (File)
Sat, May 4, 1:12 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
Lint Not Applicable
Unit
Tests Not Applicable

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.