@bobfrankston/rmfmail 1.0.647 → 1.0.649

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/rmfmail",
-  "version": "1.0.647",
+  "version": "1.0.649",
   "description": "Local-first email client with IMAP sync and standalone native app",
   "type": "module",
   "main": "bin/mailx.js",

package/packages/mailx-service/index.ts

@@ -443,18 +443,32 @@ export class MailxService {
             "Content-Type": "application/x-www-form-urlencoded",
             "User-Agent": "mailx/1.0 (https://github.com/BobFrankston/mailx)",
         };
+        // `redirect: "manual"` so we don't auto-issue a GET to whatever
+        // confirmation page the sender redirects to. Per RFC 8058 §4 the
+        // POST status is what indicates success; many senders redirect to
+        // a "you're unsubscribed" page that we never need to render. The
+        // pre-fix `redirect: "follow"` caused the user to see
+        // `Cannot GET /u/…` — that was the redirected GET hitting an
+        // unhandled route on the sender's confirmation host, NOT rmfmail
+        // using GET. Treat 2xx and 3xx as success.
         const resp = await fetch(url, {
             method: "POST",
             headers,
             body: "List-Unsubscribe=One-Click",
-            redirect: "follow",
+            redirect: "manual",
         });
-        const body = resp.ok ? "" : await resp.text().catch(() => "");
+        // status===0 with type==="opaqueredirect" is what `manual` returns
+        // for 30x; reflect that as success since the POST was accepted.
+        const isRedirect = resp.type === "opaqueredirect" || (resp.status >= 300 && resp.status < 400);
+        const ok = resp.ok || isRedirect;
+        const body = ok ? "" : await resp.text().catch(() => "");
+        const displayStatus = isRedirect ? 200 : resp.status;
+        const displayText = isRedirect ? "OK (redirect)" : resp.statusText;
         console.log(`  [unsub] POST ${url} → ${resp.status} ${resp.statusText}${body ? `; body: ${body.slice(0, 200)}` : ""}`);
-        const statusText = resp.ok
-            ? resp.statusText
+        const statusText = ok
+            ? displayText
             : ((body.trim().split("\n")[0] || resp.statusText).slice(0, 200));
-        return { ok: resp.ok, status: resp.status, statusText };
+        return { ok, status: displayStatus, statusText };
     }
 
     // ── External edit in Microsoft Word ──