Fix panic if forge is unreachable (#3944)

woodpecker-ci · Jul 20, 2024 · 8e2ff33 · 8e2ff33
1 parent 54b91db
commit 8e2ff33
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/server/api/pipeline.go b/server/api/pipeline.go
@@ -64,7 +64,11 @@ func CreatePipeline(c *gin.Context) {
 
 	user := session.User(c)
 
-	lastCommit, _ := _forge.BranchHead(c, user, repo, opts.Branch)
+	lastCommit, err := _forge.BranchHead(c, user, repo, opts.Branch)
+	if err != nil {
+		_ = c.AbortWithError(http.StatusInternalServerError, fmt.Errorf("could not fetch branch head: %w", err))
+		return
+	}
 
 	tmpPipeline := createTmpPipeline(model.EventManual, lastCommit, user, &opts)