fix: correct smtp port for local and prod

Author: dandelionn

.devcontainer/.env.production.local.example

@@ -3,6 +3,9 @@ PORT=3000
 FRONTEND_URL=http://localhost:3000
 
 DATABASE_URL=postgresql://admin_pg:password@postgres:5432/express-hybrid-auth-api-local-prod-db
+# Use the Docker service name 'postgres' as the host because the app runs inside a container.
+# Port 5432 is the default PostgreSQL port inside the container.
+# From the host machine (local Node.js), you would connect to localhost:5433 because of the port mapping in docker-compose.yml.
 
 GOOGLE_CLIENT_ID=
 GOOGLE_CLIENT_SECRET=
@@ -14,11 +17,13 @@ GITHUB_CALLBACK_URL=http://localhost:3000/auth/github/callback
 
 NODE_ENV=development
 
-REDIS_PORT=6379 
-REDIS_HOST=redis # using redis service name because the app will run in a container so localhost is not correct
+REDIS_PORT=6379
+REDIS_HOST=redis  # Use the Docker service name 'redis' because the app runs inside a container.
+                  # 'localhost' would point to the container itself, not the Redis container.
 
-SMTP_HOST=smtp4dev
-SMTP_PORT=2525
+SMTP_HOST=smtp4dev  # Docker service name for smtp4dev container
+SMTP_PORT=25        # Container's SMTP port (25). 
+                    # Use port 25 here because inside the container, Docker networking connects directly to smtp4dev's internal port.
 
 SESSION_SECRET="QeWMItONAv+x6AmdOJ2D5O1h/d//DjO5bpmjz6O0AOnxfc7Q4OlHYeXxX2HE6l0N2zEMZTAYmmRc8Z50UAFPHA=="
 ACCESS_TOKEN_SECRET="F9HHTuGGzVhbzB6rMoTLRVlZu7RhsicD6iF00xDn5K4UIlHtkcecz1bMyDRaewfjTUqmoTH6UWN22Bf7qymGBg=="

.devcontainer/docker-compose.dev.yml

@@ -43,7 +43,7 @@ services:
         image: rnwood/smtp4dev
         ports:
             - '5000:80' # Web UI
-            - '2525:2525' # SMTP server
+            - '2525:25' # SMTP server (Host 2525 -> Container 25 (SMTP))
         restart: unless-stopped
         environment:
             - Logging__LogLevel__Default=Information

.devcontainer/docker-compose.prod.yml

@@ -64,7 +64,7 @@ services:
         image: rnwood/smtp4dev
         ports:
             - '5000:80'
-            - '2525:2525'
+            - '2525:25' # SMTP server (Host 2525 -> Container 25 (SMTP))
         restart: unless-stopped
         environment:
             - Logging__LogLevel__Default=Information

src/lib/mail-transporter.ts

@@ -11,3 +11,16 @@ export const transporter = nodemailer.createTransport({
 	//     pass: '',
 	//   },
 });
+
+transporter.sendMail(
+	{
+		from: 'test@example.com',
+		to: 'user@example.com',
+		subject: 'SMTP test',
+		text: 'Hello from smtp4dev!',
+	},
+	(err, info) => {
+		if (err) console.error('Error:', err);
+		else console.log('Sent:', info);
+	}
+);

src/lib/mailer.tsx

@@ -7,6 +7,7 @@ import TwoFactorDisabledEmail from '../../emails/templates/2fa-disabled-email.js
 import PasswordResetEmail from '../../emails/templates/password-reset-email.js';
 import GlobalConfig from '../config.js';
 import { transporter } from './mail-transporter.js';
+import logger from './logger/logger.js';
 
 export async function sendVerificationEmail({
 	token,
@@ -19,6 +20,7 @@ export async function sendVerificationEmail({
 	expiresInMinutes: number;
 	t: (key: string, options?: any) => string;
 }) {
+	console.log('I am here');
 	const emailHtml = await render(
 		<VerificationEmail
 			verificationUrl={`${GlobalConfig.EMAIL_FRONTEND_BASE_URL}/verify?token=${token}`}
@@ -36,7 +38,14 @@ export async function sendVerificationEmail({
 		html: emailHtml,
 	};
 
-	await transporter.sendMail(options);
+	try {
+		console.log('I am here');
+		await transporter.sendMail(options);
+	} catch (error) {
+		console.log('error', error);
+		logger.error('Failed to send verification email', { error, userEmail });
+		throw error;
+	}
 }
 
 export async function sendAccountActivationEmail({