Merge pull request #20 from tonny-zhang/develop

delete auto deal application/json

context_post.go

@@ -1,7 +1,6 @@
 package cotton
 
 import (
-	"encoding/json"
 	"io"
 	"mime/multipart"
 	"net/http"
@@ -13,34 +12,12 @@ import (
 func (ctx *Context) initPostFormCache() {
 	if nil == ctx.postFormCache {
 		if nil != ctx.Request {
-			ct := ctx.GetRequestHeader("Content-Type")
-			if ct == "application/json" {
-				body := ctx.Request.Body
-				if body != nil {
-					values := url.Values{}
-					obj := make(map[string]interface{})
-					json.NewDecoder(body).Decode(&obj)
-					for k, v := range obj {
-						var valStr string
-						switch v.(type) {
-						case string:
-							valStr = v.(string)
-						default:
-							b, _ := json.Marshal(v)
-							valStr = string(b)
-						}
-						values[k] = []string{valStr}
-					}
-					ctx.postFormCache = values
+			if e := ctx.Request.ParseMultipartForm(defaultMultipartMemory); e != nil {
+				if e != http.ErrNotMultipart {
+					panic(e)
 				}
-			} else {
-				if e := ctx.Request.ParseMultipartForm(defaultMultipartMemory); e != nil {
-					if e != http.ErrNotMultipart {
-						panic(e)
-					}
-				}
-				ctx.postFormCache = ctx.Request.PostForm
 			}
+			ctx.postFormCache = ctx.Request.PostForm
 		} else {
 			ctx.postFormCache = url.Values{}
 		}

router_test.go

@@ -9,6 +9,17 @@ import (
 	"github.com/stretchr/testify/assert"
 )
 
+var handlerEmpty = func(c *Context) {}
+
+func TestDiffentParamNameInSamePos(t *testing.T) {
+	// same postion param name must be same
+	assert.PanicsWithError(t, "[:name] in path [/v1/abc/abc/:name/:id] conflicts with [/v1/abc/abc/:id]", func() {
+		router := NewRouter()
+		g := router.Group("/v1/abc/")
+		g.Post("/abc/:id", handlerEmpty)
+		g.Post("/abc/:name/:id", handlerEmpty)
+	})
+}
 func TestAddHandleFunc(t *testing.T) {
 	handler := func(c *Context) {}
 	assert.PanicsWithError(t, "[hello] shold absolute path", func() {