Améliorations diverses

* Correction de certains plantages eventuels si aucune donnée n'était envoyée * Amélioré l'affichage de certaines erreurs

Améliorations diverses
* Correction de certains plantages eventuels si aucune donnée n'était envoyée * Amélioré l'affichage de certaines erreurs
Geoffrey PREUD'HOMME
1 parent 097f26f7
Showing 6 changed files with 48 additions and 34 deletions Show diff stats
app/routes/ApiRtes.js
app/services/MembresServ.js
public/js/controllers/ConnectCtrl.js
public/js/controllers/MembreCtrl.js
public/js/services/ApiServ.js
public/js/services/SessionServ.js
@@ -10,6 +10,9 @@ var api = express();
 // Authentication
 reqAuth = function () {
     return function (req, res, next) {
+        if (!req.cookies) {
+            res.status(401).end();
+        }
         SessionsServ.use(req.cookies.session, function (err, session) {
             if (err) {
                 res.status(500).send(err);
@@ -51,22 +54,32 @@ reqPerm = function (perm) {
 assert = function (test) {
     return function (req, res, next) {
-        reqAuth()(req, res, function () {
-            test(req, res, function (err, verified) {
-                if (err) {
-                    res.status(500).send(err);
+        test(req, res, function (err, verified) {
+            if (err) {
+                res.status(500).send(err);
+            } else {
+                if (verified) {
+                    next();
                 } else {
-                    if (verified) {
-                        next();
-                    } else {
-                        res.status(400).end();
-                    }
+                    res.status(400).end();
                 }
-            });
+            }
         });
     };
 };
+decrypt = function () {
+    return function (req, res, next) {
+        assert(function (req, res, cb) {
+            cb(null, req.body && typeof req.body[0] == 'string' && req.body[0] !== '');
+        })(req, res, function () {
+            DecryptServ.decrypt(req.body[0], function (data) {
+                req.body = JSON.parse(data);
+                next();
+            });
+        });
+    };
+};
 // Sessions
 api.get('/session', function (req, res) { // Informations sur la session
@@ -84,16 +97,16 @@ api.get(&#39;/session&#39;, function (req, res) { // Informations sur la session
     }
 });
-api.post('/session', function (req, res) { // Se connecter
-    DecryptServ.decrypt(req.body[0], function (data) {
-        SessionsServ.open(JSON.parse(data), function (err, session) {
-            if (err) {
-                res.status(500).send(err);
-            } else {
-                res.cookie('session', session._id);
-                res.send(session);
-            }
-        });
+api.post('/session', decrypt(), assert(function (req, res, cb) {
+    cb(null, req.body && typeof req.body.login == 'string' && req.body.login !== '' && typeof req.body.pass == 'string' && req.body.pass !== '');
+}), function (req, res) { // Se connecter
+    SessionsServ.open(req.body, function (err, session) {
+        if (err) {
+            res.status(500).send(err);
+        } else {
+            res.cookie('session', session._id);
+            res.send(session);
+        }
     });
 });
@@ -49,10 +49,8 @@ MembresServ.estBureau = function (login, cb) {
     }, function (err, data) {
         if (!err && data && data.role != 'Membre') {
             cb(true);
-            console.log(true);
         } else {
             cb(false);
-            console.log(false);
         }
     });
 };
@@ -7,7 +7,7 @@ angular.module(&#39;ConnectCtrl&#39;, [&#39;SessionsServ&#39;, &#39;EncryptServ&#39;, &#39;angular-ladda&#39;]).
         $scope.connect = {
             connect: function () {
                 $scope.connecting = true;
-                SessionServ.connect($scope.connect.login, $scope.connect.pass, function(err) {
+                SessionServ.connect($scope.connect.login, $scope.connect.pass, function (err) {
                     $scope.connecting = false;
                     if (!err) {
                         window.history.back();
@@ -14,14 +14,18 @@ angular.module(&#39;MembreCtrl&#39;, [&#39;SessionsServ&#39;, &#39;ApiServ&#39;]).controller(&#39;MembreCtrl
         $scope.createMembre = function () {
             ApiServ("création du membre", 'post', 'membres', $scope.formData, function (err, membre) {
-                $scope.formData = {};
-                $scope.membres.push(membre);
+                if (!err) {
+                    $scope.formData = {};
+                    $scope.membres.push(membre);
+                }
             });
         };
         $scope.deleteMembre = function (index) {
             ApiServ("création du membre", 'delete', 'membres', $scope.membres[index]._id, function (err, data) {
-                $scope.membres.splice(index, 1);
+                if (!err) {
+                    $scope.membres.splice(index, 1);
+                }
             });
         };
     }
@@ -26,7 +26,7 @@ angular.module(&#39;ApiServ&#39;, [&#39;NotifyServ&#39;]).service(&#39;ApiServ&#39;, [&#39;$http&#39;, &#39;NotifySe
                 })
                 .error(function (data, status) {
                     cb(status);
-                    NotifyServ.error("Échec : ", name, status + (data ? ' : ' + data : ''));
+                    NotifyServ.error("Échec : ", name, status + (data ? ' : ' + JSON.stringify(data) : ''));
                     // console.error(name, status, data);
                 });
         };
@@ -47,18 +47,17 @@ angular.module(&#39;SessionsServ&#39;, [&#39;NotifyServ&#39;, &#39;EncryptServ&#39;]).service(&#39;SessionSe
                         _this.updateSessionInfos(body);
                         if (_this.cur) {
                             not.success("Connecté en tant que <strong>" + _this.cur.nom + "</strong>");
-                            if (cb)
-                                cb(null);
+                            cb(null);
                         } else {
                             if (body === 'invalid') {
                                 not.warn("Identifiants invalides");
                             }
-                            if (cb)
-                                cb(body);
+                            cb(body);
                         }
-                    }).error(function (body) {
-                        not.error("Impossible de se connecter", body);
-                        cb(body);
+                    }).error(function (data, status) {
+                        err = status + (data ? ' : ' + JSON.stringify(data) : '');
+                        not.error("Impossible de se connecter", err);
+                        cb(err);
                     });
                 });
             },