From a1634db89fe9097c1739e0c6d11265aa7b8bdd3f Mon Sep 17 00:00:00 2001 From: "Gabriel A. Giovanini" Date: Fri, 13 Dec 2024 17:31:22 +0100 Subject: ref: Better log errors Some error were being ignored and not logged, making debugging what failed impossible. --- pkg/ext/router.go | 2 ++ pkg/service/git.go | 5 +++++ 2 files changed, 7 insertions(+) diff --git a/pkg/ext/router.go b/pkg/ext/router.go index 956254d..e12a40c 100644 --- a/pkg/ext/router.go +++ b/pkg/ext/router.go @@ -3,6 +3,7 @@ package ext import ( "errors" "fmt" + "log/slog" "net/http" "git.gabrielgio.me/cerrado/pkg/service" @@ -38,6 +39,7 @@ func wrapError(next ErrorRequestHandler) http.HandlerFunc { if errors.Is(err, service.ErrRepositoryNotFound) { NotFound(w, r) } else { + slog.Error("Internal Server Error", "error", err) InternalServerError(r, w, err) } } diff --git a/pkg/service/git.go b/pkg/service/git.go index afe6103..12d238e 100644 --- a/pkg/service/git.go +++ b/pkg/service/git.go @@ -8,6 +8,7 @@ import ( "git.gabrielgio.me/cerrado/pkg/config" "git.gabrielgio.me/cerrado/pkg/git" + gogit "github.com/go-git/go-git/v5" "github.com/go-git/go-git/v5/plumbing" "github.com/go-git/go-git/v5/plumbing/object" ) @@ -49,6 +50,10 @@ func (g *GitService) ListRepositories() ([]*Repository, error) { for _, r := range rs { repo, err := git.OpenRepository(r.Path) if err != nil { + if errors.Is(err, gogit.ErrRepositoryNotExists) { + slog.Info("Path does not contain a repository", "path", r.Path) + continue + } return nil, err } -- cgit v1.2.3