imapflow 1.0.83 → 1.0.86

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # CHANGELOG
 
+-   1.0.86 - 2022-03-14
+
+    -   Do not try to decode mailbox path from utf7 as methods expect utf-8 not utf-7
+
+-   1.0.85 - 2022-02-23
+
+    -   Trim null bytes in the beginning of IMAP responses to deal with a buggy server that pads responses with null bytes
+
+-   1.0.84 - 2022-02-17
+
+    -   QRESYNC tweaks
+
 -   1.0.83 - 2022-02-16
 
     -   QRESYNC tweaks

package/LICENSE.txt

@@ -1,4 +1,4 @@
-Copyright (c) 2020-2021 Postal Systems OÜ
+Copyright (c) 2020-2022 Postal Systems OÜ
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

package/README.md

@@ -80,6 +80,6 @@ main().catch(err => console.error(err));
 
 ## License
 
-© 2020-2021 Postal Systems OÜ
+© 2020-2022 Postal Systems OÜ
 
 Licensed under **MIT-license**

package/lib/commands/select.js

@@ -40,8 +40,8 @@ module.exports = async (connection, path, options) => {
             extraArgs.push([
                 { type: 'ATOM', value: 'QRESYNC' },
                 [
-                    { type: 'ATOM', value: options.uidValidity },
-                    { type: 'ATOM', value: options.changedSince }
+                    { type: 'ATOM', value: options.uidValidity.toString() },
+                    { type: 'ATOM', value: options.changedSince.toString() }
                 ]
             ]);
             map.qresync = true;
@@ -156,6 +156,19 @@ module.exports = async (connection, path, options) => {
             }
         }
 
+        if (
+            map.qresync &&
+            // UIDVALIDITY must be the same
+            (options.uidValidity !== map.uidValidity ||
+                // HIGHESTMODSEQ response must be present
+                !map.highestModseq ||
+                // NOMODSEQ is not allowed
+                map.noModseq)
+        ) {
+            // QRESYNC does not apply here, so unset it
+            map.qresync = false;
+        }
+
         let currentMailbox = connection.mailbox;
         connection.mailbox = false;
 

package/lib/handler/imap-parser.js

@@ -6,6 +6,21 @@ const { ParserInstance } = require('./parser-instance');
 module.exports = async (command, options) => {
     options = options || {};
 
+    let nullBytesRemoved = 0;
+
+    // special case with a buggy IMAP server where responses are padded with zero bytes
+    if (command[0] === 0) {
+        // find the first non null byte and trim
+        for (let i = 0; i < command.length; i++) {
+            if (command[i] !== 0) {
+                // trim to here
+                command = command.slice(i);
+                nullBytesRemoved = i;
+                break;
+            }
+        }
+    }
+
     const parser = new ParserInstance(command, options);
     const response = {};
 
@@ -13,6 +28,10 @@ module.exports = async (command, options) => {
     await parser.getSpace();
     response.command = await parser.getCommand();
 
+    if (nullBytesRemoved) {
+        response.nullBytesRemoved = nullBytesRemoved;
+    }
+
     if (['UID', 'AUTHENTICATE'].indexOf((response.command || '').toUpperCase()) >= 0) {
         await parser.getSpace();
         response.command += ' ' + (await parser.getElement(imapFormalSyntax.command()));

package/lib/imap-flow.js

@@ -424,9 +424,9 @@ class ImapFlow extends EventEmitter {
 
             if (/^\d+$/.test(parsed.command) && parsed.attributes && parsed.attributes[0] && parsed.attributes[0].value === 'FETCH') {
                 // too many FETCH responses, might want to filter these out
-                this.log.trace({ src: 's', msg: logCompiled.toString(), cid: this.id });
+                this.log.trace({ src: 's', msg: logCompiled.toString(), cid: this.id, nullBytesRemoved: parsed.nullBytesRemoved });
             } else {
-                this.log.debug({ src: 's', msg: logCompiled.toString(), cid: this.id });
+                this.log.debug({ src: 's', msg: logCompiled.toString(), cid: this.id, nullBytesRemoved: parsed.nullBytesRemoved });
             }
 
             if (parsed.tag === '+' && this.currentRequest && this.currentRequest.options && typeof this.currentRequest.options.onPlusTag === 'function') {

package/lib/tools.js

@@ -23,12 +23,6 @@ module.exports = {
             } catch (err) {
                 // ignore, keep name as is
             }
-        } else if (connection.enabled.has('UTF8=ACCEPT') && /&/.test(path)) {
-            try {
-                path = iconv.decode(Buffer.from(path), 'utf-7-imap').toString();
-            } catch (err) {
-                // ignore, keep name as is
-            }
         }
 
         // ensure namespace prefix if needed

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "imapflow",
-    "version": "1.0.83",
+    "version": "1.0.86",
     "description": "IMAP Client for Node",
     "main": "./lib/imap-flow.js",
     "scripts": {
@@ -27,14 +27,14 @@
     "homepage": "https://imapflow.com/",
     "devDependencies": {
         "@babel/eslint-parser": "7.17.0",
-        "@babel/eslint-plugin": "7.16.5",
+        "@babel/eslint-plugin": "7.17.7",
         "@babel/plugin-syntax-class-properties": "7.12.13",
         "@babel/preset-env": "7.16.11",
-        "@types/node": "17.0.18",
+        "@types/node": "17.0.21",
         "braintree-jsdoc-template": "3.3.0",
-        "eslint": "8.9.0",
+        "eslint": "8.11.0",
         "eslint-config-nodemailer": "1.2.0",
-        "eslint-config-prettier": "8.3.0",
+        "eslint-config-prettier": "8.5.0",
         "grunt": "1.4.1",
         "grunt-cli": "1.4.3",
         "grunt-contrib-nodeunit": "4.0.0",
@@ -50,7 +50,7 @@
         "libqp": "1.1.0",
         "mailsplit": "5.3.1",
         "nodemailer": "6.7.2",
-        "pino": "7.8.0",
+        "pino": "7.8.1",
         "socks": "2.6.2"
     }
 }