wip: Email service should use an email sender adress instead of the S…

…MTP authentication user #990
kalisio · Nov 18, 2024 · ec62083 · ec62083
1 parent 1d1d91a
commit ec62083
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/core/api/hooks/hooks.push.js b/core/api/hooks/hooks.push.js
@@ -29,7 +29,7 @@ export async function sendNewSubscriptionEmail (hook) {
   const domainPath = app.get('domain') + '/#/'
   const email = {
     subject: 'Security alert - new browser detected',
-    from: mailerService.options.auth.user,
+    from: mailerService.options.from || mailerService.options.auth.user,
     // When changing email send to the new one so that it can be verified
     to: updatedUser.email,
     link: domainPath,

diff --git a/core/api/services/account/account.service.js b/core/api/services/account/account.service.js
@@ -42,7 +42,7 @@ export default function (name, app, options) {
     const mailerService = app.getService('mailer')
     const domainPath = app.get('domain') + '/#/'
     const email = {
-      from: mailerService.options.auth.user,
+      from: mailerService.options.from || mailerService.options.auth.user,
       // When changing email send to the new one so that it can be verified
       to: (type === 'identityChange' ? user.verifyChanges.email : user.email),
       domainPath