fix: revert refactoring error, fixes #3403 (#3407)

haraka · Sep 26, 2024 · ad9c65e · ad9c65e
1 parent 81578d3
commit ad9c65e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/Changes.md b/Changes.md
@@ -3,6 +3,7 @@
 
 #### Changed
 
+- fix: revert a refactoring error in queue/lmtp, #3407
 - removed dependency on ldap plugins #3399
 - doc(tls.md): add note for no_tls_hosts for outbound
 - test(tls): add tests for no_tls_hosts for inbound & outbound

diff --git a/plugins/queue/lmtp.js b/plugins/queue/lmtp.js
@@ -37,7 +37,9 @@ exports.hook_queue = (next, connection) => {
     const txn = connection?.transaction;
     if (!txn) return next();
 
-    if (txn.notes.get('queue.wants') !== 'lmtp') return next();
+    const q_wants = txn.notes.get('queue.wants');
+
+    if (q_wants && q_wants !== 'lmtp') return next();
 
     txn.notes.using_lmtp = true;
     outbound.send_trans_email(txn, next);