diff options
author | Runxi Yu <me@runxiyu.org> | 2025-02-10 13:04:15 +0800 |
---|---|---|
committer | Runxi Yu <me@runxiyu.org> | 2025-02-10 13:04:15 +0800 |
commit | 80b1e2e54224720c7b5f798c8c0a6c7ba4545fa5 (patch) | |
tree | 80b11c951bc02daa4c6d02cf78e02e118c9fce88 /handle_repo_index.go | |
parent | index: List categories (diff) | |
download | forge-80b1e2e54224720c7b5f798c8c0a6c7ba4545fa5.tar.gz forge-80b1e2e54224720c7b5f798c8c0a6c7ba4545fa5.tar.zst forge-80b1e2e54224720c7b5f798c8c0a6c7ba4545fa5.zip |
*.go: Linting
Diffstat (limited to 'handle_repo_index.go')
-rw-r--r-- | handle_repo_index.go | 30 |
1 files changed, 17 insertions, 13 deletions
diff --git a/handle_repo_index.go b/handle_repo_index.go index 20cfb89..89aeba6 100644 --- a/handle_repo_index.go +++ b/handle_repo_index.go @@ -19,19 +19,19 @@ func handle_repo_index(w http.ResponseWriter, r *http.Request) { data["project_name"], data["repo_name"] = project_name, repo_name repo, err := git.PlainOpen(filepath.Join(config.Git.Root, project_name, repo_name+".git")) if err != nil { - w.Write([]byte("Error opening repo: " + err.Error())) + _, _ = w.Write([]byte("Error opening repo: " + err.Error())) return } head, err := repo.Head() if err != nil { - w.Write([]byte("Error getting repo HEAD: " + err.Error())) + _, _ = w.Write([]byte("Error getting repo HEAD: " + err.Error())) return } data["ref"] = head.Name().Short() head_hash := head.Hash() commit_iter, err := repo.Log(&git.LogOptions{From: head_hash}) if err != nil { - w.Write([]byte("Error getting repo commits: " + err.Error())) + _, _ = w.Write([]byte("Error getting repo commits: " + err.Error())) return } recent_commits := make([]*object.Commit, 0) @@ -39,7 +39,7 @@ func handle_repo_index(w http.ResponseWriter, r *http.Request) { for range 3 { this_recent_commit, err := commit_iter.Next() if err != nil { - w.Write([]byte("Error getting a recent commit: " + err.Error())) + _, _ = w.Write([]byte("Error getting a recent commit: " + err.Error())) return } recent_commits = append(recent_commits, this_recent_commit) @@ -47,12 +47,12 @@ func handle_repo_index(w http.ResponseWriter, r *http.Request) { data["commits"] = recent_commits commit_object, err := repo.CommitObject(head_hash) if err != nil { - w.Write([]byte("Error getting commit object: " + err.Error())) + _, _ = w.Write([]byte("Error getting commit object: " + err.Error())) return } tree, err := commit_object.Tree() if err != nil { - w.Write([]byte("Error getting file tree: " + err.Error())) + _, _ = w.Write([]byte("Error getting file tree: " + err.Error())) return } readme_file, err := tree.File("README.md") @@ -60,14 +60,18 @@ func handle_repo_index(w http.ResponseWriter, r *http.Request) { data["readme"] = "There is no README for this repository." } else { readme_file_contents, err := readme_file.Contents() - var readme_rendered_unsafe bytes.Buffer - err = goldmark.Convert([]byte(readme_file_contents), &readme_rendered_unsafe) if err != nil { - readme_rendered_unsafe.WriteString("Unable to render README: " + err.Error()) - return + data["readme"] = "There is no README for this repository." + } else { + var readme_rendered_unsafe bytes.Buffer + err = goldmark.Convert([]byte(readme_file_contents), &readme_rendered_unsafe) + if err != nil { + readme_rendered_unsafe.WriteString("Unable to render README: " + err.Error()) + return + } + readme_rendered_safe := template.HTML(bluemonday.UGCPolicy().SanitizeBytes(readme_rendered_unsafe.Bytes())) + data["readme"] = readme_rendered_safe } - readme_rendered_safe := template.HTML(bluemonday.UGCPolicy().SanitizeBytes(readme_rendered_unsafe.Bytes())) - data["readme"] = readme_rendered_safe } display_git_tree := make([]display_git_tree_entry_t, 0) @@ -91,7 +95,7 @@ func handle_repo_index(w http.ResponseWriter, r *http.Request) { err = templates.ExecuteTemplate(w, "repo_index", data) if err != nil { - w.Write([]byte("Error rendering template: " + err.Error())) + _, _ = w.Write([]byte("Error rendering template: " + err.Error())) return } } |