From f537edccd8d8b4efa432e197d58f7a8d3f98a3c0 Mon Sep 17 00:00:00 2001 From: Vilius Pranckaitis Date: Mon, 16 Nov 2020 16:10:46 +0200 Subject: [PATCH] PR comments --- src/cmd/tools/dtest/docker/harness/resources/coordinator.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/cmd/tools/dtest/docker/harness/resources/coordinator.go b/src/cmd/tools/dtest/docker/harness/resources/coordinator.go index 034c271ce2..0333519ecc 100644 --- a/src/cmd/tools/dtest/docker/harness/resources/coordinator.go +++ b/src/cmd/tools/dtest/docker/harness/resources/coordinator.go @@ -325,14 +325,14 @@ func makePostRequest(logger *zap.Logger, url string, body proto.Message) (*http. if err := (&jsonpb.Marshaler{}).Marshal(data, body); err != nil { logger.Error("failed to marshal", zap.Error(err)) - return &http.Response{}, fmt.Errorf("failed to marshal: %w", err) + return nil, fmt.Errorf("failed to marshal: %w", err) } req, err := http.NewRequestWithContext(context.Background(), http.MethodPost, url, data) if err != nil { logger.Error("failed to construct request", zap.Error(err)) - return &http.Response{}, fmt.Errorf("failed to construct request: %w", err) + return nil, fmt.Errorf("failed to construct request: %w", err) } req.Header.Add("Content-Type", "application/json")