imapflow 1.0.156 → 1.0.157

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## [1.0.157](https://github.com/postalsys/imapflow/compare/v1.0.156...v1.0.157) (2024-03-19)
+
+
+### Bug Fixes
+
+* **logs:** Log mailbox lock description ([ce87cf5](https://github.com/postalsys/imapflow/commit/ce87cf5a256ddc324624893a1c17992475fafeb3))
+
 ## [1.0.156](https://github.com/postalsys/imapflow/compare/v1.0.155...v1.0.156) (2024-03-07)
 
 

package/lib/imap-flow.js

@@ -1313,7 +1313,7 @@ class ImapFlow extends EventEmitter {
 
         await new Promise((resolve, reject) => {
             this.connectTimeout = setTimeout(() => {
-                let err = new Error('Failed to established connection in required time');
+                let err = new Error('Failed to establish connection in required time');
                 err.code = 'CONNECT_TIMEOUT';
                 err.details = {
                     connectionTimeout: this.options.connectionTimeout || CONNECT_TIMEOUT
@@ -2840,17 +2840,35 @@ class ImapFlow extends EventEmitter {
 
     async processLocks(force) {
         if (!force && this.processingLock) {
+            this.log.trace({
+                msg: 'Mailbox locking queued',
+                path: this.mailbox && this.mailbox.path,
+                pending: this.locks.length,
+                idling: this.idling
+            });
             return;
         }
+
         if (!this.locks.length) {
             this.processingLock = false;
+            this.log.trace({
+                msg: 'Mailbox locking queue processed',
+                idling: this.idling
+            });
             return;
         }
+
         this.processingLock = true;
 
         const release = () => {
             if (this.currentLockId) {
-                this.log.trace({ msg: 'Mailbox lock released', lockId: this.currentLockId, path: this.mailbox && this.mailbox.path });
+                this.log.trace({
+                    msg: 'Mailbox lock released',
+                    lockId: this.currentLockId,
+                    path: this.mailbox && this.mailbox.path,
+                    pending: this.locks.length,
+                    idling: this.idling
+                });
                 this.currentLockId = 0;
             }
             this.processLocks(true).catch(err => this.log.error({ err, cid: this.id }));
@@ -2859,7 +2877,7 @@ class ImapFlow extends EventEmitter {
         const { resolve, reject, path, options, lockId } = this.locks.shift();
 
         if (!this.usable || this.socket.destroyed) {
-            this.log.trace({ msg: 'Failed to acquire mailbox lock', path, lockId });
+            this.log.trace({ msg: 'Failed to acquire mailbox lock', path, lockId, idling: this.idling });
             // reject all
             let error = new Error('Connection not available');
             error.code = 'NoConnection';
@@ -2869,14 +2887,14 @@ class ImapFlow extends EventEmitter {
 
         if (this.mailbox && this.mailbox.path === path && !!this.mailbox.readOnly === !!options.readOnly) {
             // nothing to do here, already selected
-            this.log.trace({ msg: 'Mailbox lock acquired', path, lockId });
+            this.log.trace({ msg: 'Mailbox lock acquired', path, lockId, idling: this.idling });
             this.currentLockId = lockId;
             return resolve({ path, release });
         } else {
             try {
                 // Try to open. Throws if mailbox does not exists or can't open
                 await this.mailboxOpen(path, options);
-                this.log.trace({ msg: 'Lock acquired', path, lockId });
+                this.log.trace({ msg: 'Lock acquired', path, lockId, idling: this.idling, ...(options.description && { description: options.description }) });
                 this.currentLockId = lockId;
                 return resolve({ path, release });
             } catch (err) {
@@ -2891,7 +2909,7 @@ class ImapFlow extends EventEmitter {
                     }
                 }
 
-                this.log.trace({ msg: 'Failed to acquire mailbox lock', path, lockId });
+                this.log.trace({ msg: 'Failed to acquire mailbox lock', path, lockId, idling: this.idling });
                 reject(err);
                 await this.processLocks(true);
             }
@@ -2925,7 +2943,7 @@ class ImapFlow extends EventEmitter {
         path = normalizePath(this, path);
 
         let lockId = ++this.lockCounter;
-        this.log.trace({ msg: 'Requesting lock', path, lockId });
+        this.log.trace({ msg: 'Requesting lock', path, lockId, ...(options.description && { description: options.description }) });
 
         return await new Promise((resolve, reject) => {
             this.locks.push({ resolve, reject, path, options, lockId });

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "imapflow",
-    "version": "1.0.156",
+    "version": "1.0.157",
     "description": "IMAP Client for Node",
     "main": "./lib/imap-flow.js",
     "scripts": {
@@ -32,7 +32,7 @@
         "@babel/eslint-plugin": "7.23.5",
         "@babel/plugin-syntax-class-properties": "7.12.13",
         "@babel/preset-env": "7.24.0",
-        "@types/node": "20.11.24",
+        "@types/node": "20.11.29",
         "eslint": "8.57.0",
         "eslint-config-nodemailer": "1.2.0",
         "eslint-config-prettier": "9.1.0",
@@ -52,7 +52,7 @@
         "libmime": "5.3.4",
         "libqp": "2.1.0",
         "mailsplit": "5.4.0",
-        "nodemailer": "6.9.11",
+        "nodemailer": "6.9.12",
         "pino": "8.19.0",
         "socks": "2.8.1"
     }