Merge pull request #5 from johnnybravo-xyz/master

Fix bug in no-group-job response
zerodha · Feb 25, 2020 · 7bdeaed · 7bdeaed
2 parents b346da0 + a2de1af
commit 7bdeaed
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/http.go b/http.go
@@ -57,12 +57,16 @@ func handleGetGroupStatus(w http.ResponseWriter, r *http.Request) {
 		groupID = chi.URLParam(r, "groupID")
 	)
 
+	if _, err := jobber.Machinery.GetBackend().GetState(groupID); err == redis.ErrNil {
+		sendErrorResponse(w, "group not found", http.StatusNotFound)
+		return
+	}
+
 	res, err := jobber.Machinery.GetBackend().GroupTaskStates(groupID, 0)
 	if err != nil {
 		sysLog.Printf("error fetching group status: %v", err)
 		sendErrorResponse(w, "error fetching group status", http.StatusInternalServerError)
 		return
-
 	}
 
 	var (