From f72f158cf0141df9fad75d1126a310edab96cc1b Mon Sep 17 00:00:00 2001
From: Abhinav Raut <abhinavrautcs@gmail.com>
Date: Tue, 19 Aug 2025 03:01:21 +0530
Subject: [PATCH] - show thumbnail image in widget thread instead of the entire
 image - update file imports to use shared-ui utils and remove redundant
 file.js - Implement SignedURLStore interface for fs store

---
 .../message/attachment/AttachmentsPreview.vue |  2 +-
 .../attachment/FileAttachmentPreview.vue      |  2 +-
 .../attachment/ImageAttachmentPreview.vue     |  2 +-
 frontend/apps/main/src/utils/file.js          | 13 ----
 .../src/components/MessageAttachment.vue      |  9 ++-
 frontend/shared-ui/utils/file.js              |  6 ++
 internal/media/media.go                       |  5 +-
 internal/media/stores/localfs/fs.go           | 59 +++++++++++++++++++
 8 files changed, 79 insertions(+), 19 deletions(-)
 delete mode 100644 frontend/apps/main/src/utils/file.js

diff --git a/frontend/apps/main/src/features/conversation/message/attachment/AttachmentsPreview.vue b/frontend/apps/main/src/features/conversation/message/attachment/AttachmentsPreview.vue
index f3924f70..9b9e1d26 100644
--- a/frontend/apps/main/src/features/conversation/message/attachment/AttachmentsPreview.vue
+++ b/frontend/apps/main/src/features/conversation/message/attachment/AttachmentsPreview.vue
@@ -42,7 +42,7 @@
 
 <script setup>
 import { computed } from 'vue'
-import { formatBytes } from '../../../../utils/file.js'
+import { formatBytes } from '@shared-ui/utils/file'
 import { X, Paperclip as PaperclipIcon } from 'lucide-vue-next'
 import { DotLoader } from '@shared-ui/components/ui/loader'
 import { Tooltip, TooltipContent, TooltipTrigger } from '@shared-ui/components/ui/tooltip'
diff --git a/frontend/apps/main/src/features/conversation/message/attachment/FileAttachmentPreview.vue b/frontend/apps/main/src/features/conversation/message/attachment/FileAttachmentPreview.vue
index 4fde208c..ed06d0c9 100644
--- a/frontend/apps/main/src/features/conversation/message/attachment/FileAttachmentPreview.vue
+++ b/frontend/apps/main/src/features/conversation/message/attachment/FileAttachmentPreview.vue
@@ -22,7 +22,7 @@
 </template>
 
 <script setup>
