open-code-review/internal/viewer/handler.go
kite 533b526b4c
Some checks are pending
CI / cross-compile (arm64, darwin) (push) Waiting to run
CI / cross-compile (arm64, linux) (push) Waiting to run
CI / cross-compile (arm64, windows) (push) Waiting to run
CI / test (push) Waiting to run
CI / cross-compile (amd64, darwin) (push) Waiting to run
CI / cross-compile (amd64, windows) (push) Waiting to run
CodeQL Advanced / Analyze (go) (push) Waiting to run
CodeQL Advanced / Analyze (actions) (push) Waiting to run
CodeQL Advanced / Analyze (javascript-typescript) (push) Waiting to run
Deploy Pages / build (push) Waiting to run
Deploy Pages / deploy (push) Blocked by required conditions
chore: add SPDX license headers and automated verification (#740)
* chore: add SPDX license headers to all source files

Add Apache-2.0 SPDX license identifiers and copyright notices to all
tracked .go, .sh, .js, .mjs, .ts, and .tsx source files.

Introduce scripts/verify-license.sh and scripts/add-license.sh for
automated verification and bulk addition of license headers. Integrate
the check into CI (ci.yml) and the Makefile (license-check target as
a prerequisite of the existing check target).

This satisfies the OpenSSF Best Practices Badge requirements for
copyright_per_file and license_per_file.

* fix: restore execute permissions on scripts

* docs: add license header instructions to CONTRIBUTING guides

* docs: add license header instructions to pages contributing guides

* fix(pages): strip unclosed HTML comment markers to satisfy CodeQL

* fix: apply code review suggestions for license scripts

- Fix portability: detect macOS vs Linux stat for permission copy
- Fix has_header: check both SPDX and copyright (match verify logic)
- Fix is_ignored: match on path boundaries to avoid false positives
- Fix year extraction: use consistent pipeline across both scripts
- Fix Bash 3.2 compat: quote array length expansion for set -u

* fix(pages): use loop-until-clean for HTML comment stripping (CodeQL)

* fix(pages): use split/join instead of replace to avoid CodeQL false positive

CodeQL's js/incomplete-multi-character-sanitization rule flags any
.replace() that removes multi-character sequences like '<!--...-->',
regardless of context. The data here comes from readFileSync on the
project's own index.html (no untrusted input), making this a false
positive. Using split(regex).join('') achieves the same result without
triggering the taint-tracking rule.
2026-08-05 21:26:27 +08:00

82 lines
1.6 KiB
Go

// SPDX-License-Identifier: Apache-2.0
// Copyright 2026 alibaba/open-code-review Contributors
package viewer
import (
"fmt"
"net/http"
"path/filepath"
)
func handleRepos(w http.ResponseWriter, r *http.Request, root string) {
if r.URL.Path != "/" {
http.NotFound(w, r)
return
}
repos, err := DiscoverRepos(root)
if err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError)
return
}
renderTemplate(w, "repos.html", map[string]any{
"Repos": repos,
})
}
type sessionsData struct {
EncodedRepo string
RepoName string
Sessions []SessionSummary
}
func handleSessions(w http.ResponseWriter, r *http.Request, root, repo string) {
summaries, err := ListSessions(root, repo)
if err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError)
return
}
// Derive a display name from the first session's CWD
name := repo
for _, s := range summaries {
if s.CWD != "" {
name = filepath.Base(s.CWD)
break
}
}
renderTemplate(w, "sessions.html", sessionsData{
EncodedRepo: repo,
RepoName: name,
Sessions: summaries,
})
}
type sessionPageData struct {
EncodedRepo string
RepoName string
Session *ViewSession
}
func handleSession(w http.ResponseWriter, r *http.Request, root, repo, sessionID string) {
vs, err := LoadSession(root, repo, sessionID)
if err != nil {
http.Error(w, fmt.Sprintf("Failed to load session: %v", err), http.StatusNotFound)
return
}
// Derive a display name
name := filepath.Base(vs.Summary.CWD)
if name == "." || name == "" {
name = repo
}
renderTemplate(w, "session.html", sessionPageData{
EncodedRepo: repo,
RepoName: name,
Session: vs,
})
}