From eeb657fb5b72856101ad5cfbc18bf6afb2f8f94c Mon Sep 17 00:00:00 2001 From: Jacob Kozol Date: Sun, 27 Oct 2019 18:32:12 +0100 Subject: [PATCH] api: refactor image size in composeImageHandler The composeImageHandler no longer needs to get the file size for an Image because the image struct now contains the file size. --- internal/weldr/api.go | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/internal/weldr/api.go b/internal/weldr/api.go index b1cc419a8..7e9e370f3 100644 --- a/internal/weldr/api.go +++ b/internal/weldr/api.go @@ -854,15 +854,9 @@ func (api *API) composeImageHandler(writer http.ResponseWriter, request *http.Re return } - stat, err := image.File.Stat() - if err != nil { - statusResponseError(writer, http.StatusInternalServerError) - return - } - writer.Header().Set("Content-Disposition", "attachment; filename="+uuid.String()+"-"+image.Name) writer.Header().Set("Content-Type", image.Mime) - writer.Header().Set("Content-Length", fmt.Sprintf("%d", stat.Size())) + writer.Header().Set("Content-Length", fmt.Sprintf("%d", image.Size)) io.Copy(writer, image.File) }