Merge pull request #566 from antoniomika/am/fix-relay-email

Don't send auth when both user and pass are empty
sinamics · Oct 2, 2024 · 01e9c08 · 01e9c08
2 parents 5fd0bee + 77bedc1
commit 01e9c08
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 5 deletions.
diff --git a/.devcontainer/docker-compose.yml b/.devcontainer/docker-compose.yml
@@ -24,7 +24,7 @@ services:
     volumes:
       - .:/workspaces:cached
     environment:
-      NEXTAUTH_URL: "http://10.0.0.217:3000" # Update this to the URL of your dev environment
+      NEXTAUTH_URL: "${NEXTAUTH_URL:-http://10.0.0.217:3000}"
     networks:
       - app-network
     depends_on:

diff --git a/src/utils/mail.ts b/src/utils/mail.ts
@@ -277,10 +277,13 @@ export async function createTransporter() {
 		host: globalOptions.smtpHost,
 		port: globalOptions.smtpPort,
 		secure: globalOptions.smtpUseSSL,
-		auth: {
-			user: globalOptions.smtpUsername,
-			pass: globalOptions.smtpPassword,
-		},
+		auth:
+			globalOptions.smtpUsername || globalOptions.smtpPassword
+				? {
+						user: globalOptions.smtpUsername,
+						pass: globalOptions.smtpPassword,
+				  }
+				: undefined,
 		tls: {
 			rejectUnauthorized: true,
 			minVersion: "TLSv1.2",