Merge pull request #3070 from qhyou11/follow-the-upstream

Fixed the memory leak problem

Author: fiftin

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

@@ -4,15 +4,16 @@ import (
 	"bytes"
 	"encoding/json"
 	"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
@@ -103,6 +104,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)