diff options
Diffstat (limited to 'internal/authz')
| -rw-r--r-- | internal/authz/middleware.go | 8 | ||||
| -rw-r--r-- | internal/authz/middleware_test.go | 2 | ||||
| -rw-r--r-- | internal/authz/model.go | 3 |
3 files changed, 7 insertions, 6 deletions
diff --git a/internal/authz/middleware.go b/internal/authz/middleware.go index 3c029b0..9031c99 100644 --- a/internal/authz/middleware.go +++ b/internal/authz/middleware.go @@ -5,7 +5,7 @@ import ( "context" "encoding/hex" "fmt" - "log" + "log/slog" "net/http" "git.ofmax.li/go-git-server/internal/admin" @@ -60,16 +60,16 @@ func Authorization(adminSvc *admin.Servicer, next http.Handler) http.Handler { action := req.Method ok, err := adminSvc.Enforce(urn, repo, action) if err != nil { - log.Printf("error running enforce %s", err) + slog.Info("error unning enforce", "error", err) http.Error(rw, "Bad Request", http.StatusBadRequest) return } if !ok { - log.Printf("Not Authorized - %s attempted access %s", urn, repo) + slog.Info("Not Authorized", "urn", urn, "repo", repo) http.Error(rw, "Access denied", http.StatusForbidden) return } - log.Printf("Method %s Url %s", action, repo) + slog.Debug("Access Attempt", "action", action, "repo", repo) next.ServeHTTP(rw, req.WithContext(ctx)) }) } diff --git a/internal/authz/middleware_test.go b/internal/authz/middleware_test.go index 99ca0b0..3dfa997 100644 --- a/internal/authz/middleware_test.go +++ b/internal/authz/middleware_test.go @@ -113,7 +113,7 @@ func TestAuthorization(t *testing.T) { body: []byte("Access denied\n"), }, } - svcr := admin.NewService( + svcr, _ := admin.NewService( "../../auth_model.ini", "../../tests/testpolicy.csv", "../../gitserver.yaml", diff --git a/internal/authz/model.go b/internal/authz/model.go index 1ed61cf..32730c2 100644 --- a/internal/authz/model.go +++ b/internal/authz/model.go @@ -5,6 +5,7 @@ import ( "encoding/csv" "encoding/hex" "fmt" + "log/slog" "math/big" "os" @@ -24,7 +25,7 @@ func (tm TokenMap) LoadTokensFromFile(path string) error { // TODO this should be configurable contents, err := os.Open(path) if err != nil { - fmt.Println("File reading error", err) + slog.Error("File reading error", err) return err } defer contents.Close() |