feat: add authenticated URL delete endpoint

collision.go

@@ -0,0 +1,28 @@
+package main
+
+import (
+	"context"
+	"database/sql"
+
+	"url-short/internal/database"
+	"url-short/internal/shortener"
+)
+
+func hashCollisionDetection(DB *database.Queries, url string, count int, requestContext context.Context) (string, error) {
+	hashURL := shortener.Hash(url, count)
+	shortURLHash := shortener.Shorten(hashURL)
+
+	_, err := DB.SelectURL(requestContext, shortURLHash)
+
+	if err == sql.ErrNoRows {
+		return shortURLHash, nil
+	}
+
+	if err != nil && err != sql.ErrNoRows {
+		return "", err
+	}
+
+	count++
+
+	return hashCollisionDetection(DB, url, count, requestContext)
+}

handlers.go

@@ -1,7 +1,6 @@
 package main
 
 import (
-	"context"
 	"crypto/rand"
 	"database/sql"
 	"encoding/hex"
@@ -15,11 +14,10 @@ import (
 	"strings"
 	"time"
 
-	"url-short/internal/database"
-	"url-short/internal/shortener"
-
 	"github.com/golang-jwt/jwt/v5"
 	"golang.org/x/crypto/bcrypt"
+
+	"url-short/internal/database"
 )
 
 type apiConfig struct {
@@ -114,25 +112,6 @@ func (apiCfg *apiConfig) postLongURL(w http.ResponseWriter, r *http.Request, use
 	})
 }
 
-func hashCollisionDetection(DB *database.Queries, url string, count int, requestContext context.Context) (string, error) {
-	hashURL := shortener.Hash(url, count)
-	shortURLHash := shortener.Shorten(hashURL)
-
-	_, err := DB.SelectURL(requestContext, shortURLHash)
-
-	if err == sql.ErrNoRows {
-		return shortURLHash, nil
-	}
-
-	if err != nil && err != sql.ErrNoRows {
-		return "", err
-	}
-
-	count++
-
-	return hashCollisionDetection(DB, url, count, requestContext)
-}
-
 func (apiCfg *apiConfig) getShortURL(w http.ResponseWriter, r *http.Request) {
 	query := r.PathValue("shortUrl")
 
@@ -147,6 +126,27 @@ func (apiCfg *apiConfig) getShortURL(w http.ResponseWriter, r *http.Request) {
 	http.Redirect(w, r, row.LongUrl, http.StatusMovedPermanently)
 }
 
+func (apiCfg *apiConfig) deleteShortURL(w http.ResponseWriter, r * http.Request, user database.User) {
+	query := r.PathValue("shortUrl")
+
+	if query == "" {
+		respondWithError(w, http.StatusBadRequest, "no short url supplied")
+		return
+	}
+
+	err := apiCfg.DB.DeleteURL(r.Context(), database.DeleteURLParams{
+		UserID: user.ID,
+		ShortUrl: query,
+	})
+
+	if err != nil {
+		respondWithError(w, http.StatusBadRequest,  "could not delete short url")
+		return 
+	}
+
+	w.WriteHeader(http.StatusOK)
+}
+
 func (apiCfg *apiConfig) postAPIUsers(w http.ResponseWriter, r *http.Request) {
 	payload := APIUserRequest{}
 

main.go

@@ -41,6 +41,8 @@ func main() {
 	// url management endpoints
 	mux.HandleFunc("POST /api/v1/data/shorten", apiCfg.authenticationMiddlewear(apiCfg.postLongURL))
 	mux.HandleFunc("GET /api/v1/{shortUrl}", apiCfg.getShortURL)
+	mux.HandleFunc("DELETE /api/v1/{shortUrl}", apiCfg.authenticationMiddlewear(apiCfg.deleteShortURL))
+
 
 	// user management endpoints
 	mux.HandleFunc("POST /api/v1/users", apiCfg.postAPIUsers)

sql/queries/urls.sql

@@ -7,3 +7,8 @@ RETURNING *;
 SELECT * 
 FROM urls
 WHERE short_url = $1;
+
+-- name: DeleteURL :exec
+DELETE FROM urls
+WHERE user_id = $1 AND 
+short_url = $2;