debug: add logging to blob deletion.

Author: sebdeveloper6952

internal/bud02/delete.go

@@ -4,6 +4,8 @@ import (
 	"context"
 	"errors"
 
+	"go.uber.org/zap"
+
 	"github.com/sebdeveloper6952/blossom-server/internal/core"
 )
 
@@ -13,28 +15,43 @@ func DeleteBlob(
 	pubkey string,
 	hash string,
 	authHash string,
+	log *zap.Logger,
 ) error {
 	var (
 		blobs = services.Blob()
 	)
+
+	log.Debug("delete blob request", zap.String("pubkey", pubkey), zap.String("hash", hash))
+
 	blobDescriptor, err := blobs.GetFromHash(ctx, hash)
 	if err != nil {
+		log.Debug("blob not found", zap.String("hash", hash), zap.Error(err))
 		return core.ErrBlobNotFound
 	}
 
 	// only the owner can delete the file
 	if blobDescriptor.Pubkey != pubkey {
+		log.Debug("delete unauthorized: pubkey mismatch",
+			zap.String("owner", blobDescriptor.Pubkey),
+			zap.String("requester", pubkey),
+		)
 		return errors.New("unauthorized")
 	}
 
 	// verify both hashes are the same
 	if hash != authHash {
+		log.Debug("delete unauthorized: hash mismatch",
+			zap.String("hash", hash),
+			zap.String("authHash", authHash),
+		)
 		return errors.New("unauthorized")
 	}
 
 	if err := blobs.DeleteFromHash(ctx, hash); err != nil {
+		log.Error("failed to delete blob", zap.String("hash", hash), zap.Error(err))
 		return err
 	}
 
+	log.Debug("blob deleted", zap.String("hash", hash))
 	return nil
 }

internal/bud02/delete_test.go

@@ -8,6 +8,8 @@ import (
 
 	_ "github.com/mattn/go-sqlite3"
 	"github.com/nbd-wtf/go-nostr"
+	"go.uber.org/zap"
+
 	"github.com/sebdeveloper6952/blossom-server/db"
 	"github.com/sebdeveloper6952/blossom-server/internal/core"
 	"github.com/sebdeveloper6952/blossom-server/internal/pkg/config"
@@ -73,6 +75,7 @@ func TestDeleteBlob(t *testing.T) {
 		pk,
 		authHash,
 		authHash,
+		zap.NewNop(),
 	)
 	assert.NoError(t, err, "delete should succeed")
 
@@ -138,6 +141,7 @@ func TestDeleteBlobNotOwner(t *testing.T) {
 		otherPk,
 		authHash,
 		authHash,
+		zap.NewNop(),
 	)
 	assert.Error(t, err, "delete by non-owner should fail")
 }
@@ -184,6 +188,7 @@ func TestDeleteBlobNotFound(t *testing.T) {
 		pk,
 		"nonexistenthash",
 		"nonexistenthash",
+		zap.NewNop(),
 	)
 	assert.ErrorIs(t, err, core.ErrBlobNotFound, "should return blob not found error")
 }

internal/httpapi/bud02_controller.go

@@ -8,6 +8,8 @@ import (
 	"strconv"
 
 	"github.com/gin-gonic/gin"
+	"go.uber.org/zap"
+
 	bud02 "github.com/sebdeveloper6952/blossom-server/internal/bud02"
 	"github.com/sebdeveloper6952/blossom-server/internal/core"
 )
@@ -103,6 +105,7 @@ func listBlobs(
 
 func deleteBlob(
 	services core.Services,
+	log *zap.Logger,
 ) gin.HandlerFunc {
 	return func(ctx *gin.Context) {
 		if err := bud02.DeleteBlob(
@@ -111,6 +114,7 @@ func deleteBlob(
 			ctx.GetString("pk"),
 			ctx.Param("path"),
 			ctx.GetString("x"),
+			log,
 		); err != nil {
 			ctx.AbortWithStatusJSON(
 				http.StatusBadRequest,

internal/httpapi/routes.go

@@ -79,7 +79,7 @@ func SetupRoutes(
 	r.DELETE(
 		"/:path",
 		nostrAuthMiddleware("delete", log),
-		deleteBlob(services),
+		deleteBlob(services, log),
 	)
 
 	// server stats