infra/binary_file_repository_test.go

@@ -45,3 +45,15 @@ func TestBinaryRepoNoSideEffect(t *testing.T) {
 	require.Equal(t, file2.Name, "name2")
 	require.Equal(t, file2.Data, []byte("222"))
 }
+
+func TestBinaryWithSpaceInName(t *testing.T) {
+	repo := setupBinaryRepo()
+
+	file, err := repo.Create("name with space", []byte("111"), nil)
+	require.NoError(t, err)
+
+	file, err = repo.FindById(file.Id)
+	require.NoError(t, err)
+	require.Equal(t, file.Name, "name with space")
+	require.Equal(t, file.Data, []byte("111"))
+}

web/media_handler.go

@@ -1,6 +1,7 @@
 package web
 
 import (
+	"net/url"
 	"owl-blogs/app"
 
 	"github.com/gofiber/fiber/v2"
@@ -16,6 +17,11 @@ func NewMediaHandler(binaryService *app.BinaryService) *MediaHandler {
 
 func (h *MediaHandler) Handle(c *fiber.Ctx) error {
 	id := c.Params("+")
+	// urldecode
+	id, err := url.PathUnescape(id)
+	if err != nil {
+		return err
+	}
 	binary, err := h.binaryService.FindById(id)
 	if err != nil {
 		return err