From 9d2e761494ef9783f1ad29c9a79b0007fe8dbea9 Mon Sep 17 00:00:00 2001
From: Mechiel Lukkien <mechiel@ueber.net>
Date: Wed, 22 Nov 2023 22:01:23 +0100
Subject: [PATCH] turns out the esearch tag is a string before imap4rev2, so
 can't blame new outlook

---
 imapserver/search.go | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/imapserver/search.go b/imapserver/search.go
index 0591ae2..63448c6 100644
--- a/imapserver/search.go
+++ b/imapserver/search.go
@@ -221,8 +221,8 @@ func (c *conn) cmdxSearch(isUID bool, tag, cmd string, p *parser) {
 
 		// No untagged ESEARCH response if nothing was requested. ../rfc/9051:4160
 		if len(eargs) > 0 {
-			// Microsoft Outlook "new" (Microsoft Office 365/15.20.7025.17) seems to fail when
-			// the tag value doesn't have double quotes. These quotes are optional.
+			// The tag was originally a string, became an astring in IMAP4rev2, better stick to
+			// string. ../rfc/4466:707 ../rfc/5259:1163 ../rfc/9051:7087
 			resp := fmt.Sprintf(`* ESEARCH (TAG "%s")`, tag)
 			if isUID {
 				resp += " UID"