Fixed the memory leak while using gorilla/context.Set. Clear the global map object with current request after tasks api return or manually release all the expired objects in context using api of gc.

Author: qhyou11

api/debug/gc.go

@@ -3,9 +3,12 @@ package debug
 import (
 	"net/http"
 	"runtime"
+
+	"github.com/gorilla/context"
 )
 
 func GC(w http.ResponseWriter, r *http.Request) {
+	context.Purge(600)
 	runtime.GC()
 	w.WriteHeader(http.StatusNoContent)
 }

api/projects/tasks.go

@@ -3,15 +3,16 @@ package projects
 import (
 	"bytes"
 	"errors"
+	"net/http"
+	"strconv"
+	"time"
+
 	"github.com/gorilla/context"
 	"github.com/semaphoreui/semaphore/api/helpers"
 	"github.com/semaphoreui/semaphore/db"
 	"github.com/semaphoreui/semaphore/services/tasks"
 	"github.com/semaphoreui/semaphore/util"
 	log "github.com/sirupsen/logrus"
-	"net/http"
-	"strconv"
-	"time"
 )
 
 // AddTask inserts a task into the database and returns a header or returns error
@@ -102,6 +103,7 @@ func GetTask(w http.ResponseWriter, r *http.Request) {
 // GetTaskMiddleware is middleware that gets a task by id and sets the context to it or panics
 func GetTaskMiddleware(next http.Handler) http.Handler {
 	return http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		defer context.Clear(r)
 		project := context.Get(r, "project").(db.Project)
 		taskID, err := helpers.GetIntParam("task_id", w, r)