Merge pull request #18 from augustuswm/master

chipbell4 · web-flow · commit 444d07270a1a · 2018-08-21T10:56:57.000-04:00
next is not defined when gsmtp authentication fails
diff --git a/app/routes/forgot.js b/app/routes/forgot.js
@@ -4,7 +4,7 @@ var crypto = require('crypto');
 var async = require('async');
 var sendmail = require('../helpers/sendmail');
 
-router.post('/', function(req, res)
+router.post('/', function(req, res, next)
 {
 	async.waterfall([
 		function(done)
@@ -13,7 +13,7 @@ router.post('/', function(req, res)
 		},
 		function(token, done)
 		{
-			User.getByEmail(req.body.email, function(err, user) 
+			User.getByEmail(req.body.email, function(err, user)
 			{
 				if (!user)
 				{
@@ -48,7 +48,7 @@ router.post('/', function(req, res)
 				done(err);
 			});
 		}
-	], 
+	],
 	function(err)
 	{
 		if (err) return next(err);

Original file line number	Diff line number	Diff line change
`@@ -4,7 +4,7 @@ var crypto = require('crypto');`
`4`	`4`	`var async = require('async');`
`5`	`5`	`var sendmail = require('../helpers/sendmail');`
`6`	`6`
`7`		`-router.post('/', function(req, res)`
	`7`	`+router.post('/', function(req, res, next)`
`8`	`8`	`{`
`9`	`9`	`async.waterfall([`
`10`	`10`	`function(done)`
`@@ -13,7 +13,7 @@ router.post('/', function(req, res)`
`13`	`13`	`},`
`14`	`14`	`function(token, done)`
`15`	`15`	`{`
`16`		`- User.getByEmail(req.body.email, function(err, user)`
	`16`	`+ User.getByEmail(req.body.email, function(err, user)`
`17`	`17`	`{`
`18`	`18`	`if (!user)`
`19`	`19`	`{`
`@@ -48,7 +48,7 @@ router.post('/', function(req, res)`
`48`	`48`	`done(err);`
`49`	`49`	`});`
`50`	`50`	`}`
`51`		`- ],`
	`51`	`+ ],`
`52`	`52`	`function(err)`
`53`	`53`	`{`
`54`	`54`	`if (err) return next(err);`