From 9a2102f5edac025825f079865e0a8d581611c6b5 Mon Sep 17 00:00:00 2001 From: Klaus Post Date: Tue, 8 Jun 2021 18:51:55 +0200 Subject: [PATCH] Always get actual size in CopyObjectPart (#12466) Always use `GetActualSize` to get the part size, not just when encrypted. Fixes mint test io.minio.MinioClient.uploadPartCopy, error "Range specified is not valid for source object". --- cmd/object-handlers.go | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/cmd/object-handlers.go b/cmd/object-handlers.go index 77ecb3cd0..abcb30325 100644 --- a/cmd/object-handlers.go +++ b/cmd/object-handlers.go @@ -2330,13 +2330,10 @@ func (api objectAPIHandlers) CopyObjectPartHandler(w http.ResponseWriter, r *htt defer gr.Close() srcInfo := gr.ObjInfo - actualPartSize := srcInfo.Size - if _, ok := crypto.IsEncrypted(srcInfo.UserDefined); ok { - actualPartSize, err = srcInfo.GetActualSize() - if err != nil { - writeErrorResponse(ctx, w, toAPIError(ctx, err), r.URL, guessIsBrowserReq(r)) - return - } + actualPartSize, err := srcInfo.GetActualSize() + if err != nil { + writeErrorResponse(ctx, w, toAPIError(ctx, err), r.URL, guessIsBrowserReq(r)) + return } if err := enforceBucketQuota(ctx, dstBucket, actualPartSize); err != nil {