From e9d9f04763be7681c0cfb05b85f0da176c9f7df5 Mon Sep 17 00:00:00 2001
From: kolaente <k@knt.li>
Date: Thu, 30 Jan 2025 17:38:36 +0100
Subject: [PATCH] chore: improve debug logging

---
 pkg/modules/auth/openid/openid.go | 3 ++-
 pkg/routes/api_tokens.go          | 1 +
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/pkg/modules/auth/openid/openid.go b/pkg/modules/auth/openid/openid.go
index a42b59478..c229cf6e3 100644
--- a/pkg/modules/auth/openid/openid.go
+++ b/pkg/modules/auth/openid/openid.go
@@ -111,7 +111,6 @@ func HandleCallback(c echo.Context) error {
 	// Check if the provider exists
 	providerKey := c.Param("provider")
 	provider, err := GetProvider(providerKey)
-	log.Debugf("Provider: %v", provider)
 	if err != nil {
 		return handler.HandleHTTPError(err)
 	}
@@ -119,6 +118,8 @@ func HandleCallback(c echo.Context) error {
 		return c.JSON(http.StatusBadRequest, models.Message{Message: "Provider does not exist"})
 	}
 
+	log.Debugf("Trying to authenticate user using provider: %s", provider.Key)
+
 	provider.Oauth2Config.RedirectURL = cb.RedirectURL
 
 	// Parse the access & ID token
diff --git a/pkg/routes/api_tokens.go b/pkg/routes/api_tokens.go
index 4220dbcf0..0e57d74a0 100644
--- a/pkg/routes/api_tokens.go
+++ b/pkg/routes/api_tokens.go
@@ -79,6 +79,7 @@ func checkAPITokenAndPutItInContext(tokenHeaderValue string, c echo.Context) err
 	}
 
 	if !models.CanDoAPIRoute(c, token) {
+		log.Debugf("[auth] Tried authenticating with token %d but it does not have permission to do this route", token.ID)
 		return echo.NewHTTPError(http.StatusUnauthorized)
 	}