feat(handlers/notification): use slogger #14

merged
opened by brookjeynes.dev targeting master from push-trrpxxyxxmot
Changed files
+10 -7
internal
server
handlers
+10 -7
internal/server/handlers/notification.go
···
package handlers
import (
-
"log"
"net/http"
"strconv"
···
)
func (h *Handler) HandleNotificationFeed(w http.ResponseWriter, r *http.Request) {
user, err := bsky.GetUserWithBskyProfile(h.Oauth, r)
if err != nil {
-
log.Println("failed to get logged-in user:", err)
htmx.HxRedirect(w, "/login")
return
}
···
}
page, err := strconv.ParseInt(pageStr, 10, 64)
if err != nil {
-
log.Println("failed to parse page value:", err)
page = 1
}
if page == 0 {
···
case http.MethodGet:
notifications, err := db.GetNotificationsByDid(h.Db, user.Did, pageSize+1, int(offset))
if err != nil {
-
log.Println("failed to retrieve notifications:", err)
htmx.HxError(w, http.StatusInternalServerError, "Failed to get notifications, try again later.")
return
}
hydratedNotifications, err := h.getBskyProfileHydratedNotificationFeed(notifications)
if err != nil {
-
log.Println("failed to hydrate notifications with bsky profile:", err)
htmx.HxError(w, http.StatusInternalServerError, "Failed to get notifications, try again later.")
return
}
···
}
func (h *Handler) HandleNotificationMarkAllRead(w http.ResponseWriter, r *http.Request) {
user, err := bsky.GetUserWithBskyProfile(h.Oauth, r)
if err != nil {
-
log.Println("failed to get logged-in user:", err)
htmx.HxRedirect(w, "/login")
return
}
err = db.MarkAllNotificationsAsRead(h.Db, user.Did)
if err != nil {
-
log.Println("failed to mark all notifications:", err)
htmx.HxError(w, http.StatusInternalServerError, "Failed to mark all notifications as read, try again later.")
return
}
···
package handlers
import (
"net/http"
"strconv"
···
)
func (h *Handler) HandleNotificationFeed(w http.ResponseWriter, r *http.Request) {
+
l := h.Logger.With("handler", "HandleNotificationFeed")
+
user, err := bsky.GetUserWithBskyProfile(h.Oauth, r)
if err != nil {
+
l.Error("failed to get logged-in user", "err", err)
htmx.HxRedirect(w, "/login")
return
}
···
}
page, err := strconv.ParseInt(pageStr, 10, 64)
if err != nil {
+
l.Error("failed to parse page value", "err", err)
page = 1
}
if page == 0 {
···
case http.MethodGet:
notifications, err := db.GetNotificationsByDid(h.Db, user.Did, pageSize+1, int(offset))
if err != nil {
+
l.Error("failed to retrieve notifications", "err", err)
htmx.HxError(w, http.StatusInternalServerError, "Failed to get notifications, try again later.")
return
}
hydratedNotifications, err := h.getBskyProfileHydratedNotificationFeed(notifications)
if err != nil {
+
l.Error("failed to hydrate notifications with bsky profile", "err", err)
htmx.HxError(w, http.StatusInternalServerError, "Failed to get notifications, try again later.")
return
}
···
}
func (h *Handler) HandleNotificationMarkAllRead(w http.ResponseWriter, r *http.Request) {
+
l := h.Logger.With("handler", "HandleNotificationMarkAllRead")
+
user, err := bsky.GetUserWithBskyProfile(h.Oauth, r)
if err != nil {
+
l.Error("failed to get logged-in user", "err", err)
htmx.HxRedirect(w, "/login")
return
}
err = db.MarkAllNotificationsAsRead(h.Db, user.Did)
if err != nil {
+
l.Error("failed to mark all notifications", "err", err)
htmx.HxError(w, http.StatusInternalServerError, "Failed to mark all notifications as read, try again later.")
return
}