diff options
author | Runxi Yu <me@runxiyu.org> | 2025-02-13 09:33:19 +0800 |
---|---|---|
committer | Runxi Yu <me@runxiyu.org> | 2025-02-13 09:33:19 +0800 |
commit | 91ca7bf1baf7ab077bdd63a7a3930c15af5be325 (patch) | |
tree | 1c670aa1da48f1e3edc7144a0c63231aa7b465e2 /http_handle_repo_log.go | |
parent | TODO: Fix diff view (diff) | |
download | forge-91ca7bf1baf7ab077bdd63a7a3930c15af5be325.tar.gz forge-91ca7bf1baf7ab077bdd63a7a3930c15af5be325.tar.zst forge-91ca7bf1baf7ab077bdd63a7a3930c15af5be325.zip |
http_*.go: Use http.Error
Diffstat (limited to '')
-rw-r--r-- | http_handle_repo_log.go | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/http_handle_repo_log.go b/http_handle_repo_log.go index 0f79ecb..bdc0d78 100644 --- a/http_handle_repo_log.go +++ b/http_handle_repo_log.go @@ -1,7 +1,6 @@ package main import ( - "fmt" "net/http" "github.com/go-git/go-git/v5/plumbing" @@ -12,26 +11,26 @@ func handle_repo_log(w http.ResponseWriter, r *http.Request, params map[string]a group_name, repo_name, ref_name := params["group_name"].(string), params["repo_name"].(string), params["ref"].(string) repo, description, err := open_git_repo(r.Context(), group_name, repo_name) if err != nil { - fmt.Fprintln(w, "Error opening repo:", err.Error()) + http.Error(w, "Error opening repo:: "+err.Error(), http.StatusInternalServerError) return } params["repo_description"] = description ref, err := repo.Reference(plumbing.NewBranchReferenceName(ref_name), true) if err != nil { - fmt.Fprintln(w, "Error getting repo reference:", err.Error()) + http.Error(w, "Error getting repo reference:: "+err.Error(), http.StatusInternalServerError) return } ref_hash := ref.Hash() commits, err := get_recent_commits(repo, ref_hash, -1) if err != nil { - fmt.Fprintln(w, "Error getting recent commits:", err.Error()) + http.Error(w, "Error getting recent commits:: "+err.Error(), http.StatusInternalServerError) return } params["commits"] = commits err = templates.ExecuteTemplate(w, "repo_log", params) if err != nil { - fmt.Fprintln(w, "Error rendering template:", err.Error()) + http.Error(w, "Error rendering template:: "+err.Error(), http.StatusInternalServerError) return } } |