@remit/smtp-service 0.0.17 → 0.0.19

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@remit/smtp-service",
-  "version": "0.0.17",
+  "version": "0.0.19",
   "type": "module",
   "main": "src/index.ts",
   "exports": {
@@ -11,6 +11,10 @@
     "./message-builder": {
       "types": "./src/message-builder.ts",
       "default": "./src/message-builder.ts"
+    },
+    "./test-fake-smtp": {
+      "types": "./src/test-fake-smtp.ts",
+      "default": "./src/test-fake-smtp.ts"
     }
   },
   "scripts": {
@@ -24,7 +28,7 @@
   "dependencies": {
     "nodemailer": "^10.0.6",
     "@remit/data-ports": "*",
-    "@types/nodemailer": "*"
+    "@types/nodemailer": "^8.0.1"
   },
   "license": "MIT",
   "publishConfig": {

package/src/message-builder.ts

@@ -5,17 +5,17 @@ import nodemailer, { type SendMailOptions } from "nodemailer";
 export interface MailMessage {
 	from: string;
 	to: string[];
-	cc?: string[];
-	bcc?: string[];
-	replyTo?: string;
-	subject?: string;
-	text?: string;
-	html?: string;
+	cc?: string[] | undefined;
+	bcc?: string[] | undefined;
+	replyTo?: string | undefined;
+	subject?: string | undefined;
+	text?: string | undefined;
+	html?: string | undefined;
 	messageId: string;
-	inReplyTo?: string;
-	references?: string;
-	date?: Date;
-	attachments?: MailAttachment[];
+	inReplyTo?: string | undefined;
+	references?: string | undefined;
+	date?: Date | undefined;
+	attachments?: MailAttachment[] | undefined;
 }
 
 export interface MailAttachment {

package/src/smtp-client.test.ts

@@ -3,81 +3,23 @@ import net from "node:net";
 import { after, describe, it } from "node:test";
 import type { MailMessage } from "./message-builder.js";
 import {
+	directSmtpSocket,
 	type SmtpConfig,
 	SmtpConnectionError,
 	type SmtpCredentials,
 	sendMail,
 } from "./smtp-client.js";
-
-interface FakeSmtpOptions {
-	authResponse?: string;
-	dataFinalResponse?: string;
-	dropAfterGreeting?: boolean;
-}
-
-interface FakeSmtp {
-	port: number;
-	authCommands: string[];
-	close: () => void;
+import {
+	type FakeSmtp,
+	type FakeSmtpOptions,
+	startFakeSmtp,
+} from "./test-fake-smtp.js";
+
+interface SocketRequest {
+	host: string | undefined;
+	port: string | number | undefined;
 }
 
-const startFakeSmtp = (opts: FakeSmtpOptions = {}): Promise<FakeSmtp> =>
-	new Promise((resolve) => {
-		const authCommands: string[] = [];
-		const server = net.createServer((socket) => {
-			if (opts.dropAfterGreeting) {
-				socket.destroy();
-				return;
-			}
-			socket.write("220 fake ESMTP\r\n");
-			let inData = false;
-			socket.on("data", (buffer) => {
-				const line = buffer.toString();
-				if (inData) {
-					if (line.includes("\r\n.\r\n")) {
-						inData = false;
-						socket.write(opts.dataFinalResponse ?? "250 2.0.0 Ok\r\n");
-					}
-					return;
-				}
-				const command = line.slice(0, 4).toUpperCase();
-				if (command.startsWith("EHLO") || command.startsWith("HELO")) {
-					socket.write("250-fake\r\n250 AUTH PLAIN LOGIN XOAUTH2\r\n");
-					return;
-				}
-				if (command.startsWith("AUTH")) {
-					authCommands.push(line.trim());
-					socket.write(opts.authResponse ?? "235 2.7.0 Accepted\r\n");
-					return;
-				}
-				if (command.startsWith("MAIL") || command.startsWith("RCPT")) {
-					socket.write("250 2.1.0 Ok\r\n");
-					return;
-				}
-				if (command.startsWith("DATA")) {
-					socket.write("354 End data\r\n");
-					inData = true;
-					return;
-				}
-				if (command.startsWith("QUIT")) {
-					socket.write("221 Bye\r\n");
-					socket.end();
-					return;
-				}
-				socket.write("250 Ok\r\n");
-			});
-		});
-		server.listen(0, "127.0.0.1", () => {
-			const address = server.address();
-			const port = typeof address === "object" && address ? address.port : 0;
-			resolve({
-				port,
-				authCommands,
-				close: () => server.close(),
-			});
-		});
-	});
-
 const configFor = (
 	port: number,
 	credentials: SmtpCredentials = { kind: "password", password: "secret" },
@@ -88,6 +30,7 @@ const configFor = (
 	user: "user@example.com",
 	credentials,
 	connectionTimeout: 2000,
+	getSocket: directSmtpSocket,
 });
 
 const testMessage: MailMessage = {
@@ -121,6 +64,28 @@ describe("sendMail", () => {
 		assert.ok(result.response);
 	});
 
+	it("talks to the server over the socket the host's factory opens", async () => {
+		const server = await spawn();
+		const requests: SocketRequest[] = [];
+		const result = await sendMail(
+			{
+				...configFor(server.port),
+				host: "smtp.invalid",
+				port: 2525,
+				getSocket: (request, callback) => {
+					requests.push({ host: request.host, port: request.port });
+					const connection = net.connect(server.port, "127.0.0.1", () =>
+						callback(null, { connection }),
+					);
+				},
+			},
+			testMessage,
+		);
+
+		assert.equal(result.success, true);
+		assert.deepEqual(requests, [{ host: "smtp.invalid", port: 2525 }]);
+	});
+
 	it("sends attachments through the transport", async () => {
 		const server = await spawn();
 		const result = await sendMail(configFor(server.port), {

package/src/smtp-client.ts

@@ -1,4 +1,5 @@
 import nodemailer from "nodemailer";
+import type { SMTPTransportGetSocket } from "nodemailer/lib/smtp-transport";
 import { type MailMessage, toNodemailerOptions } from "./message-builder.js";
 
 /**
@@ -31,7 +32,9 @@ export class SmtpConnectionError extends Error {
 	}
 }
 
-export interface SmtpConfig {
+export type SmtpGetSocket = SMTPTransportGetSocket;
+
+export interface SmtpServerConfig {
 	host: string;
 	port: number;
 	secure: boolean; // true for TLS (465), false for STARTTLS (587)
@@ -43,12 +46,20 @@ export interface SmtpConfig {
 	connectionTimeout?: number; // milliseconds, default 30000
 }
 
+export const directSmtpSocket: SmtpGetSocket = (_request, callback) => {
+	queueMicrotask(() => callback(null, false));
+};
+
+export interface SmtpConfig extends SmtpServerConfig {
+	getSocket: SmtpGetSocket;
+}
+
 export interface SendResult {
 	success: boolean;
-	messageId?: string;
-	response?: string;
-	error?: Error;
-	smtpCode?: number;
+	messageId?: string | undefined;
+	response?: string | undefined;
+	error?: Error | undefined;
+	smtpCode?: number | undefined;
 	isTransient: boolean;
 }
 
@@ -95,6 +106,7 @@ export const sendMail = async (
 		connectionTimeout: timeout,
 		greetingTimeout: timeout,
 		socketTimeout: 300_000,
+		getSocket: config.getSocket,
 	});
 
 	return transporter

package/src/test-fake-smtp.ts

@@ -0,0 +1,74 @@
+import net from "node:net";
+
+export interface FakeSmtpOptions {
+	authResponse?: string;
+	dataFinalResponse?: string;
+	dropAfterGreeting?: boolean;
+}
+
+export interface FakeSmtp {
+	port: number;
+	authCommands: string[];
+	received: string[];
+	close: () => void;
+}
+
+export const startFakeSmtp = (opts: FakeSmtpOptions = {}): Promise<FakeSmtp> =>
+	new Promise((resolve) => {
+		const authCommands: string[] = [];
+		const received: string[] = [];
+		const server = net.createServer((socket) => {
+			if (opts.dropAfterGreeting) {
+				socket.destroy();
+				return;
+			}
+			socket.write("220 fake ESMTP\r\n");
+			let inData = false;
+			socket.on("data", (buffer) => {
+				const line = buffer.toString();
+				if (inData) {
+					received.push(line);
+					if (line.includes("\r\n.\r\n")) {
+						inData = false;
+						socket.write(opts.dataFinalResponse ?? "250 2.0.0 Ok\r\n");
+					}
+					return;
+				}
+				const command = line.slice(0, 4).toUpperCase();
+				if (command.startsWith("EHLO") || command.startsWith("HELO")) {
+					socket.write("250-fake\r\n250 AUTH PLAIN LOGIN XOAUTH2\r\n");
+					return;
+				}
+				if (command.startsWith("AUTH")) {
+					authCommands.push(line.trim());
+					socket.write(opts.authResponse ?? "235 2.7.0 Accepted\r\n");
+					return;
+				}
+				if (command.startsWith("MAIL") || command.startsWith("RCPT")) {
+					socket.write("250 2.1.0 Ok\r\n");
+					return;
+				}
+				if (command.startsWith("DATA")) {
+					socket.write("354 End data\r\n");
+					inData = true;
+					return;
+				}
+				if (command.startsWith("QUIT")) {
+					socket.write("221 Bye\r\n");
+					socket.end();
+					return;
+				}
+				socket.write("250 Ok\r\n");
+			});
+		});
+		server.listen(0, "127.0.0.1", () => {
+			const address = server.address();
+			const port = typeof address === "object" && address ? address.port : 0;
+			resolve({
+				port,
+				authCommands,
+				received,
+				close: () => server.close(),
+			});
+		});
+	});