fix: only convert commands query param when provided (#1140)

* fix: only convert commands query param when provided * fix: lint
go-vela · Jun 10, 2024 · d086065 · d086065
1 parent a083311
commit d086065
Showing 1 changed file with 11 additions and 5 deletions.
diff --git a/api/build/id_request_token.go b/api/build/id_request_token.go
@@ -113,13 +113,19 @@ func GetIDRequestToken(c *gin.Context) {
 		return
 	}
 
-	commands, err := strconv.ParseBool(c.Query("commands"))
-	if err != nil {
-		retErr := fmt.Errorf("unable to parse 'commands' query parameter as boolean %s: %w", c.Query("commands"), err)
+	commands := false
 
-		util.HandleError(c, http.StatusBadRequest, retErr)
+	var err error
 
-		return
+	if len(c.Query("commands")) > 0 {
+		commands, err = strconv.ParseBool(c.Query("commands"))
+		if err != nil {
+			retErr := fmt.Errorf("unable to parse 'commands' query parameter as boolean %s: %w", c.Query("commands"), err)
+
+			util.HandleError(c, http.StatusBadRequest, retErr)
+
+			return
+		}
 	}
 
 	// retrieve token manager from context