molex-ftp-client 1.2.1 → 2.1.0

package/lib/connection.js

@@ -1,4 +1,4 @@
-const net = require('net');
+const { createOptimizedSocket } = require('./performance');
 
 /**
  * Handle FTP connection establishment and authentication
@@ -21,14 +21,10 @@ class FTPConnection {
     this.client._debug(`Connecting to ${host}:${port} as ${user}`);
     
     return new Promise((resolve, reject) => {
-      this.client.socket = net.createConnection({ host, port }, () => {
+      this.client.socket = createOptimizedSocket({ host, port }, () => {
         this.client.connected = true;
         this.client._debug('TCP connection established');
         
-        if (this.client.keepAlive) {
-          this.client.socket.setKeepAlive(true, 10000);
-        }
-        
         this.client.emit('connected');
       });
 
@@ -72,7 +68,7 @@ class FTPConnection {
         this.client.emit('close');
       });
 
-      setTimeout(() => reject(new Error('Connection timeout')), 10000);
+      setTimeout(() => reject(new Error('Connection timeout')), this.client.timeout || 10000);
     });
   }
 
@@ -161,11 +157,18 @@ class FTPConnection {
     if (this.client.connected) {
       this.client._debug('Closing connection...');
       try {
+        // Clean up data socket if exists
+        if (this.client.dataSocket) {
+          this.client.dataSocket.destroy();
+          this.client.dataSocket = null;
+        }
         await this.sendCommand('QUIT');
       } catch (err) {
         this.client._debug('Error during QUIT:', err.message);
       }
-      this.client.socket.end();
+      if (this.client.socket) {
+        this.client.socket.end();
+      }
       this.client.connected = false;
       this.client.authenticated = false;
       this.client._debug('Connection closed');

package/lib/performance.js

@@ -0,0 +1,29 @@
+/**
+ * TCP Performance optimization utilities
+ * Applies sensible defaults for FTP connections
+ */
+
+const net = require('net');
+
+/**
+ * Create an optimized TCP socket connection
+ * Automatically applies TCP_NODELAY and keep-alive
+ * @param {Object} options - Connection options (host, port)
+ * @param {Function} callback - Callback on connection
+ * @returns {net.Socket}
+ */
+function createOptimizedSocket(options, callback) {
+  const socket = net.createConnection(options, callback);
+  
+  // TCP_NODELAY - Disable Nagle's algorithm for lower latency
+  socket.setNoDelay(true);
+  
+  // SO_KEEPALIVE - Detect dead connections
+  socket.setKeepAlive(true, 10000);
+  
+  return socket;
+}
+
+module.exports = {
+  createOptimizedSocket
+};

package/lib/utils.js

@@ -3,24 +3,6 @@
  * Utilities for building and parsing FTP commands
  */
 
-/**
- * Parse PASV response to extract host and port
- * @param {string} message - PASV response message
- * @returns {Object} - { host, port }
- */
-function parsePasvResponse(message) {
-  const match = message.match(/\((\d+),(\d+),(\d+),(\d+),(\d+),(\d+)\)/);
-  
-  if (!match) {
-    throw new Error('Failed to parse PASV response');
-  }
-
-  const host = `${match[1]}.${match[2]}.${match[3]}.${match[4]}`;
-  const port = parseInt(match[5]) * 256 + parseInt(match[6]);
-
-  return { host, port };
-}
-
 /**
  * Parse MDTM response to Date object
  * @param {string} message - MDTM response message
@@ -76,7 +58,6 @@ function maskPassword(command) {
 }
 
 module.exports = {
-  parsePasvResponse,
   parseMdtmResponse,
   normalizePath,
   getParentDir,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "molex-ftp-client",
-  "version": "1.2.1",
+  "version": "2.1.0",
   "description": "Lightweight FTP client using native Node.js TCP sockets (net module) with zero dependencies",
   "main": "index.js",
   "scripts": {