change []error to []string to cover all errors

Signed-off-by: Tarun Pothulapati <tarun@gitpod.io>

Author: Pothulapati

plugins/deployed-labeler/main.go

@@ -106,25 +106,19 @@ func (s *server) markDeployedPR(w http.ResponseWriter, req *http.Request) {
 	}
 
 	if team == "" || commitSHA == "" {
-
-		var preconditionFailed struct {
-			Errors []string
-		}
-
-		preconditionFailed.Errors = make([]string, 0, 1)
-		preconditionFailed.Errors = append(preconditionFailed.Errors, fmt.Sprintf("team and commit are required parameters. Team: '%v', commit: '%v'", team, commitSHA))
-
 		s.log.WithFields(
 			logrus.Fields{
 				"team":   team,
 				"commit": commitSHA,
 			},
 		).Error("team and commit are required parameters")
 
-		w.Header().Set("Content-Type", "application/json; charset=utf-8")
-		w.Header().Set("X-Content-Type-Options", "nosniff")
 		w.WriteHeader(http.StatusPreconditionFailed)
-		json.NewEncoder(w).Encode(preconditionFailed)
+		enc := json.NewEncoder(w)
+		enc.SetIndent("", "  ")
+		enc.Encode(map[string]string{
+			"error": "team and commit are required parameters",
+		})
 		return
 	}
 
@@ -161,9 +155,9 @@ func (s *server) markDeployedPR(w http.ResponseWriter, req *http.Request) {
 				Team []string `json:"team"`
 				All  []string `json:"all"`
 			} `json:"deployedPRs"`
-			Errors []error `json:"errors"`
+			Errors []string `json:"errors"`
 		}
-		msg.DeployedPRs.Team, msg.DeployedPRs.All, msg.Errors = s.handleMarkDeployedPRs(req.Context(), commitSHA, team)
+		msg.DeployedPRs.Team, msg.DeployedPRs.All, errs = s.handleMarkDeployedPRs(req.Context(), commitSHA, team)
 
 		msg.Errors = make([]string, 0, len(errs))
 		for _, err := range errs {