From 47c6062b1c21f7f9f4163e0832c8e893c43c888c Mon Sep 17 00:00:00 2001
From: Ralph Slooten <axllent@gmail.com>
Date: Sat, 26 Oct 2024 23:14:55 +1300
Subject: [PATCH 1/5] Chore: Separate attachments and inline images in download
 nav and badges (#379)

---
 server/ui-src/components/message/Message.vue | 19 ++++++++----
 server/ui-src/views/MessageView.vue          | 31 ++++++++++++++++++--
 2 files changed, 42 insertions(+), 8 deletions(-)

diff --git a/server/ui-src/components/message/Message.vue b/server/ui-src/components/message/Message.vue
index 159a043c36..edb7ea7922 100644
--- a/server/ui-src/components/message/Message.vue
+++ b/server/ui-src/components/message/Message.vue
@@ -457,11 +457,20 @@ export default {
 					</tbody>
 				</table>
 			</div>
-			<div class="col-md-auto d-none d-md-block text-end mt-md-3">
-				<div class="mt-2 mt-md-0" v-if="allAttachments(message)">
-					<span class="badge rounded-pill text-bg-secondary p-2">
-						Attachment<span v-if="allAttachments(message).length > 1">s</span>
-						({{ allAttachments(message).length }})
+			<div class="col-md-auto d-none d-md-block text-end mt-md-3"
+				v-if="message.Attachments && message.Attachments.length || message.Inline && message.Inline.length">
+				<div class="mt-2 mt-md-0">
+					<template v-if="message.Attachments.length">
+						<span class="badge rounded-pill text-bg-secondary p-2 mb-2" title="Attachments in this message">
+							Attachment<span v-if="message.Attachments.length > 1">s</span>
+							({{ message.Attachments.length }})
+						</span>
+						<br>
+					</template>
+					<span class="badge rounded-pill text-bg-secondary p-2" v-if="message.Inline.length"
+						title="Inline images in this message">
+						Inline image<span v-if="message.Inline.length > 1">s</span>
+						({{ message.Inline.length }})
 					</span>
 				</div>
 			</div>
diff --git a/server/ui-src/views/MessageView.vue b/server/ui-src/views/MessageView.vue
index d8040d3b7c..50d9c12b6c 100644
--- a/server/ui-src/views/MessageView.vue
+++ b/server/ui-src/views/MessageView.vue
@@ -504,16 +504,41 @@ export default {
 							Text body
 						</button>
 					</li>
-					<template v-if="allAttachments(message).length">
+					<template v-if="message.Attachments && message.Attachments.length">
 						<li>
 							<hr class="dropdown-divider">
 						</li>
 						<li>
 							<h6 class="dropdown-header">
-								Attachment<template v-if="allAttachments(message).length > 1">s</template>
+								Attachments
 							</h6>
 						</li>
-						<li v-for="part in allAttachments(message)">
+						<li v-for="part in message.Attachments">
+							<RouterLink :to="'/api/v1/message/' + message.ID + '/part/' + part.PartID"
+								class="row m-0 dropdown-item d-flex" target="_blank"
+								:title="part.FileName != '' ? part.FileName : '[ unknown ]'" style="min-width: 350px">
+								<div class="col-auto p-0 pe-1">
+									<i class="bi" :class="attachmentIcon(part)"></i>
+								</div>
+								<div class="col text-truncate p-0 pe-1">
+									{{ part.FileName != '' ? part.FileName : '[ unknown ]' }}
+								</div>
+								<div class="col-auto text-muted small p-0">
+									{{ getFileSize(part.Size) }}
+								</div>
+							</RouterLink>
+						</li>
+					</template>
+					<template v-if="message.Inline && message.Inline.length">
+						<li>
+							<hr class="dropdown-divider">
+						</li>
+						<li>
+							<h6 class="dropdown-header">
+								Inline image<span v-if="message.Inline.length > 1">s</span>
+							</h6>
+						</li>
+						<li v-for="part in message.Inline">
 							<RouterLink :to="'/api/v1/message/' + message.ID + '/part/' + part.PartID"
 								class="row m-0 dropdown-item d-flex" target="_blank"
 								:title="part.FileName != '' ? part.FileName : '[ unknown ]'" style="min-width: 350px">

From bd47c19058e8f63a9bce4e96976975fd0b9e3985 Mon Sep 17 00:00:00 2001
From: Ralph Slooten <axllent@gmail.com>
Date: Sat, 26 Oct 2024 23:33:08 +1300
Subject: [PATCH 2/5] Feature: Add ability to search for messages containing
 inline images (`has:inline`)

---
 internal/storage/search.go | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/internal/storage/search.go b/internal/storage/search.go
index c69a6a6120..4a8a48d328 100644
--- a/internal/storage/search.go
+++ b/internal/storage/search.go
@@ -355,6 +355,12 @@ func searchQueryBuilder(searchString, timezone string) *sqlf.Stmt {
 			} else {
 				q.Where(`m.ID IN (SELECT DISTINCT mt.ID FROM ` + tenant("message_tags") + ` mt JOIN tags t ON mt.TagID = t.ID)`)
 			}
+		} else if lw == "has:inline" || lw == "has:inlines" {
+			if exclude {
+				q.Where("Inline = 0")
+			} else {
+				q.Where("Inline > 0")
+			}
 		} else if lw == "has:attachment" || lw == "has:attachments" {
 			if exclude {
 				q.Where("Attachments = 0")

From 5675abef8439bd1d4da94b54c197e36428ccbdc4 Mon Sep 17 00:00:00 2001
From: Ralph Slooten <axllent@gmail.com>
Date: Sun, 27 Oct 2024 02:15:13 +1300
Subject: [PATCH 3/5] Feature: Add ability to search by size smaller or larger
 than a value (eg: `larger:1M` / `smaller:2.5M`)

---
 internal/storage/search.go      | 54 +++++++++++++++++++++++++++++++++
 internal/storage/search_test.go | 22 ++++++++++++++
 2 files changed, 76 insertions(+)

diff --git a/internal/storage/search.go b/internal/storage/search.go
index 4a8a48d328..1380798545 100644
--- a/internal/storage/search.go
+++ b/internal/storage/search.go
@@ -4,7 +4,9 @@ import (
 	"context"
 	"database/sql"
 	"encoding/json"
+	"fmt"
 	"regexp"
+	"strconv"
 	"strings"
 	"time"
 
@@ -397,6 +399,22 @@ func searchQueryBuilder(searchString, timezone string) *sqlf.Stmt {
 					}
 				}
 			}
+		} else if strings.HasPrefix(lw, "larger:") && sizeToBytes(cleanString(w[7:])) > 0 {
+			w = cleanString(w[7:])
+			size := sizeToBytes(w)
+			if exclude {
+				q.Where("Size < ?", size)
+			} else {
+				q.Where("Size > ?", size)
+			}
+		} else if strings.HasPrefix(lw, "smaller:") && sizeToBytes(cleanString(w[8:])) > 0 {
+			w = cleanString(w[8:])
+			size := sizeToBytes(w)
+			if exclude {
+				q.Where("Size > ?", size)
+			} else {
+				q.Where("Size < ?", size)
+			}
 		} else {
 			// search text
 			if exclude {
@@ -409,3 +427,39 @@ func searchQueryBuilder(searchString, timezone string) *sqlf.Stmt {
 
 	return q
 }
+
+// Simple function to return a size in bytes, eg 2kb, 4MB or 1.5m.
+//
+// K, k, Kb, KB, kB and kb are treated as Kilobytes.
+// M, m, Mb, MB and mb are treated as Megabytes.
+func sizeToBytes(v string) int64 {
+	v = strings.ToLower(v)
+	re := regexp.MustCompile(`^(\d+)(\.\d+)?\s?([a-z]{1,2})?$`)
+
+	m := re.FindAllStringSubmatch(v, -1)
+	if len(m) == 0 {
+		return 0
+	}
+
+	val := fmt.Sprintf("%s%s", m[0][1], m[0][2])
+	unit := m[0][3]
+
+	i, err := strconv.ParseFloat(strings.TrimSpace(val), 64)
+	if err != nil {
+		return 0
+	}
+
+	if unit == "" {
+		return int64(i)
+	}
+
+	if unit == "k" || unit == "kb" {
+		return int64(i * 1024)
+	}
+
+	if unit == "m" || unit == "mb" {
+		return int64(i * 1024 * 1024)
+	}
+
+	return 0
+}
diff --git a/internal/storage/search_test.go b/internal/storage/search_test.go
index 100786ad84..affd1a6faf 100644
--- a/internal/storage/search_test.go
+++ b/internal/storage/search_test.go
@@ -201,3 +201,25 @@ func TestEscPercentChar(t *testing.T) {
 		assertEqual(t, res, expected, "no match")
 	}
 }
+
+func TestSizeToBytes(t *testing.T) {
+	tests := map[string]int64{}
+	tests["1m"] = 1048576
+	tests["1mb"] = 1048576
+	tests["1 M"] = 1048576
+	tests["1 MB"] = 1048576
+	tests["1k"] = 1024
+	tests["1kb"] = 1024
+	tests["1 K"] = 1024
+	tests["1 kB"] = 1024
+	tests["1.5M"] = 1572864
+	tests["1234567890"] = 1234567890
+	tests["invalid"] = 0
+	tests["1.2.3"] = 0
+	tests["1.2.3M"] = 0
+
+	for search, expected := range tests {
+		res := sizeToBytes(search)
+		assertEqual(t, res, expected, "size does not match")
+	}
+}

From 4be92633f2f9c39c9af0a52eb93ed79c6d88ca8f Mon Sep 17 00:00:00 2001
From: Ralph Slooten <axllent@gmail.com>
Date: Fri, 1 Nov 2024 22:19:23 +1300
Subject: [PATCH 4/5] Chore: Update Go dependencies

---
 go.mod | 2 +-
 go.sum | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/go.mod b/go.mod
index 7d616b5749..88565dd199 100644
--- a/go.mod
+++ b/go.mod
@@ -21,7 +21,7 @@ require (
 	github.com/sirupsen/logrus v1.9.3
 	github.com/spf13/cobra v1.8.1
 	github.com/spf13/pflag v1.0.5
-	github.com/tg123/go-htpasswd v1.2.2
+	github.com/tg123/go-htpasswd v1.2.3
 	github.com/vanng822/go-premailer v1.22.0
 	golang.org/x/net v0.30.0
 	golang.org/x/text v0.19.0
diff --git a/go.sum b/go.sum
index 97b3b98ca3..da752b99cb 100644
--- a/go.sum
+++ b/go.sum
@@ -111,8 +111,8 @@ github.com/stretchr/testify v1.8.2/go.mod h1:w2LPCIKwWwSfY2zedu0+kehJoqGctiVI29o
 github.com/stretchr/testify v1.8.4/go.mod h1:sz/lmYIOXD/1dqDmKjjqLyZ2RngseejIcXlSw2iwfAo=
 github.com/stretchr/testify v1.9.0 h1:HtqpIVDClZ4nwg75+f6Lvsy/wHu+3BoSGCbBAcpTsTg=
 github.com/stretchr/testify v1.9.0/go.mod h1:r2ic/lqez/lEtzL7wO/rwa5dbSLXVDPFyf8C91i36aY=
-github.com/tg123/go-htpasswd v1.2.2 h1:tmNccDsQ+wYsoRfiONzIhDm5OkVHQzN3w4FOBAlN6BY=
-github.com/tg123/go-htpasswd v1.2.2/go.mod h1:FcIrK0J+6zptgVwK1JDlqyajW/1B4PtuJ/FLWl7nx8A=
+github.com/tg123/go-htpasswd v1.2.3 h1:ALR6ZBIc2m9u70m+eAWUFt5p43ISbIvAvRFYzZPTOY8=
+github.com/tg123/go-htpasswd v1.2.3/go.mod h1:FcIrK0J+6zptgVwK1JDlqyajW/1B4PtuJ/FLWl7nx8A=
 github.com/unrolled/render v1.7.0/go.mod h1:LwQSeDhjml8NLjIO9GJO1/1qpFJxtfVIpzxXKjfVkoI=
 github.com/valyala/bytebufferpool v1.0.0 h1:GqA5TC/0021Y/b9FG4Oi9Mr3q7XYx6KllzawFIhcdPw=
 github.com/valyala/bytebufferpool v1.0.0/go.mod h1:6bBcMArwyJ5K/AmCkWv1jt77kVWyCJ6HpOuEn7z0Csc=

From 2743e2e0cb6e704d264b11619a474e0cd7f9dde2 Mon Sep 17 00:00:00 2001
From: Ralph Slooten <axllent@gmail.com>
Date: Fri, 1 Nov 2024 22:24:22 +1300
Subject: [PATCH 5/5] Release v1.21.1

---
 CHANGELOG.md | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index beb13e76cf..ccff7d0141 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -2,6 +2,17 @@
 
 Notable changes to Mailpit will be documented in this file.
 
+## [v1.21.1]
+
+### Feature
+- Add ability to search by size smaller or larger than a value (eg: `larger:1M` / `smaller:2.5M`)
+- Add ability to search for messages containing inline images (`has:inline`)
+
+### Chore
+- Update Go dependencies
+- Separate attachments and inline images in download nav and badges ([#379](https://github.com/axllent/mailpit/issues/379))
+
+
 ## [v1.21.0]
 
 ### Feature