imapflow 1.1.0 → 1.1.1

package/.release-please-manifest.json

@@ -1,3 +1,3 @@
 {
-    ".": "1.1.0"
+    ".": "1.1.1"
 }

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## [1.1.1](https://github.com/postalsys/imapflow/compare/imapflow-v1.1.0...imapflow-v1.1.1) (2025-10-27)
+
+
+### Bug Fixes
+
+* prevent race condition in initial IMAP greeting reception ([4841f79](https://github.com/postalsys/imapflow/commit/4841f79cd4a94067747c3139fb1de41df49ed533))
+
 ## [1.1.0](https://github.com/postalsys/imapflow/compare/imapflow-v1.0.200...imapflow-v1.1.0) (2025-10-23)
 
 

package/lib/imap-flow.js

@@ -1598,6 +1598,7 @@ class ImapFlow extends EventEmitter {
                     this.log.info(logInfo);
 
                     this.setSocketHandlers();
+                    this.setEventHandlers();
                     this.socket.pipe(this.streamer);
 
                     // executed by initial "* OK"
@@ -1633,8 +1634,6 @@ class ImapFlow extends EventEmitter {
                 this.log.error({ err, cid: this.id });
                 reject(err);
             });
-
-            this.setEventHandlers();
         });
     }
 

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "imapflow",
-    "version": "1.1.0",
+    "version": "1.1.1",
     "description": "IMAP Client for Node",
     "main": "lib/imap-flow.js",
     "types": "lib/imap-flow.d.ts",

package/test/connection-edge-cases-test.js

@@ -647,3 +647,56 @@ module.exports['Connection Edge: Capability update after STARTTLS'] = test => {
             test.done();
         });
 };
+
+module.exports['Connection Edge: Event handlers attached before piping'] = test => {
+    let client = new ImapFlow({
+        host: 'imap.example.com',
+        port: 993,
+        auth: { user: 'test', pass: 'test' }
+    });
+
+    // Track the order of operations during the connection flow
+    let pipeCalledBeforeHandlers = false;
+    let eventHandlersAttached = false;
+    let pipeWasCalled = false;
+
+    // Override setEventHandlers to track when it's called
+    let originalSetEventHandlers = client.setEventHandlers.bind(client);
+    client.setEventHandlers = function () {
+        eventHandlersAttached = true;
+        return originalSetEventHandlers();
+    };
+
+    // Create a mock socket with pipe tracking
+    let mockSocket = new EventEmitter();
+    mockSocket.setKeepAlive = () => {};
+    mockSocket.setTimeout = () => {};
+    mockSocket.remotePort = 993;
+    mockSocket.remoteAddress = '127.0.0.1';
+    mockSocket.localAddress = '127.0.0.1';
+    mockSocket.localPort = 12345;
+    mockSocket.destroyed = false;
+    mockSocket.pipe = function (dest) {
+        pipeWasCalled = true;
+        if (!eventHandlersAttached) {
+            pipeCalledBeforeHandlers = true;
+        }
+        // Mock pipe behavior - just return the destination
+        return dest;
+    };
+
+    // Assign mock socket
+    client.socket = mockSocket;
+    client.writeSocket = mockSocket;
+
+    // Simulate the onConnect flow that happens in the actual code
+    client.setSocketHandlers();
+    client.setEventHandlers();
+    client.socket.pipe(client.streamer);
+
+    test.ok(eventHandlersAttached, 'Event handlers should be attached');
+    test.ok(pipeWasCalled, 'Socket pipe should be called');
+    test.ok(!pipeCalledBeforeHandlers, 'Event handlers should be attached before piping socket to streamer');
+
+    test.done();
+};