diff options
author | Runxi Yu <me@runxiyu.org> | 2025-02-14 11:29:49 +0800 |
---|---|---|
committer | Runxi Yu <me@runxiyu.org> | 2025-02-14 11:31:32 +0800 |
commit | 3f44e5773e9edd3044cd5235a36a06d01de13e31 (patch) | |
tree | 26fdb7daa55e3b5c9e69aa57592eb7059d365810 /http_handle_repo_log.go | |
parent | http_handle_repo_*.go: Remove redundant return (diff) | |
download | forge-3f44e5773e9edd3044cd5235a36a06d01de13e31.tar.gz forge-3f44e5773e9edd3044cd5235a36a06d01de13e31.tar.zst forge-3f44e5773e9edd3044cd5235a36a06d01de13e31.zip |
repo_log: Use parameters for refspec
Diffstat (limited to '')
-rw-r--r-- | http_handle_repo_log.go | 12 |
1 files changed, 5 insertions, 7 deletions
diff --git a/http_handle_repo_log.go b/http_handle_repo_log.go index b37923b..cbc3284 100644 --- a/http_handle_repo_log.go +++ b/http_handle_repo_log.go @@ -2,25 +2,23 @@ package main import ( "net/http" - - "github.com/go-git/go-git/v5/plumbing" ) // TODO: I probably shouldn't include *all* commits here... func handle_repo_log(w http.ResponseWriter, r *http.Request, params map[string]any) { - group_name, repo_name, ref_name := params["group_name"].(string), params["repo_name"].(string), params["ref_name"].(string) - repo, description, err := open_git_repo(r.Context(), group_name, repo_name) + repo, description, err := open_git_repo(r.Context(), params["group_name"].(string), params["repo_name"].(string)) if err != nil { http.Error(w, "Error opening repo: "+err.Error(), http.StatusInternalServerError) return } params["repo_description"] = description - ref, err := repo.Reference(plumbing.NewBranchReferenceName(ref_name), true) + + ref_hash, err := get_ref_hash_from_type_and_name(repo, params["ref_type"].(string), params["ref_name"].(string)) if err != nil { - http.Error(w, "Error getting repo reference: "+err.Error(), http.StatusInternalServerError) + http.Error(w, "Error getting ref hash: "+err.Error(), http.StatusInternalServerError) return } - ref_hash := ref.Hash() + commits, err := get_recent_commits(repo, ref_hash, -1) if err != nil { http.Error(w, "Error getting recent commits: "+err.Error(), http.StatusInternalServerError) |