-import { formatBytes } from '../../../../utils/file.js'
+import { formatBytes } from '@shared-ui/utils/file'
 import { Download } from 'lucide-vue-next';
 
 const props = defineProps({
diff --git a/frontend/apps/main/src/features/conversation/message/attachment/ImageAttachmentPreview.vue b/frontend/apps/main/src/features/conversation/message/attachment/ImageAttachmentPreview.vue
index 788cc5fa..9487c89e 100644
--- a/frontend/apps/main/src/features/conversation/message/attachment/ImageAttachmentPreview.vue
+++ b/frontend/apps/main/src/features/conversation/message/attachment/ImageAttachmentPreview.vue
@@ -18,7 +18,7 @@
 </template>
 
 <script setup>
-import { formatBytes, getThumbFilepath } from '../../../../utils/file.js'
+import { formatBytes, getThumbFilepath } from '@shared-ui/utils/file'
 import { Download } from 'lucide-vue-next';
 
 const props = defineProps({
diff --git a/frontend/apps/main/src/utils/file.js b/frontend/apps/main/src/utils/file.js
deleted file mode 100644
index 85433115..00000000
--- a/frontend/apps/main/src/utils/file.js
+++ /dev/null
@@ -1,13 +0,0 @@
-export function formatBytes(bytes) {
-  if (bytes < 1024 * 1024) {
-    return (bytes / 1024).toFixed(2) + ' KB'
-  } else {
-    return (bytes / (1024 * 1024)).toFixed(2) + ' MB'
-  }
-}
-
-export function getThumbFilepath (filepath) {
-  const urlParts = filepath.split('/')
-  const filename = urlParts.pop()
-  return `/uploads/thumb_${filename}`
-}
diff --git a/frontend/apps/widget/src/components/MessageAttachment.vue b/frontend/apps/widget/src/components/MessageAttachment.vue
index f915ec5e..c450deda 100644
--- a/frontend/apps/widget/src/components/MessageAttachment.vue
+++ b/frontend/apps/widget/src/components/MessageAttachment.vue
@@ -8,7 +8,7 @@
       <!-- Image preview -->
       <div v-if="isImage(attachment)" class="relative">
         <img
-          :src="attachment.url"
+          :src="getThumbnailUrl(attachment)"
           :alt="attachment.name"
           class="max-w-48 max-h-32 rounded-lg object-cover"
           @click="openImage(attachment.url)"
@@ -35,7 +35,7 @@
 
 <script setup>
 import { File } from 'lucide-vue-next';
-import { formatBytes } from '@shared-ui/utils/file';
+import { formatBytes, getThumbFilepath } from '@shared-ui/utils/file';
 defineProps({
   attachments: {
     type: Array,
@@ -47,6 +47,11 @@ const isImage = (attachment) => {
   return attachment.content_type && attachment.content_type.startsWith('image/')
 }
 
+const getThumbnailUrl = (attachment) => {
+  if (!isImage(attachment)) return attachment.url
+  return getThumbFilepath(attachment.url)
+}
+
 const openImage = (url) => {
   window.open(url, '_blank')
 }
diff --git a/frontend/shared-ui/utils/file.js b/frontend/shared-ui/utils/file.js
index a2a8b6ff..73db96a9 100644
--- a/frontend/shared-ui/utils/file.js
+++ b/frontend/shared-ui/utils/file.js
@@ -5,3 +5,9 @@ export function formatBytes (bytes) {
         return (bytes / (1024 * 1024)).toFixed(2) + ' MB'
     }
 }
+
+export function getThumbFilepath (filepath) {
+    const urlParts = filepath.split('/')
+    const filename = urlParts.pop()
+    return `/uploads/thumb_${filename}`
+}
diff --git a/internal/media/media.go b/internal/media/media.go
index 1ff4c322..04d5dbe7 100644
--- a/internal/media/media.go
+++ b/internal/media/media.go
@@ -9,6 +9,7 @@ import (
 	"fmt"
 	"io"
 	"os"
+	"strings"
 	"time"
 
 	"github.com/abhinavxd/libredesk/internal/dbutil"
@@ -266,7 +267,9 @@ func (m *Manager) VerifySignature(r *fastglue.Request) error {
 	
 	// Check if store supports signature verification
 	if signedStore, ok := m.store.(SignedURLStore); ok {
-		if !signedStore.VerifySignature(uuid.(string), signature, expiresAt, []byte(m.secret)) {
+		// Strip thumb_ prefix for signature verification to match the base UUID
+		verificationName := strings.TrimPrefix(uuid.(string), "thumb_")
+		if !signedStore.VerifySignature(verificationName, signature, expiresAt, []byte(m.secret)) {
 			return fmt.Errorf("signature verification failed")
 		}
 		return nil
diff --git a/internal/media/stores/localfs/fs.go b/internal/media/stores/localfs/fs.go
index 372f386f..de6d6883 100644
--- a/internal/media/stores/localfs/fs.go
+++ b/internal/media/stores/localfs/fs.go
@@ -1,10 +1,17 @@
 package fs
 
 import (
+	"crypto/hmac"
+	"crypto/sha256"
+	"crypto/subtle"
+	"encoding/base64"
 	"fmt"
 	"io"
+	"net/url"
 	"os"
 	"path/filepath"
+	"strconv"
+	"time"
 
 	"github.com/abhinavxd/libredesk/internal/media"
 )
@@ -73,6 +80,58 @@ func (c *Client) Name() string {
 	return "fs"
 }
 
+// GetSignedURL generates a signed URL for the file with expiration.
+// This implements the SignedURLStore interface for secure public access.
+func (c *Client) GetSignedURL(name string, expiresAt time.Time, secret []byte) string {
+	// Generate base URL
+	baseURL := c.GetURL(name)
+	
+	// Create the signature payload: name + expires timestamp
+	expires := expiresAt.Unix()
+	payload := name + strconv.FormatInt(expires, 10)
+	
+	// Generate HMAC-SHA256 signature
+	h := hmac.New(sha256.New, secret)
+	h.Write([]byte(payload))
+	signature := base64.URLEncoding.EncodeToString(h.Sum(nil))
+	
+	// Parse base URL and add query parameters
+	u, err := url.Parse(baseURL)
+	if err != nil {
+		// Fallback to base URL if parsing fails
+		return baseURL
+	}
+	
+	// Add signature and expires parameters
+	query := u.Query()
+	query.Set("signature", signature)
+	query.Set("expires", strconv.FormatInt(expires, 10))
+	u.RawQuery = query.Encode()
+	
+	return u.String()
+}
+
+// VerifySignature verifies that a signature is valid for the given parameters.
+// This implements the SignedURLStore interface for secure public access.
+func (c *Client) VerifySignature(name, signature string, expiresAt time.Time, secret []byte) bool {
+	// Check if URL has expired
+	if time.Now().After(expiresAt) {
+		return false
+	}
+	
+	// Recreate the signature payload: name + expires timestamp
+	expires := expiresAt.Unix()
+	payload := name + strconv.FormatInt(expires, 10)
+	
+	// Generate expected HMAC-SHA256 signature
+	h := hmac.New(sha256.New, secret)
+	h.Write([]byte(payload))
+	expectedSignature := base64.URLEncoding.EncodeToString(h.Sum(nil))
+	
+	// Use constant-time comparison to prevent timing attacks
+	return subtle.ConstantTimeCompare([]byte(signature), []byte(expectedSignature)) == 1
+}
+
 // getDir returns the current working directory path if no directory is specified,
 // else returns the directory path specified itself.
 func getDir(dir string) string {