From 57430fe183f09aca24f5e88da6ad6bdda9fcddaf Mon Sep 17 00:00:00 2001
From: Harshavardhana <harsha@harshavardhana.net>
Date: Mon, 7 Dec 2015 11:57:33 -0800
Subject: [PATCH] ignore-handlers: Enhance ignore handlers to cater for bucket
 resources with or without separators

Fixes an issue which we saw with minio-py
---
 api-definitions.go  |  1 -
 generic-handlers.go | 33 +++++++++++++++++++++++----------
 2 files changed, 23 insertions(+), 11 deletions(-)

diff --git a/api-definitions.go b/api-definitions.go
index ee2d65cb6..48b886f66 100644
--- a/api-definitions.go
+++ b/api-definitions.go
@@ -199,7 +199,6 @@ var notimplementedBucketResourceNames = map[string]bool{
 	"policy":         true,
 	"cors":           true,
 	"lifecycle":      true,
-	"location":       true,
 	"logging":        true,
 	"notification":   true,
 	"replication":    true,
diff --git a/generic-handlers.go b/generic-handlers.go
index 822160c76..0cf9f2654 100644
--- a/generic-handlers.go
+++ b/generic-handlers.go
@@ -136,27 +136,40 @@ func IgnoreResourcesHandler(h http.Handler) http.Handler {
 	return resourceHandler{h}
 }
 
-const (
-	separator = "/"
-)
-
 // Resource handler ServeHTTP() wrapper
 func (h resourceHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
-	splits := strings.SplitN(r.URL.Path, separator, 3)
-	switch len(splits) {
-	// bucket exists
-	case 2:
+	// Skip the first element which is usally '/' and split the rest.
+	splits := strings.SplitN(r.URL.Path[1:], "/", 2)
+
+	// Save bucketName and objectName extracted from url Path.
+	var bucketName, objectName string
+	if len(splits) == 1 {
+		bucketName = splits[0]
+	}
+	if len(splits) == 2 {
+		bucketName = splits[0]
+		objectName = splits[1]
+	}
+	// If bucketName is present and not objectName check for bucket
+	// level resource queries.
+	if bucketName != "" && objectName == "" {
 		if ignoreNotImplementedBucketResources(r) {
 			writeErrorResponse(w, r, NotImplemented, r.URL.Path)
 			return
 		}
-	// object exists
-	case 3:
+	}
+	// If bucketName and objectName are present check for its resource queries.
+	if bucketName != "" && objectName != "" {
 		if ignoreNotImplementedObjectResources(r) {
 			writeErrorResponse(w, r, NotImplemented, r.URL.Path)
 			return
 		}
 	}
+	// A put method on path "/" doesn't make sense, ignore it.
+	if r.Method == "PUT" && r.URL.Path == "/" {
+		writeErrorResponse(w, r, NotImplemented, r.URL.Path)
+		return
+	}
 	h.handler.ServeHTTP(w, r)
 }