Page MenuHomePhabricator
Differential D9966

[keyserver] Add a function that deletes a blob
ClosedPublic
Actions

Authored by tomek on Nov 24 2023, 6:48 AM.
Tags
None
Referenced Files
F4090025: D9966.id36406.diff
Sun, Feb 16, 2:14 PM
Unknown Object (File)
Mon, Feb 3, 6:03 AM
Unknown Object (File)
Sun, Feb 2, 1:50 PM
Unknown Object (File)
Sun, Feb 2, 11:19 AM
Unknown Object (File)
Sun, Feb 2, 11:03 AM
Unknown Object (File)
Sun, Feb 2, 10:59 AM
Unknown Object (File)
Sat, Feb 1, 9:49 PM
Unknown Object (File)
Wed, Jan 29, 12:20 PM
View All Files
Subscribers
ashoat

Details

Reviewers
michal
bartek
rohan
Commits
rCOMM7053a0f8c5be: [keyserver] Add a function that deletes a blob
Summary

This function calls an endpoint of Blob service that deletes a blob.

Depends on D9944

Test Plan

Create a blob and delete it - check if a holder is deleted.

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

tomek created this revision.Nov 24 2023, 6:48 AM
Herald added a subscriber: ashoat. · View Herald TranscriptNov 24 2023, 6:48 AM
Harbormaster completed remote builds in B24387: Diff 33586.Nov 24 2023, 7:08 AM
tomek requested review of this revision.Nov 24 2023, 7:08 AM
tomek added a child revision: D9968: [keyserver] Fetch invite link blob holder.Nov 24 2023, 8:14 AM
rohan accepted this revision.Nov 24 2023, 10:42 AM
This revision is now accepted and ready to land.Nov 24 2023, 10:42 AM
tomek updated this revision to Diff 34052.Nov 30 2023, 6:52 AM

Use a param object instead of two args for hash and holder to avoid the confusion between them

Harbormaster completed remote builds in B24640: Diff 34052.Nov 30 2023, 7:11 AM
tomek updated this revision to Diff 36404.Jan 31 2024, 8:39 AM

Destructure params inside function

tomek updated this revision to Diff 36406.Jan 31 2024, 8:47 AM

Set instant flag

Harbormaster completed remote builds in B26336: Diff 36404.Jan 31 2024, 8:57 AM
Harbormaster completed remote builds in B26338: Diff 36406.Jan 31 2024, 9:05 AM
bartek accepted this revision.Feb 7 2024, 2:05 AM
bartek added inline comments.
keyserver/src/services/blob.js
103–105 ↗(On Diff #36406)

Before landing, could you update this diff to use the function from D10908?

tomek updated this revision to Diff 36846.Feb 8 2024, 6:31 AM

Use auth headers

keyserver/src/services/blob.js
103–105 ↗(On Diff #36406)

Sure!

Harbormaster completed remote builds in B26649: Diff 36846.Feb 8 2024, 6:47 AM
Closed by commit rCOMM7053a0f8c5be: [keyserver] Add a function that deletes a blob (authored by tomek). · Explain WhyFeb 14 2024, 7:58 AM
This revision was automatically updated to reflect the committed changes.
tomek added a commit: rCOMM7053a0f8c5be: [keyserver] Add a function that deletes a blob.

Revision Contents
Changeset List

PathSize
keyserver/
src/
services/
22 lines

Diff 36846

View Options

keyserver/src/services/blob.js

Loading...