From 6d3bdaeaea164b1c46734286bcdd96fb32b777a3 Mon Sep 17 00:00:00 2001
From: Sahat Yalkabov <sakhat@gmail.com>
Date: Tue, 18 Feb 2014 03:13:00 -0500
Subject: [PATCH] Added callback to async.waterfall for error handling via
 express middleware

---
 controllers/forgot.js | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/controllers/forgot.js b/controllers/forgot.js
index 5e48af19eb..a66095c13d 100644
--- a/controllers/forgot.js
+++ b/controllers/forgot.js
@@ -24,7 +24,7 @@ exports.getForgot = function(req, res) {
  * @param email
  */
 
-exports.postForgot = function(req, res) {
+exports.postForgot = function(req, res, next) {
   req.assert('email', 'Please enter a valid email address.').isEmail();
 
   var errors = req.validationErrors();
@@ -76,8 +76,10 @@ exports.postForgot = function(req, res) {
       smtpTransport.sendMail(mailOptions, function(err) {
         req.flash('info', { msg: 'An e-mail has been sent to ' + user.email + ' with further instructions.' });
         done(err, 'done');
-        res.redirect('/forgot');
       });
     }
-  ]);
+  ], function(err) {
+    if (err) return next(err);
+    res.redirect('/forgot');
+  });
 };