From 07460aaaca61b61ce29e98c41528b2f60331365f Mon Sep 17 00:00:00 2001 From: "Gustavo \"Guz\" L de Mello" Date: Tue, 25 Mar 2025 14:31:21 -0300 Subject: [PATCH] refactor(router): add trailing slash to all endpoints --- router/projects.go | 14 +++++++------- router/router.go | 7 ++++--- templates/layouts/layout_page.html | 2 +- 3 files changed, 12 insertions(+), 11 deletions(-) diff --git a/router/projects.go b/router/projects.go index efb8882..4819f84 100644 --- a/router/projects.go +++ b/router/projects.go @@ -17,7 +17,7 @@ func (router *router) projects(w http.ResponseWriter, r *http.Request) { switch getMethod(r) { case http.MethodGet, http.MethodHead: - if id := r.PathValue("id"); id != "" { + if id := r.PathValue("ID"); id != "" { router.getProject(w, r) } else { router.listProjects(w, r) @@ -27,11 +27,11 @@ func (router *router) projects(w http.ResponseWriter, r *http.Request) { router.createProject(w, r) case http.MethodDelete: - if id := r.PathValue("id"); id != "" { + if id := r.PathValue("ID"); id != "" { router.deleteProject(w, r) } else { exception. - BadRequest(errors.New(`missing "id" path value`)). + BadRequest(errors.New(`missing "ID" path value`)). ServeHTTP(w, r) } @@ -81,10 +81,10 @@ func (router *router) getProject(w http.ResponseWriter, r *http.Request) { return } - id := r.PathValue("id") + id := r.PathValue("ID") if id == "" { exception. - BadRequest(fmt.Errorf(`a valid path value of "id" must be provided`)). + BadRequest(fmt.Errorf(`a valid path value of "ID" must be provided`)). ServeHTTP(w, r) return } @@ -154,10 +154,10 @@ func (router *router) deleteProject(w http.ResponseWriter, r *http.Request) { return } - id := r.PathValue("id") + id := r.PathValue("ID") if id == "" { exception. - BadRequest(fmt.Errorf(`a valid path value of "id" must be provided`)). + BadRequest(fmt.Errorf(`a valid path value of "ID" must be provided`)). ServeHTTP(w, r) return } diff --git a/router/router.go b/router/router.go index e484b9a..866640a 100644 --- a/router/router.go +++ b/router/router.go @@ -91,11 +91,12 @@ func (router *router) setup() http.Handler { r.Use(exception.PanicMiddleware()) r.Use(exception.Middleware()) - r.Handle("/assets", http.StripPrefix("/assets/", http.FileServerFS(router.assets))) + r.Handle("/assets/", http.StripPrefix("/assets/", http.FileServerFS(router.assets))) - r.HandleFunc("/dashboard", router.dashboard) + r.HandleFunc("/dashboard/", router.dashboard) - r.HandleFunc("/projects/{id...}", router.projects) + r.HandleFunc("/projects/{$}", router.projects) + r.HandleFunc("/projects/{ID}/", router.projects) return r } diff --git a/templates/layouts/layout_page.html b/templates/layouts/layout_page.html index 2e329ba..af0dc82 100644 --- a/templates/layouts/layout_page.html +++ b/templates/layouts/layout_page.html @@ -5,7 +5,7 @@

Comicverse

{{end}}