Added custom mail host via MAILER_SERVICE_PROVIDER, MAILER_SMTP_PORT, and MAILER_SMTP_SECURE env vars

app/controllers/users/users.authentication.server.controller.js

@@ -95,7 +95,6 @@ exports.resendVerificationEmail = function(req, res, next){
  * Signup
  */
 exports.signup = function(req, res) {
-
 	// For security measures we remove the roles from the req.body object
 	delete req.body.roles;
 
@@ -106,13 +105,13 @@ exports.signup = function(req, res) {
 	user.provider = 'local';
 	// Then save the temporary user
 	nev.createTempUser(user, function (err, existingPersistentUser, newTempUser) {
-		debugger;
+		if (err) {
-        if (err) {
 			return res.status(400).send({
 				message: errorHandler.getErrorMessage(err)
 			});
 		}
 
+
 		// new user created
 		if (newTempUser) {
 			var URL = newTempUser[nev.options.URLFieldName];

app/models/user.server.model.js

@@ -15,6 +15,14 @@ var mongoose = require('mongoose'),
 
 var smtpTransport = nodemailer.createTransport(config.mailer.options);
 
+// verify connection configuration on startup
+smtpTransport.verify(function(error, success) {
+	if (error) {
+			 console.log('Your mail configuration is incorrect', error);
+	} else {
+			 console.log('Mail server is ready to take our messages');
+	}
+});
 
 /**
  * A Validation function for local strategy properties

config/env/development.js

@@ -43,7 +43,15 @@ module.exports = {
 	},
 	mailer: {
 		from: process.env.MAILER_FROM || 'no-reply@tellform.com',
-		options: {
+		options: process.env.MAILER_SMTP_HOST ? { //Uses custom SMTP if MAILER_SMTP_HOST is set
+			host: process.env.MAILER_SMTP_HOST || '',
+			port: process.env.MAILER_SMTP_PORT || 465,
+			secure: process.env.MAILER_SMTP_SECURE || true,
+			auth: {
+				user: process.env.MAILER_EMAIL_ID || '',
+				pass: process.env.MAILER_PASSWORD || ''
+			}
+		} : {
 			service: process.env.MAILER_SERVICE_PROVIDER || '',
 			auth: {
 				user: process.env.MAILER_EMAIL_ID || '',

config/env/production.js

@@ -59,11 +59,19 @@ module.exports = {
 	},
 	mailer: {
 		from: process.env.MAILER_FROM || 'testing@'+process.env.SPARKPOST_SANDBOX_DOMAIN || 'no-reply@tellform.com',
-		options: {
+		options: process.env.MAILER_SMTP_HOST ? { //Uses custom SMTP if MAILER_SMTP_HOST is set
+			host: process.env.MAILER_SMTP_HOST || '',
+			port: process.env.MAILER_SMTP_PORT || 465,
+			secure: process.env.MAILER_SMTP_SECURE || true,
+			auth: {
+				user: process.env.MAILER_EMAIL_ID || '',
+				pass: process.env.MAILER_PASSWORD || ''
+			}
+		} : {
 			service: process.env.MAILER_SERVICE_PROVIDER || '',
 			auth: {
-				user: process.env.MAILER_EMAIL_ID || process.env.SPARKPOST_SMTP_USERNAME || '',
+				user: process.env.MAILER_EMAIL_ID || '',
-				pass: process.env.MAILER_PASSWORD || process.env.SPARKPOST_SMTP_PASSWORD || ''
+				pass: process.env.MAILER_PASSWORD || ''
 			}
 		}
 	}

config/env/secure.js

@@ -66,7 +66,15 @@ module.exports = {
 	},
 	mailer: {
 		from: process.env.MAILER_FROM || '',
-		options: {
+		options: process.env.MAILER_SMTP_HOST ? { //Uses custom SMTP if MAILER_SMTP_HOST is set
+			host: process.env.MAILER_SMTP_HOST || '',
+			port: process.env.MAILER_SMTP_PORT || 587,
+			secure: process.env.MAILER_SMTP_SECURE || true,
+			auth: {
+				user: process.env.MAILER_EMAIL_ID || '',
+				pass: process.env.MAILER_PASSWORD || ''
+			}
+		} : {
 			service: process.env.MAILER_SERVICE_PROVIDER || '',
 			auth: {
 				user: process.env.MAILER_EMAIL_ID || '',

config/env/test.js

@@ -52,7 +52,15 @@ module.exports = {
 	},
 	mailer: {
 		from: process.env.MAILER_FROM || 'MAILER_FROM',
-		options: {
+		options: process.env.MAILER_SMTP_HOST ? { //Uses custom SMTP if MAILER_SMTP_HOST is set
+			host: process.env.MAILER_SMTP_HOST || '',
+			port: process.env.MAILER_SMTP_PORT || 587,
+			secure: process.env.MAILER_SMTP_SECURE || true,
+			auth: {
+				user: process.env.MAILER_EMAIL_ID || '',
+				pass: process.env.MAILER_PASSWORD || ''
+			}
+		} : {
 			service: process.env.MAILER_SERVICE_PROVIDER || '',
 			auth: {
 				user: process.env.MAILER_EMAIL_ID || '',