From 584cb61bb84746c92f415ab78c74820791b34858 Mon Sep 17 00:00:00 2001 From: Aditya Manthramurthy Date: Fri, 21 Sep 2018 13:48:58 -0700 Subject: [PATCH] Switch back to GetObjectInfo for HEAD requests (#6513) --- cmd/object-handlers.go | 13 +++---------- 1 file changed, 3 insertions(+), 10 deletions(-) diff --git a/cmd/object-handlers.go b/cmd/object-handlers.go index 02223a03d..515923926 100644 --- a/cmd/object-handlers.go +++ b/cmd/object-handlers.go @@ -450,9 +450,9 @@ func (api objectAPIHandlers) HeadObjectHandler(w http.ResponseWriter, r *http.Re bucket := vars["bucket"] object := vars["object"] - getObjectNInfo := objectAPI.GetObjectNInfo + getObjectInfo := objectAPI.GetObjectInfo if api.CacheAPI() != nil { - getObjectNInfo = api.CacheAPI().GetObjectNInfo + getObjectInfo = api.CacheAPI().GetObjectInfo } opts := ObjectOptions{} @@ -477,11 +477,6 @@ func (api objectAPIHandlers) HeadObjectHandler(w http.ResponseWriter, r *http.Re ConditionValues: getConditionValues(r, ""), IsOwner: false, }) { - getObjectInfo := objectAPI.GetObjectInfo - if api.CacheAPI() != nil { - getObjectInfo = api.CacheAPI().GetObjectInfo - } - _, err := getObjectInfo(ctx, bucket, object, opts) if toAPIErrorCode(err) == ErrNoSuchKey { s3Error = ErrNoSuchKey @@ -492,13 +487,11 @@ func (api objectAPIHandlers) HeadObjectHandler(w http.ResponseWriter, r *http.Re return } - gr, err := getObjectNInfo(ctx, bucket, object, nil, r.Header) + objInfo, err := getObjectInfo(ctx, bucket, object, opts) if err != nil { writeErrorResponseHeadersOnly(w, toAPIErrorCode(err)) return } - defer gr.Close() - objInfo := gr.ObjInfo var encrypted bool if objectAPI.IsEncryptionSupported() {