aboutsummaryrefslogtreecommitdiff
path: root/internal/ui/oauth2_redirect.go
diff options
context:
space:
mode:
authorGravatar Frédéric Guillot <f@miniflux.net> 2023-09-24 16:32:09 -0700
committerGravatar Frédéric Guillot <f@miniflux.net> 2023-09-24 22:37:33 -0700
commitc0e954f19d707fef8ef8271636ec661634a4c4c7 (patch)
tree5aa052a048f470e233a454e5ad9071eed1fa37c0 /internal/ui/oauth2_redirect.go
parent54cb8fa0286e4a2f1a81c32b5a89722d93b30bf7 (diff)
downloadv2-c0e954f19d707fef8ef8271636ec661634a4c4c7.tar.gz
v2-c0e954f19d707fef8ef8271636ec661634a4c4c7.tar.zst
v2-c0e954f19d707fef8ef8271636ec661634a4c4c7.zip
Implement structured logging using log/slog package
Diffstat (limited to 'internal/ui/oauth2_redirect.go')
-rw-r--r--internal/ui/oauth2_redirect.go9
1 files changed, 6 insertions, 3 deletions
diff --git a/internal/ui/oauth2_redirect.go b/internal/ui/oauth2_redirect.go
index 622b544c..78f9fd1a 100644
--- a/internal/ui/oauth2_redirect.go
+++ b/internal/ui/oauth2_redirect.go
@@ -4,12 +4,12 @@
package ui // import "miniflux.app/v2/internal/ui"
import (
+ "log/slog"
"net/http"
"miniflux.app/v2/internal/http/request"
"miniflux.app/v2/internal/http/response/html"
"miniflux.app/v2/internal/http/route"
- "miniflux.app/v2/internal/logger"
"miniflux.app/v2/internal/oauth2"
"miniflux.app/v2/internal/ui/session"
)
@@ -19,14 +19,17 @@ func (h *handler) oauth2Redirect(w http.ResponseWriter, r *http.Request) {
provider := request.RouteStringParam(r, "provider")
if provider == "" {
- logger.Error("[OAuth2] Invalid or missing provider: %s", provider)
+ slog.Warn("Invalid or missing OAuth2 provider")
html.Redirect(w, r, route.Path(h.router, "login"))
return
}
authProvider, err := getOAuth2Manager(r.Context()).FindProvider(provider)
if err != nil {
- logger.Error("[OAuth2] %v", err)
+ slog.Error("Unable to initialize OAuth2 provider",
+ slog.String("provider", provider),
+ slog.Any("error", err),
+ )
html.Redirect(w, r, route.Path(h.router, "login"))
return
}