Don't read response body twice on putSignaturesToAPIExtension failure

registryHTTPResponseToError expects the response body to be available; don't read it first. Signed-off-by: Miloslav Trmač <[email protected]>
containers · Aug 26, 2021 · ebdfeda · ebdfeda
1 parent 9ec2deb
commit ebdfeda
Showing 1 changed file with 0 additions and 5 deletions.
diff --git a/docker/docker_image_dest.go b/docker/docker_image_dest.go
@@ -16,7 +16,6 @@ import (
 
 	"github.com/containers/image/v5/docker/reference"
 	"github.com/containers/image/v5/internal/blobinfocache"
-	"github.com/containers/image/v5/internal/iolimits"
 	"github.com/containers/image/v5/internal/putblobdigest"
 	"github.com/containers/image/v5/internal/uploadreader"
 	"github.com/containers/image/v5/manifest"
@@ -649,10 +648,6 @@ sigExists:
 		}
 		defer res.Body.Close()
 		if res.StatusCode != http.StatusCreated {
-			body, err := iolimits.ReadAtMost(res.Body, iolimits.MaxErrorBodySize)
-			if err == nil {
-				logrus.Debugf("Error body %s", string(body))
-			}
 			logrus.Debugf("Error uploading signature, status %d, %#v", res.StatusCode, res)
 			return errors.Wrapf(registryHTTPResponseToError(res), "uploading signature to %s in %s", path, d.c.registry)
 		}