Trap a potential error (#1448)

* Removed WARNING comment as it's now trapped

Post #604

Auto-merge

Author: Martii

controllers/auth.js

@@ -149,10 +149,15 @@ exports.auth = function (aReq, aRes, aNext) {
 
   User.findOne({ name: { $regex: new RegExp('^' + username + '$', 'i') } },
     function (aErr, aUser) {
-      // WARNING: No error handling at this stage
       var strategies = null;
       var strat = null;
 
+      if (aErr) {
+        console.error('Authfail with no User found of', username, aErr);
+        aRes.redirect('/login?usernamefail');
+        return;
+      }
+
       if (aUser) {
         // Ensure that casing is identical so we still have it, correctly, when they
         // get back from authentication