fix: restructure handler yet again #181

handler/auth.go

@@ -1,7 +1,6 @@
 package handler
 
 import (
-	"log/slog"
 	"me-fit/service"
 
 	"database/sql"
@@ -9,7 +8,6 @@ import (
 )
 
 func authUi(db *sql.DB) http.Handler {
-
 	router := http.NewServeMux()
 
 	router.Handle("/auth/signin", service.HandleSignInPage(db))
@@ -21,15 +19,10 @@ func authUi(db *sql.DB) http.Handler {
 	router.Handle("/auth/reset-password", service.HandleResetPasswordPage(db))
 	router.Handle("/", service.HandleIndexAnd404(db))
 
-	// return router
-	return http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		slog.Warn(r.URL.Path)
-		router.ServeHTTP(w, r)
-	})
+	return router
 }
 
 func authApi(db *sql.DB) http.Handler {
-
 	router := http.NewServeMux()
 
 	router.Handle("/api/auth/signup", service.HandleSignUpComp(db))

handler/default.go

@@ -21,6 +21,7 @@ func GetHandler(db *sql.DB) http.Handler {
 
 	router.Handle("/workout", auth(db, workoutUi(db)))
 	router.Handle("/api/workout", auth(db, workoutApi(db)))
+	// Needed a second time with trailing slash, otherwise either /api/workout or /api/workout/{id} does not match
 	router.Handle("/api/workout/", auth(db, workoutApi(db)))
 
 	return middleware.Logging(

handler/workout.go

@@ -1,7 +1,6 @@
 package handler
 
 import (
-	"log/slog"
 	"me-fit/service"
 
 	"database/sql"
@@ -9,11 +8,9 @@ import (
 )
 
 func workoutUi(db *sql.DB) http.Handler {
-
 	router := http.NewServeMux()
 
 	router.Handle("/workout", service.HandleWorkoutPage(db))
-	router.Handle("/", service.HandleIndexAnd404(db))
 
 	return router
 }
@@ -21,14 +18,9 @@ func workoutUi(db *sql.DB) http.Handler {
 func workoutApi(db *sql.DB) http.Handler {
 	router := http.NewServeMux()
 
-	// root = "/api/workout/"
 	router.Handle("POST /api/workout", service.HandleWorkoutNewComp(db))
 	router.Handle("GET /api/workout", service.HandleWorkoutGetComp(db))
 	router.Handle("DELETE /api/workout/{id}", service.HandleWorkoutDeleteComp(db))
 
-	// return router
-	return http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		slog.Warn(r.URL.Path)
-		router.ServeHTTP(w, r)
-	})
+	return router
 }