[comm-lib][blob-client] Extract error handling logic
Needs ReviewPublic
Actions

Authored by bartek on Wed, Oct 9, 1:17 AM.

Details

Reviewers

varun
will
kamil

Summary

Noticed that response handling is repetitive. Ectracted a common utility function, and refactored some of these functions to use it and return early when possible.

Depends on D13644

Test Plan

cargo build -p comm-lib

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

bartek created this revision.Wed, Oct 9, 1:17 AM

bartek held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptWed, Oct 9, 1:17 AM

bartek added a child revision: D13646: [comm-lib][blob] Add tags support to remove holders request payload.Wed, Oct 9, 1:20 AM

Harbormaster completed remote builds in B32125: Diff 44995.Wed, Oct 9, 1:39 AM

bartek published this revision for review.Wed, Oct 9, 1:57 AM

Revision Contents
Changeset List

Path

Size

shared/

comm-lib/

src/

blob/

client.rs

81 lines

Diff 44995

View Options

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

[comm-lib][blob-client] Extract error handling logicNeeds ReviewPublicActions