Task middleware remove (#1826)

* adding middleware remove

middleware.go

@@ -71,6 +71,29 @@ func (ms *MiddlewareStack) Use(mw ...MiddlewareFunc) {
 	ms.stack = append(ms.stack, mw...)
 }
 
+// Remove the specified Middleware(s) for the App/group. This is useful when
+// the middleware will be skipped by the entire group.
+/*
+	a.Middleware.Remove(Authorization)
+*/
+func (ms *MiddlewareStack) Remove(mws ...MiddlewareFunc) {
+	result := []MiddlewareFunc{}
+
+base:
+	for _, existing := range ms.stack {
+		for _, banned := range mws {
+			if funcKey(existing) == funcKey(banned) {
+				continue base
+			}
+		}
+
+		result = append(result, existing)
+	}
+
+	ms.stack = result
+
+}
+
 // Skip a specified piece of middleware the specified Handlers.
 // This is useful for things like wrapping your application in an
 // authorization middleware, but skipping it for things the home

middleware_test.go

@@ -199,3 +199,46 @@ func Test_Middleware_Clear(t *testing.T) {
 	r.Len(mws.stack, 0)
 	r.Len(mws.skips, 0)
 }
+
+func Test_Middleware_Remove(t *testing.T) {
+	r := require.New(t)
+	log := []string{}
+
+	mw1 := func(h Handler) Handler {
+		log = append(log, "mw1")
+		return h
+	}
+
+	mw2 := func(h Handler) Handler {
+		log = append(log, "mw2")
+		return h
+	}
+
+	a := New(Options{})
+	a.Use(mw2)
+	a.Use(mw1)
+
+	var cr Resource = &carsResource{}
+	g := a.Resource("/autos", cr)
+	g.Middleware.Remove(mw2)
+
+	a.Resource("/all_log_autos", cr)
+	w := httptest.New(a)
+
+	ng := a.Resource("/no_log_autos", cr)
+	ng.Middleware.Remove(mw1, mw2)
+
+	_ = w.HTML("/autos/1").Get()
+	r.Len(log, 1)
+	r.Equal("mw1", log[0])
+
+	log = []string{}
+	_ = w.HTML("/all_log_autos/1").Get()
+	r.Len(log, 2)
+	r.Contains(log, "mw2")
+	r.Contains(log, "mw1")
+
+	log = []string{}
+	_ = w.HTML("/no_log_autos/1").Get()
+	r.Len(log, 0)
+}