Paths

Table of Contentst

Differential D8964

[backup] Introduce download endpoints
ClosedPublic
Actions

Authored by michal on Aug 28 2023, 1:49 AM.

Tags

None

Referenced Files

	F2110761: D8964.id30422.diff
	Tue, Jun 25, 8:58 PM

	F2103263: D8964.id30531.diff
	Tue, Jun 25, 2:42 AM

	F2103262: D8964.id30422.diff
	Tue, Jun 25, 2:42 AM

	F2103207: D8964.id.diff
	Tue, Jun 25, 2:41 AM

	F2103147: D8964.diff
	Tue, Jun 25, 2:33 AM

	Unknown Object (File)
	Sun, Jun 23, 11:26 AM

	Unknown Object (File)
	Sun, Jun 23, 2:57 AM

	Unknown Object (File)
	Sat, Jun 22, 3:49 AM

View All 43 Files

Subscribers

Details

Reviewers

bartek
• patryk
kamil
• jon

Commits

rCOMM02d2d035d2ed: [backup] Introduce download endpoints

Summary

Endpoints for downloading backup blobs: user_keys and user_data.

Test Plan

Run:

GET http://127.0.0.1:50052/backups/id7/user_keys
Authorization: Bearer eyJ1c2VySUQiOiAiMSIsICJhY2Nlc3NUb2tlbiI6ICIyIiwgImRldmljZUlEIjogIjMifQ==

GET http://127.0.0.1:50052/backups/id7/user_data
Authorization: Bearer eyJ1c2VySUQiOiAiMSIsICJhY2Nlc3NUb2tlbiI6ICIyIiwgImRldmljZUlEIjogIjMifQ==

Test if the returned data is correct. Run with non-existant backup id, check if 404 is returned.

Diff Detail

Repository

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

michal created this revision.Aug 28 2023, 1:49 AM

Herald added a reviewer: • jon. · View Herald TranscriptAug 28 2023, 1:49 AM

Herald added subscribers: tomek, ashoat. · View Herald Transcript

michal added a parent revision: D8963: [backup] Introduce upload endpoint.Aug 28 2023, 1:49 AM

michal added inline comments.

services/comm-services-lib/src/blob/client.rs
87	Modified so we don't have to unnecessarily allocate a vec.

michal added a child revision: D8965: [backup] Implement download latest endpoints.Aug 28 2023, 2:01 AM

Harbormaster completed remote builds in B22156: Diff 30422.Aug 28 2023, 2:07 AM

michal requested review of this revision.Aug 28 2023, 2:07 AM

bartek mentioned this in D8974: [services-lib][blob-client] Use Bytes instead of Vec<u8>.Aug 28 2023, 4:06 AM

bartek accepted this revision.Aug 28 2023, 5:28 AM

This revision is now accepted and ready to land.Aug 28 2023, 5:28 AM

kamil mentioned this in D8990: [client-backup] add backup-service facts.Aug 29 2023, 2:04 AM

bartek mentioned this in rCOMM758d5a4d6450: [services-lib][blob-client] Use Bytes instead of Vec<u8>.Aug 29 2023, 11:24 AM

Closed by commit rCOMM02d2d035d2ed: [backup] Introduce download endpoints (authored by Michal Gniadek <michal.gniadek@swmansion.com>). · Explain WhyAug 30 2023, 1:28 AM

This revision was automatically updated to reflect the committed changes.

Michal Gniadek <michal.gniadek@swmansion.com> added a commit: rCOMM02d2d035d2ed: [backup] Introduce download endpoints.

kamil mentioned this in rCOMM84ce71bd0714: [client-backup] add backup-service facts.Aug 30 2023, 11:29 AM

Revision Contents
Changeset List

Path

Size

services/

backup/

2 lines

src/

database/

21 lines

4 lines

http/

handlers/

63 lines

19 lines

comm-services-lib/

src/

blob/

5 lines

Diff 30422

services/backup/Cargo.lock

Loading...

services/backup/Cargo.toml

Loading...

services/backup/src/database/mod.rs

Loading...

services/backup/src/error.rs

Loading...

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

Loading...

services/backup/src/http/mod.rs

Loading...

services/comm-services-lib/src/blob/client.rs

Loading...