postal-mime 2.4.3 → 2.4.5

package/.prettierignore

@@ -0,0 +1,9 @@
+node_modules
+dist
+.git
+*.min.js
+coverage
+.DS_Store
+yarn.lock
+package-lock.json
+CLAUDE.md

package/.prettierrc.json

@@ -0,0 +1,10 @@
+{
+    "semi": true,
+    "trailingComma": "none",
+    "singleQuote": true,
+    "printWidth": 120,
+    "tabWidth": 4,
+    "useTabs": false,
+    "bracketSpacing": true,
+    "arrowParens": "avoid"
+}

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 # Changelog
 
+## [2.4.5](https://github.com/postalsys/postal-mime/compare/v2.4.4...v2.4.5) (2025-09-29)
+
+
+### Bug Fixes
+
+* handle broken quoted-printable sequences and add prettier formatter ([ec3bc64](https://github.com/postalsys/postal-mime/commit/ec3bc647f6e262a7ac3a51e30bbe3b07b6e7db7a))
+
+## [2.4.4](https://github.com/postalsys/postal-mime/compare/v2.4.3...v2.4.4) (2025-06-26)
+
+
+### Bug Fixes
+
+* **TextDecoder:** Fall back to windows-1252 for an unknown charset instead of throwing ([d5b917d](https://github.com/postalsys/postal-mime/commit/d5b917d5b09fab9183733cd76ebdc896467ae31e))
+
 ## [2.4.3](https://github.com/postalsys/postal-mime/compare/v2.4.2...v2.4.3) (2025-01-24)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "postal-mime",
-    "version": "2.4.3",
+    "version": "2.4.5",
     "description": "Email parser for browser environments",
     "main": "./src/postal-mime.js",
     "exports": {
@@ -12,7 +12,9 @@
     "types": "postal-mime.d.ts",
     "scripts": {
         "test": "eslint && node --test",
-        "update": "rm -rf node_modules package-lock.json && ncu -u && npm install"
+        "update": "rm -rf node_modules package-lock.json && ncu -u && npm install",
+        "format": "prettier --write \"**/*.{js,mjs,json}\" --ignore-path .prettierignore",
+        "format:check": "prettier --check \"**/*.{js,mjs,json}\" --ignore-path .prettierignore"
     },
     "keywords": [
         "mime",
@@ -28,11 +30,12 @@
     "author": "Andris Reinman",
     "license": "MIT-0",
     "devDependencies": {
-        "@types/node": "22.10.10",
+        "@types/node": "24.0.4",
         "cross-blob": "3.0.2",
         "cross-env": "7.0.3",
         "eslint": "8.57.0",
         "eslint-cli": "1.1.1",
-        "iframe-resizer": "4.3.6"
+        "iframe-resizer": "4.3.6",
+        "prettier": "^3.6.2"
     }
 }

package/src/decode-strings.js

@@ -44,7 +44,15 @@ export function decodeBase64(base64) {
 
 export function getDecoder(charset) {
     charset = charset || 'utf8';
-    return new TextDecoder(charset);
+    let decoder;
+
+    try {
+        decoder = new TextDecoder(charset);
+    } catch (err) {
+        decoder = new TextDecoder('windows-1252');
+    }
+
+    return decoder;
 }
 
 /**
@@ -73,7 +81,11 @@ export async function blobToArrayBuffer(blob) {
 }
 
 export function getHex(c) {
-    if ((c >= 0x30 /* 0 */ && c <= 0x39) /* 9 */ || (c >= 0x61 /* a */ && c <= 0x66) /* f */ || (c >= 0x41 /* A */ && c <= 0x46) /* F */) {
+    if (
+        (c >= 0x30 /* 0 */ && c <= 0x39) /* 9 */ ||
+        (c >= 0x61 /* a */ && c <= 0x66) /* f */ ||
+        (c >= 0x41 /* A */ && c <= 0x46) /* F */
+    ) {
         return String.fromCharCode(c);
     }
     return false;
@@ -144,36 +156,44 @@ export function decodeWords(str) {
         let result = (str || '')
             .toString()
             // find base64 words that can be joined
-            .replace(/(=\?([^?]+)\?[Bb]\?([^?]*)\?=)\s*(?==\?([^?]+)\?[Bb]\?[^?]*\?=)/g, (match, left, chLeft, encodedLeftStr, chRight) => {
-                if (!joinString) {
+            .replace(
+                /(=\?([^?]+)\?[Bb]\?([^?]*)\?=)\s*(?==\?([^?]+)\?[Bb]\?[^?]*\?=)/g,
+                (match, left, chLeft, encodedLeftStr, chRight) => {
+                    if (!joinString) {
+                        return match;
+                    }
+                    // only mark b64 chunks to be joined if charsets match and left side does not end with =
+                    if (chLeft === chRight && encodedLeftStr.length % 4 === 0 && !/=$/.test(encodedLeftStr)) {
+                        // set a joiner marker
+                        return left + '__\x00JOIN\x00__';
+                    }
+
                     return match;
                 }
-                // only mark b64 chunks to be joined if charsets match and left side does not end with =
-                if (chLeft === chRight && encodedLeftStr.length % 4 === 0 && !/=$/.test(encodedLeftStr)) {
-                    // set a joiner marker
-                    return left + '__\x00JOIN\x00__';
-                }
-
-                return match;
-            })
+            )
             // find QP words that can be joined
-            .replace(/(=\?([^?]+)\?[Qq]\?[^?]*\?=)\s*(?==\?([^?]+)\?[Qq]\?[^?]*\?=)/g, (match, left, chLeft, chRight) => {
-                if (!joinString) {
+            .replace(
+                /(=\?([^?]+)\?[Qq]\?[^?]*\?=)\s*(?==\?([^?]+)\?[Qq]\?[^?]*\?=)/g,
+                (match, left, chLeft, chRight) => {
+                    if (!joinString) {
+                        return match;
+                    }
+                    // only mark QP chunks to be joined if charsets match
+                    if (chLeft === chRight) {
+                        // set a joiner marker
+                        return left + '__\x00JOIN\x00__';
+                    }
                     return match;
                 }
-                // only mark QP chunks to be joined if charsets match
-                if (chLeft === chRight) {
-                    // set a joiner marker
-                    return left + '__\x00JOIN\x00__';
-                }
-                return match;
-            })
+            )
             // join base64 encoded words
             .replace(/(\?=)?__\x00JOIN\x00__(=\?([^?]+)\?[QqBb]\?)?/g, '')
             // remove spaces between mime encoded words
             .replace(/(=\?[^?]+\?[QqBb]\?[^?]*\?=)\s+(?==\?[^?]+\?[QqBb]\?[^?]*\?=)/g, '$1')
             // decode words
-            .replace(/=\?([\w_\-*]+)\?([QqBb])\?([^?]*)\?=/g, (m, charset, encoding, text) => decodeWord(charset, encoding, text));
+            .replace(/=\?([\w_\-*]+)\?([QqBb])\?([^?]*)\?=/g, (m, charset, encoding, text) =>
+                decodeWord(charset, encoding, text)
+            );
 
         if (joinString && result.indexOf('\ufffd') >= 0) {
             // text contains \ufffd (EF BF BD), so unicode conversion failed, retry without joining strings

package/src/postal-mime.js

@@ -190,7 +190,8 @@ export default class PostalMime {
 
                 // is it an attachment
                 else if (node.content) {
-                    const filename = node.contentDisposition.parsed.params.filename || node.contentType.parsed.params.name || null;
+                    const filename =
+                        node.contentDisposition.parsed.params.filename || node.contentType.parsed.params.name || null;
                     const attachment = {
                         filename: filename ? decodeWords(filename) : null,
                         mimeType: node.contentType.parsed.value,
@@ -214,7 +215,10 @@ export default class PostalMime {
                         case 'text/calendar':
                         case 'application/ics': {
                             if (node.contentType.parsed.params.method) {
-                                attachment.method = node.contentType.parsed.params.method.toString().toUpperCase().trim();
+                                attachment.method = node.contentType.parsed.params.method
+                                    .toString()
+                                    .toUpperCase()
+                                    .trim();
                             }
 
                             // Enforce into unicode
@@ -341,7 +345,8 @@ export default class PostalMime {
         if (node.contentType.parsed.value !== 'message/rfc822') {
             return false;
         }
-        let disposition = node.contentDisposition.parsed.value || (this.options.rfc822Attachments ? 'attachment' : 'inline');
+        let disposition =
+            node.contentDisposition.parsed.value || (this.options.rfc822Attachments ? 'attachment' : 'inline');
         return disposition === 'inline';
     }
 

package/src/qp-decoder.js

@@ -1,5 +1,11 @@
 import { blobToArrayBuffer } from './decode-strings.js';
 
+// Regex patterns compiled once for performance
+const VALID_QP_REGEX = /^=[a-f0-9]{2}$/i;
+const QP_SPLIT_REGEX = /(?==[a-f0-9]{2})/i;
+const SOFT_LINE_BREAK_REGEX = /=\r?\n/g;
+const PARTIAL_QP_ENDING_REGEX = /=[a-fA-F0-9]?$/;
+
 export default class QPDecoder {
     constructor(opts) {
         opts = opts || {};
@@ -24,9 +30,9 @@ export default class QPDecoder {
 
     decodeChunks(str) {
         // unwrap newlines
-        str = str.replace(/=\r?\n/g, '');
+        str = str.replace(SOFT_LINE_BREAK_REGEX, '');
 
-        let list = str.split(/(?==)/);
+        let list = str.split(QP_SPLIT_REGEX);
         let encodedBytes = [];
         for (let part of list) {
             if (part.charAt(0) !== '=') {
@@ -39,17 +45,38 @@ export default class QPDecoder {
             }
 
             if (part.length === 3) {
-                encodedBytes.push(part.substr(1));
+                // Validate that this is actually a valid QP sequence
+                if (VALID_QP_REGEX.test(part)) {
+                    encodedBytes.push(part.substr(1));
+                } else {
+                    // Not a valid QP sequence, treat as literal text
+                    if (encodedBytes.length) {
+                        this.chunks.push(this.decodeQPBytes(encodedBytes));
+                        encodedBytes = [];
+                    }
+                    this.chunks.push(part);
+                }
                 continue;
             }
 
             if (part.length > 3) {
-                encodedBytes.push(part.substr(1, 2));
-                this.chunks.push(this.decodeQPBytes(encodedBytes));
-                encodedBytes = [];
+                // First 3 chars should be a valid QP sequence
+                const firstThree = part.substr(0, 3);
+                if (VALID_QP_REGEX.test(firstThree)) {
+                    encodedBytes.push(part.substr(1, 2));
+                    this.chunks.push(this.decodeQPBytes(encodedBytes));
+                    encodedBytes = [];
 
-                part = part.substr(3);
-                this.chunks.push(part);
+                    part = part.substr(3);
+                    this.chunks.push(part);
+                } else {
+                    // Not a valid QP sequence, treat entire part as literal
+                    if (encodedBytes.length) {
+                        this.chunks.push(this.decodeQPBytes(encodedBytes));
+                        encodedBytes = [];
+                    }
+                    this.chunks.push(part);
+                }
             }
         }
         if (encodedBytes.length) {
@@ -71,7 +98,7 @@ export default class QPDecoder {
 
         this.remainder = '';
 
-        let partialEnding = str.match(/=[a-fA-F0-9]?$/);
+        let partialEnding = str.match(PARTIAL_QP_ENDING_REGEX);
         if (partialEnding) {
             if (partialEnding.index === 0) {
                 this.remainder = str;

package/src/text-format.js

@@ -177,7 +177,10 @@ function foldLines(str, lineLength, afterSpace) {
             result += line;
             pos += line.length;
             continue;
-        } else if ((match = line.match(/(\s+)[^\s]*$/)) && match[0].length - (afterSpace ? (match[1] || '').length : 0) < line.length) {
+        } else if (
+            (match = line.match(/(\s+)[^\s]*$/)) &&
+            match[0].length - (afterSpace ? (match[1] || '').length : 0) < line.length
+        ) {
             line = line.substr(0, line.length - (match[0].length - (afterSpace ? (match[1] || '').length : 0)));
         } else if ((match = str.substr(pos + line.length).match(/^[^\s]+(\s*)/))) {
             line = line + match[0].substr(0, match[0].length - (!afterSpace ? (match[1] || '').length : 0));
@@ -215,7 +218,10 @@ export function formatTextHeader(message) {
             hour12: false
         };
 
-        let dateStr = typeof Intl === 'undefined' ? message.date : new Intl.DateTimeFormat('default', dateOptions).format(new Date(message.date));
+        let dateStr =
+            typeof Intl === 'undefined'
+                ? message.date
+                : new Intl.DateTimeFormat('default', dateOptions).format(new Date(message.date));
 
         rows.push({ key: 'Date', val: dateStr });
     }
@@ -283,7 +289,9 @@ export function formatHtmlHeader(message) {
     let rows = [];
 
     if (message.from) {
-        rows.push(`<div class="postal-email-header-key">From</div><div class="postal-email-header-value">${formatHtmlAddress(message.from)}</div>`);
+        rows.push(
+            `<div class="postal-email-header-key">From</div><div class="postal-email-header-value">${formatHtmlAddress(message.from)}</div>`
+        );
     }
 
     if (message.subject) {
@@ -305,7 +313,10 @@ export function formatHtmlHeader(message) {
             hour12: false
         };
 
-        let dateStr = typeof Intl === 'undefined' ? message.date : new Intl.DateTimeFormat('default', dateOptions).format(new Date(message.date));
+        let dateStr =
+            typeof Intl === 'undefined'
+                ? message.date
+                : new Intl.DateTimeFormat('default', dateOptions).format(new Date(message.date));
 
         rows.push(
             `<div class="postal-email-header-key">Date</div><div class="postal-email-header-value postal-email-header-date" data-date="${escapeHtml(
@@ -315,15 +326,21 @@ export function formatHtmlHeader(message) {
     }
 
     if (message.to && message.to.length) {
-        rows.push(`<div class="postal-email-header-key">To</div><div class="postal-email-header-value">${formatHtmlAddresses(message.to)}</div>`);
+        rows.push(
+            `<div class="postal-email-header-key">To</div><div class="postal-email-header-value">${formatHtmlAddresses(message.to)}</div>`
+        );
     }
 
     if (message.cc && message.cc.length) {
-        rows.push(`<div class="postal-email-header-key">Cc</div><div class="postal-email-header-value">${formatHtmlAddresses(message.cc)}</div>`);
+        rows.push(
+            `<div class="postal-email-header-key">Cc</div><div class="postal-email-header-value">${formatHtmlAddresses(message.cc)}</div>`
+        );
     }
 
     if (message.bcc && message.bcc.length) {
-        rows.push(`<div class="postal-email-header-key">Bcc</div><div class="postal-email-header-value">${formatHtmlAddresses(message.bcc)}</div>`);
+        rows.push(
+            `<div class="postal-email-header-key">Bcc</div><div class="postal-email-header-value">${formatHtmlAddresses(message.bcc)}</div>`
+        );
     }
 
     let template = `<div class="postal-email-header">${rows.length ? '<div class="postal-email-header-row">' : ''}${rows.join(