Paths

Table of Contentst

Differential D10131

[keyserver] Change blob upload functions to return more usable results
ClosedPublic
Actions

Authored by tomek on Dec 1 2023, 11:59 AM.

Tags

None

Referenced Files

None

Subscribers

Details

Reviewers

michal
bartek
marcin
rohan

Commits

rCOMM48441ea35eca: [keyserver] Change blob upload functions to return more usable results

Summary

We would like to be able to differentiate between hash in use and a plain failure and to achieve that we need a different API

Depends on D10099

Test Plan

Run a script and check if both SUCCESS and HASH_IN_USE results are returned

Diff Detail

Repository

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

tomek created this revision.Dec 1 2023, 11:59 AM

Herald added a subscriber: ashoat. · View Herald TranscriptDec 1 2023, 11:59 AM

tomek added a reviewer: marcin.Dec 1 2023, 11:59 AM

Harbormaster completed remote builds in B24683: Diff 34100.Dec 1 2023, 12:37 PM

tomek requested review of this revision.Dec 1 2023, 12:37 PM

tomek added a child revision: D10134: [keyserver] Create a blob while adding / editing an invite link.Dec 1 2023, 2:55 PM

marcin added inline comments.Dec 1 2023, 3:13 PM

keyserver/src/services/blob.js
36 ↗	(On Diff #34100)	I don't think we should ignore `status` and `statusText` in case of failure since we loos some meaningful information on what actually failed and make debugging harder. Please update types so that we can return more details information if blob upload fails.

Adding @marcin as blocking to make sure his concern is somehow addressed

keyserver/src/services/blob.js
36 ↗	(On Diff #34100)	Soon we'll have to support another status code here (401/403, see ENG-4765) so @marcin's comment makes sense. I don't have a single idea what's the best way here, maybe this kind of union? Also this touches a broader area - ENG-3841 and related tasks, so not sure what is actionable now.

I think @marcin and/or @bartek have more context here than I do, so I'll leave the review to them for this diff

Request changes to add error information to objects returned from functions.

This revision now requires changes to proceed.Dec 14 2023, 3:16 PM

Include status and text in a result

Harbormaster completed remote builds in B26411: Diff 36521.Feb 1 2024, 2:57 PM

bartek accepted this revision.Feb 5 2024, 7:36 AM

marcin accepted this revision.Feb 5 2024, 10:06 AM

This revision is now accepted and ready to land.Feb 5 2024, 10:06 AM

Rebase

Harbormaster failed remote builds in B26651: Diff 36848!Feb 8 2024, 2:46 PM

Fix Flow

Harbormaster failed remote builds in B26687: Diff 36901!Feb 9 2024, 10:51 AM

Rebase

Harbormaster completed remote builds in B26820: Diff 37063.Feb 14 2024, 12:05 PM

Closed by commit rCOMM48441ea35eca: [keyserver] Change blob upload functions to return more usable results. · Explain WhyFeb 14 2024, 3:58 PM

This revision was automatically updated to reflect the committed changes.

tomek added a commit: rCOMM48441ea35eca: [keyserver] Change blob upload functions to return more usable results.

Revision Contents
Changeset List

Path

Size

keyserver/

src/

services/

70 lines

Diff 36848

keyserver/src/services/blob.js

Loading...