Skip to content

Commit 3ec2dde

Browse files
author
Codex
committed
fix: use POST for delete instead of DELETE method (CSRF compatibility)
1 parent 4948365 commit 3ec2dde

3 files changed

Lines changed: 7 additions & 7 deletions

File tree

internal/web/handlers/server.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ func NewServer(cfg web.Config, db *sql.DB) *http.Server {
7878

7979
r.Post("/items/{id}/reclassify", ReclassifyItemHandler(cfg, db))
8080
r.Post("/items/{id}", UpdateItemHandler(db))
81-
r.Delete("/items/{id}", DeleteItemHandler(db))
81+
r.Post("/items/{id}/delete", DeleteItemHandler(db))
8282
r.Get("/items/{id}/edit", EditItemForm(db))
8383
r.Post("/generate", GenerateReport(cfg, db))
8484
r.Get("/generate/{jobID}/status", GenerateStatus())

internal/web/templates/item_row.templ

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -75,9 +75,9 @@ templ ItemRow(item ItemData, allSections []SectionData, isManager bool) {
7575
title="Edit"
7676
>&#9998;</button>
7777
<button
78-
hx-delete={ fmt.Sprintf("/items/%d", item.ID) }
78+
hx-post={ fmt.Sprintf("/items/%d/delete", item.ID) }
7979
hx-target={ fmt.Sprintf("#item-%d", item.ID) }
80-
hx-swap="outerHTML"
80+
hx-swap="delete"
8181
hx-confirm="Delete this item?"
8282
title="Delete"
8383
>&times;</button>

internal/web/templates/item_row_templ.go

Lines changed: 4 additions & 4 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)