wip: Email service should use an email sender adress instead of the SMTP authentication user #990

Author: cnouguier

core/api/hooks/hooks.authentication.js

@@ -17,7 +17,7 @@ export async function verifyGuest (hook) {
   if (!user) return hook
   debug('verifyGuest hook called on ', user._id)
 
-  // Check whether the user has been inivted. If not, nothing to do
+  // Check whether the user has been invited. If not, nothing to do
   if (!user.sponsor) {
     debug('Logged user is not a guest')
     return hook

core/api/hooks/hooks.push.js

@@ -25,12 +25,12 @@ export async function sendNewSubscriptionEmail (hook) {
 
   // Data
   const app = hook.app
+  console.log(app)
   const mailerService = app.getService('mailer')
   const domainPath = app.get('domain') + '/#/'
   const email = {
     subject: 'Security alert - new browser detected',
     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,
     domainPath