Page MenuHomePhabricator
Differential D13877

[backup-service] implement endpoint to upload User Data
ClosedPublic
Actions

Authored by kamil on Nov 5 2024, 9:07 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Apr 6, 8:19 AM
Unknown Object (File)
Sun, Mar 23, 1:02 PM
Unknown Object (File)
Fri, Mar 14, 4:58 AM
Unknown Object (File)
Mar 8 2025, 6:24 AM
Unknown Object (File)
Mar 8 2025, 6:24 AM
Unknown Object (File)
Mar 8 2025, 6:24 AM
Unknown Object (File)
Mar 8 2025, 6:24 AM
Unknown Object (File)
Mar 8 2025, 6:24 AM
View All 66 Files
Subscribers
ashoat
tomek

Details

Reviewers
varun
bartek
Commits
rCOMM556469ea7e45: [backup-service] implement endpoint to upload User Data
Summary

Part of ENG-9673.

Tracked in ENG-9672.

Depends on D13876

Test Plan

Tested in D13879

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

kamil created this revision.Nov 5 2024, 9:07 AM
kamil held this revision as a draft.
Herald added subscribers: tomek, ashoat. · View Herald TranscriptNov 5 2024, 9:07 AM
kamil added a child revision: D13878: [backup-service] make `user_data` and `user_keys` optional in `BackupData`.Nov 5 2024, 9:10 AM
kamil updated this revision to Diff 45616.Nov 5 2024, 9:14 AM

update

kamil edited the test plan for this revision. (Show Details)Nov 5 2024, 9:16 AM
Harbormaster completed remote builds in B32544: Diff 45612.Nov 5 2024, 9:25 AM
Harbormaster completed remote builds in B32548: Diff 45616.Nov 5 2024, 9:56 AM
kamil published this revision for review.Nov 6 2024, 3:43 AM
kamil added inline comments.
services/backup/src/http/handlers/backup.rs
215

we need to have User Keys to be able to upload User Data only

bartek accepted this revision.Nov 6 2024, 9:32 PM
This revision is now accepted and ready to land.Nov 6 2024, 9:32 PM
Closed by commit rCOMM556469ea7e45: [backup-service] implement endpoint to upload User Data (authored by kamil). · Explain WhyNov 7 2024, 4:07 AM
This revision was automatically updated to reflect the committed changes.
kamil added a commit: rCOMM556469ea7e45: [backup-service] implement endpoint to upload User Data.

Revision Contents
Changeset List

PathSize
services/
backup/
src/
http/
handlers/
59 lines
6 lines

Diff 45616

View Options

services/backup/src/http/handlers/backup.rs

use anyhow::Result; use anyhow::Result;
use comm_lib::{auth::AuthService, blob::client::BlobServiceClient}; use comm_lib::{auth::AuthService, blob::client::BlobServiceClient};
use tracing::Level; use tracing::Level;
use tracing_subscriber::EnvFilter; use tracing_subscriber::EnvFilter;
pub mod config; pub mod config;
pub mod constants; pub mod constants;
pub mod database; pub mod database;
pub mod error; pub mod error;
pub mod http; pub mod http;
pub mod identity;
// re-export this to be available as crate::CONFIG // re-export this to be available as crate::CONFIG
pub use config::CONFIG; pub use config::CONFIG;
fn configure_logging() -> Result<()> { fn configure_logging() -> Result<()> {
let filter = EnvFilter::builder() let filter = EnvFilter::builder()
.with_default_directive(Level::INFO.into()) .with_default_directive(Level::INFO.into())
.with_env_var(constants::LOG_LEVEL_ENV_VAR) .with_env_var(constants::LOG_LEVEL_ENV_VAR)
Show All 21 Lines
View Options

services/backup/src/http/mod.rs

use anyhow::Result; use anyhow::Result;
use comm_lib::{auth::AuthService, blob::client::BlobServiceClient}; use comm_lib::{auth::AuthService, blob::client::BlobServiceClient};
use tracing::Level; use tracing::Level;
use tracing_subscriber::EnvFilter; use tracing_subscriber::EnvFilter;
pub mod config; pub mod config;
pub mod constants; pub mod constants;
pub mod database; pub mod database;
pub mod error; pub mod error;
pub mod http; pub mod http;
pub mod identity;
// re-export this to be available as crate::CONFIG // re-export this to be available as crate::CONFIG
pub use config::CONFIG; pub use config::CONFIG;
fn configure_logging() -> Result<()> { fn configure_logging() -> Result<()> {
let filter = EnvFilter::builder() let filter = EnvFilter::builder()
.with_default_directive(Level::INFO.into()) .with_default_directive(Level::INFO.into())
.with_env_var(constants::LOG_LEVEL_ENV_VAR) .with_env_var(constants::LOG_LEVEL_ENV_VAR)
Show All 21 Lines