imapflow 1.0.153 → 1.0.154

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## [1.0.154](https://github.com/postalsys/imapflow/compare/v1.0.153...v1.0.154) (2024-03-05)
+
+
+### Bug Fixes
+
+* **FETCH:** throw on failing FETCH ([12f9a45](https://github.com/postalsys/imapflow/commit/12f9a45530c6f3a2f35ba2968819011eea9c32b2))
+
 ## [1.0.153](https://github.com/postalsys/imapflow/compare/v1.0.152...v1.0.153) (2024-02-29)
 
 

package/lib/commands/fetch.js

@@ -203,7 +203,7 @@ module.exports = async (connection, range, query, options) => {
             }
 
             connection.log.warn({ err, cid: connection.id });
-            return false;
+            throw err;
         }
     }
 };

package/lib/imap-flow.js

@@ -2254,93 +2254,94 @@ class ImapFlow extends EventEmitter {
      */
     async *fetch(range, query, options) {
         options = options || {};
-        try {
-            if (!this.mailbox) {
-                // no mailbox selected, nothing to do
-                return;
-            }
 
-            range = await this.resolveRange(range, options);
-            if (!range) {
-                return false;
-            }
+        if (!this.mailbox) {
+            // no mailbox selected, nothing to do
+            return;
+        }
 
-            let finished = false;
-            let push = false;
-            let rowQueue = [];
-            let getNext = () =>
-                new Promise((resolve, reject) => {
-                    let check = () => {
-                        if (rowQueue.length) {
-                            let entry = rowQueue.shift();
-                            if (entry.err) {
-                                return reject(entry.err);
-                            } else {
-                                return resolve(entry.value);
-                            }
-                        }
-                        if (finished) {
-                            return resolve(null);
-                        }
+        range = await this.resolveRange(range, options);
+        if (!range) {
+            return false;
+        }
 
-                        // wait until data is pushed to queue and try again
-                        push = () => {
-                            push = false;
-                            check();
-                        };
-                    };
-                    check();
-                });
+        let finished = false;
+        let push = false;
+        let rowQueue = [];
 
-            this.run('FETCH', range, query, {
-                uid: !!options.uid,
-                binary: options.binary,
-                changedSince: options.changedSince,
-                onUntaggedFetch: (untagged, next) => {
-                    rowQueue.push({
-                        value: {
-                            response: untagged,
-                            next
+        let getNext = () =>
+            new Promise((resolve, reject) => {
+                let check = () => {
+                    if (rowQueue.length) {
+                        let entry = rowQueue.shift();
+                        if (entry.err) {
+                            return reject(entry.err);
+                        } else {
+                            return resolve(entry.value);
                         }
-                    });
-                    if (typeof push === 'function') {
-                        push();
                     }
-                }
-            })
-                .then(() => {
-                    finished = true;
-                    if (typeof push === 'function') {
-                        push();
+
+                    if (finished) {
+                        return resolve(null);
                     }
-                })
-                .catch(err => {
-                    rowQueue.push({ err });
-                });
 
-            let res;
-            while ((res = await getNext())) {
-                if (this.isClosed || this.socket.destroyed) {
-                    let error = new Error('Connection closed');
-                    error.code = 'EConnectionClosed';
-                    throw error;
-                }
+                    // wait until data is pushed to queue and try again
+                    push = () => {
+                        push = false;
+                        check();
+                    };
+                };
+                check();
+            });
 
-                if (res !== null) {
-                    yield res.response;
-                    res.next();
+        this.run('FETCH', range, query, {
+            uid: !!options.uid,
+            binary: options.binary,
+            changedSince: options.changedSince,
+            onUntaggedFetch: (untagged, next) => {
+                rowQueue.push({
+                    value: {
+                        response: untagged,
+                        next
+                    }
+                });
+                if (typeof push === 'function') {
+                    push();
                 }
             }
+        })
+            .then(() => {
+                finished = true;
+                if (typeof push === 'function') {
+                    push();
+                }
+            })
+            .catch(err => {
+                rowQueue.push({ err });
+                if (typeof push === 'function') {
+                    push();
+                }
+            });
 
-            if (!finished) {
-                // FETCH never finished!
-                let error = new Error('FETCH did not finish');
-                error.code = 'ENotFinished';
+        let res;
+        while ((res = await getNext())) {
+            if (this.isClosed || this.socket.destroyed) {
+                let error = new Error('Connection closed');
+                error.code = 'EConnectionClosed';
                 throw error;
             }
-        } catch (err) {
-            setImmediate(() => this.close());
-            throw err;
+
+            if (res !== null) {
+                yield res.response;
+                res.next();
+            }
+        }
+
+        if (!finished) {
+            // FETCH never finished!
+            let error = new Error('FETCH did not finish');
+            error.code = 'ENotFinished';
+            throw error;
         }
     }
 

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "imapflow",
-    "version": "1.0.153",
+    "version": "1.0.154",
     "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.22",
+        "@types/node": "20.11.24",
         "eslint": "8.57.0",
         "eslint-config-nodemailer": "1.2.0",
         "eslint-config-prettier": "9.1.0",