[#xxx] client: Return status from all methods
Some checks failed
DCO / DCO (pull_request) Failing after 40s
Tests and linters / Tests (pull_request) Successful in 1m9s
Tests and linters / Lint (pull_request) Successful in 1m44s

Since status is checked first in handleError method, it should be returned from client methods

Signed-off-by: Marina Biryukova <m.biryukova@yadro.com>
This commit is contained in:
Marina Biryukova 2024-09-24 16:31:56 +03:00
parent 99d5bf913b
commit 3063b525d5
8 changed files with 8 additions and 8 deletions

View file

@ -65,7 +65,7 @@ func (c *Client) APEManagerListChains(ctx context.Context, prm PrmAPEManagerList
var res ResAPEManagerListChains var res ResAPEManagerListChains
res.st, err = c.processResponse(resp) res.st, err = c.processResponse(resp)
if err != nil || !apistatus.IsSuccessful(res.st) { if err != nil || !apistatus.IsSuccessful(res.st) {
return nil, err return &res, err
} }
for _, ch := range resp.GetBody().GetChains() { for _, ch := range resp.GetBody().GetChains() {

View file

@ -240,7 +240,7 @@ func (c *Client) NetMapSnapshot(ctx context.Context, _ PrmNetMapSnapshot) (*ResN
var res ResNetMapSnapshot var res ResNetMapSnapshot
res.st, err = c.processResponse(resp) res.st, err = c.processResponse(resp)
if err != nil { if err != nil {
return nil, err return &res, err
} }
if !apistatus.IsSuccessful(res.st) { if !apistatus.IsSuccessful(res.st) {

View file

@ -149,7 +149,7 @@ func (c *Client) ObjectDelete(ctx context.Context, prm PrmObjectDelete) (*ResObj
var res ResObjectDelete var res ResObjectDelete
res.st, err = c.processResponse(resp) res.st, err = c.processResponse(resp)
if err != nil { if err != nil {
return nil, err return &res, err
} }
if !apistatus.IsSuccessful(res.st) { if !apistatus.IsSuccessful(res.st) {

View file

@ -493,7 +493,7 @@ func (c *Client) ObjectHead(ctx context.Context, prm PrmObjectHead) (*ResObjectH
var res ResObjectHead var res ResObjectHead
res.st, err = c.processResponse(resp) res.st, err = c.processResponse(resp)
if err != nil { if err != nil {
return nil, err return &res, err
} }
if !apistatus.IsSuccessful(res.st) { if !apistatus.IsSuccessful(res.st) {

View file

@ -190,7 +190,7 @@ func (c *Client) ObjectHash(ctx context.Context, prm PrmObjectHash) (*ResObjectH
var res ResObjectHash var res ResObjectHash
res.st, err = c.processResponse(resp) res.st, err = c.processResponse(resp)
if err != nil { if err != nil {
return nil, err return &res, err
} }
if !apistatus.IsSuccessful(res.st) { if !apistatus.IsSuccessful(res.st) {

View file

@ -247,7 +247,7 @@ func (x *objectPatcher) Close(_ context.Context) (*ResObjectPatch, error) {
x.res.st, x.err = x.client.processResponse(&x.respV2) x.res.st, x.err = x.client.processResponse(&x.respV2)
if x.err != nil { if x.err != nil {
return nil, x.err return &x.res, x.err
} }
if !apistatus.IsSuccessful(x.res.st) { if !apistatus.IsSuccessful(x.res.st) {

View file

@ -157,7 +157,7 @@ func (x *objectWriterRaw) Close(_ context.Context) (*ResObjectPut, error) {
x.res.st, x.err = x.client.processResponse(&x.respV2) x.res.st, x.err = x.client.processResponse(&x.respV2)
if x.err != nil { if x.err != nil {
return nil, x.err return &x.res, x.err
} }
if !apistatus.IsSuccessful(x.res.st) { if !apistatus.IsSuccessful(x.res.st) {

View file

@ -167,7 +167,7 @@ func (c *Client) ObjectPutSingle(ctx context.Context, prm PrmObjectPutSingle) (*
var res ResObjectPutSingle var res ResObjectPutSingle
res.st, err = c.processResponse(resp) res.st, err = c.processResponse(resp)
if err != nil { if err != nil {
return nil, err return &res, err
} }
res.epoch = resp.GetMetaHeader().GetEpoch() res.epoch = resp.GetMetaHeader().GetEpoch()