default to needFullParse, otherwise do not do full parse

nodemailer · Jan 5, 2024 · 5b65843 · 5b65843
1 parent 44ce8f7
commit 5b65843
Showing 1 changed file with 5 additions and 12 deletions.
diff --git a/lib/imap-notifier.js b/lib/imap-notifier.js
@@ -83,18 +83,11 @@ class ImapNotifier extends EventEmitter {
                 // if p -> json parse ONLY p
                 // if e not in beginning but p is -> json parse whole
 
-                let needFullParse = false;
-
-                if (
-                    (message[2] === 'e' && message[33] === 'p') ||
-                    message[2] === 'p' ||
-                    (message[2] === 'e' && message[5] !== `"` && message[6 + 24 + 1] !== `"`)
-                ) {
-                    // 1) there is e and p OR
-                    // 2) p is first OR
-                    // 2) there is e but it is not between quotes (not 24 chars long)
-                    // do full parse
-                    needFullParse = true;
+                let needFullParse = true;
+
+                if (message.length === 32 && message[2] === 'e' && message[5] === '"' && message[6 + 24] === '"') {
+                    // there is only e, no p -> no need for full parse
+                    needFullParse = false;
                 }
 
                 if (!needFullParse) {