imapflow 1.0.134 → 1.0.135

package/CHANGELOG.md

@@ -1,5 +1,9 @@
 # CHANGELOG
 
+-   1.0.135 - 2023-07-29
+
+    -   Improved handling of unexpected close
+
 -   1.0.134 - 2023-07-20
 
     -   Fixed unicode search if UTF8=ACCEPT extension was enabled

package/lib/imap-flow.js

@@ -923,6 +923,7 @@ class ImapFlow extends EventEmitter {
                 this.log.error({ err, cid: this.id });
 
                 if (typeof this.initialReject === 'function') {
+                    clearTimeout(this.greetingTimeout);
                     let reject = this.initialReject;
                     this.initialResolve = false;
                     this.initialReject = false;
@@ -958,6 +959,7 @@ class ImapFlow extends EventEmitter {
                 this.log.error({ err, cid: this.id });
 
                 if (typeof this.initialReject === 'function') {
+                    clearTimeout(this.greetingTimeout);
                     let reject = this.initialReject;
                     this.initialResolve = false;
                     this.initialReject = false;
@@ -1370,12 +1372,14 @@ class ImapFlow extends EventEmitter {
         this.idling = false;
 
         if (typeof this.initialReject === 'function' && !this.options.verifyOnly) {
+            clearTimeout(this.greetingTimeout);
             let reject = this.initialReject;
             this.initialResolve = false;
             this.initialReject = false;
             let err = new Error('Unexpected close');
             err.code = `ClosedAfterConnect${this.secureConnection ? 'TLS' : 'Text'}`;
-            reject(err);
+            // still has to go through the logic below
+            setImmediate(() => reject(err));
         }
 
         if (typeof this.preCheck === 'function') {
@@ -1412,7 +1416,7 @@ class ImapFlow extends EventEmitter {
 
         if (this.writeSocket && !this.writeSocket.destroyed) {
             try {
-                this.writeSocket.end();
+                this.writeSocket.destroySoon();
             } catch (err) {
                 this.log.error({ err, cid: this.id });
             }
@@ -1420,7 +1424,7 @@ class ImapFlow extends EventEmitter {
 
         if (this.socket && !this.socket.destroyed && this.writeSocket !== this.socket) {
             try {
-                this.socket.end();
+                this.socket.destroySoon();
             } catch (err) {
                 this.log.error({ err, cid: this.id });
             }

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "imapflow",
-    "version": "1.0.134",
+    "version": "1.0.135",
     "description": "IMAP Client for Node",
     "main": "./lib/imap-flow.js",
     "scripts": {
@@ -31,11 +31,11 @@
         "@babel/eslint-plugin": "7.22.5",
         "@babel/plugin-syntax-class-properties": "7.12.13",
         "@babel/preset-env": "7.22.9",
-        "@types/node": "20.4.2",
+        "@types/node": "20.4.5",
         "braintree-jsdoc-template": "3.3.0",
-        "eslint": "8.45.0",
+        "eslint": "8.46.0",
         "eslint-config-nodemailer": "1.2.0",
-        "eslint-config-prettier": "8.8.0",
+        "eslint-config-prettier": "8.9.0",
         "grunt": "1.6.1",
         "grunt-cli": "1.4.3",
         "grunt-contrib-nodeunit": "5.0.0",
@@ -52,7 +52,7 @@
         "libqp": "2.0.1",
         "mailsplit": "5.4.0",
         "nodemailer": "6.9.4",
-        "pino": "8.14.1",
+        "pino": "8.14.2",
         "socks": "2.7.1"
     }
 }