From eb18c82976251d9a0c0730df27e10204c89beae2 Mon Sep 17 00:00:00 2001 From: Aditya Manthramurthy Date: Wed, 28 Aug 2019 16:58:40 -0700 Subject: [PATCH] Remove policy query param from being rejected for objects (#8144) --- cmd/generic-handlers.go | 1 - cmd/server_test.go | 15 --------------- 2 files changed, 16 deletions(-) diff --git a/cmd/generic-handlers.go b/cmd/generic-handlers.go index 5fc76a037..ee8a000c4 100644 --- a/cmd/generic-handlers.go +++ b/cmd/generic-handlers.go @@ -487,7 +487,6 @@ var notimplementedBucketResourceNames = map[string]bool{ // List of not implemented object queries var notimplementedObjectResourceNames = map[string]bool{ "acl": true, - "policy": true, "restore": true, "tagging": true, "torrent": true, diff --git a/cmd/server_test.go b/cmd/server_test.go index bb57d9d65..f9ff6a689 100644 --- a/cmd/server_test.go +++ b/cmd/server_test.go @@ -87,7 +87,6 @@ func runAllTests(suite *TestSuiteCommon, c *check) { suite.TestObjectGetAnonymous(c) suite.TestObjectGet(c) suite.TestMultipleObjects(c) - suite.TestNotImplemented(c) suite.TestHeader(c) suite.TestPutBucket(c) suite.TestCopyObject(c) @@ -996,20 +995,6 @@ func (s *TestSuiteCommon) TestMultipleObjects(c *check) { c.Assert(true, bytes.Equal(responseBody, []byte("hello three"))) } -// TestNotImplemented - validates if object policy is implemented, should return 'NotImplemented'. -func (s *TestSuiteCommon) TestNotImplemented(c *check) { - // Generate a random bucket name. - bucketName := getRandomBucketName() - request, err := newTestSignedRequest("GET", s.endPoint+SlashSeparator+bucketName+"/object?policy", - 0, nil, s.accessKey, s.secretKey, s.signer) - c.Assert(err, nil) - - client := http.Client{Transport: s.transport} - response, err := client.Do(request) - c.Assert(err, nil) - c.Assert(response.StatusCode, http.StatusNotImplemented) -} - // TestHeader - Validates the error response for an attempt to fetch non-existent object. func (s *TestSuiteCommon) TestHeader(c *check) { // generate a random bucket name.