Serveur: Vérification des permissions de modifications de membres

Code amélioré en divers points

Serveur: Vérification des permissions de modifications de membres
Code amélioré en divers points
Geoffrey PREUD'HOMME
1 parent cb1c1601
Showing 7 changed files with 77 additions and 46 deletions Show diff stats
app/controllers/sessions.js
app/routes/api.js
public/js/controllers/MembreCtrl.js
public/js/controllers/SessionCtrl.js
public/js/services/SessionServ.js
public/views/index.html
public/views/membres.html
@@ -3,6 +3,8 @@ var noms = require(&#39;../controllers/noms&#39;);
  
 var sessions = {}
  
+sessions.cur = false
+
 sessions.addData = function (session, cb) {
     noms.get(session.login, function (nom) {
         if (typeof nom == 'string') {
@@ -10,6 +12,8 @@ sessions.addData = function (session, cb) {
         } else {
             session.nom = 'Inconnu'
         }
+        session.canAddMembre = session.login == 'gbontoux'
+        session.canDelMembre = session.login == 'gbontoux'
         cb(session)
     })
 }
@@ -45,7 +49,7 @@ sessions.verify = function (id, cb) {
         } else {
             if (session) {
                 if (sessions.valid(session)) {
-                    cb(err, session);
+                    cb(null, session);
                 } else {
                     cb('expired');
                     _this.delete(id)
@@ -57,6 +61,18 @@ sessions.verify = function (id, cb) {
     });
 }
  
+sessions.use = function (id, cb) {
+    _this = this
+    _this.verify(id, function (err, session) {
+        if (err) {
+            cb(err)
+        } else {
+            _this.cur = session
+            cb(null)
+        }
+    })
+}
+
 sessions.create = function (login, cb) {
     Session.create({
         login: login
@@ -89,7 +105,7 @@ sessions.open = function (data, cb) {
                     if (err) {
                         cb('error');
                     } else {
-                        _this.find(session._id, cb)
+                        _this.use(session._id, cb)
                     }
                 });
             } else {
@@ -7,11 +7,11 @@ var api = express()
 // Sessions
 api.get('/session', function (req, res) { // Informations sur la session
     if (req.cookies && req.cookies.session) {
-        sessions.verify(req.cookies.session, function (err, session) {
+        sessions.use(req.cookies.session, function (err) {
             if (err) {
                 res.send(err)
             } else {
-                res.send(session)
+                res.send(sessions.cur)
             }
         })
         // TODO si pas bon : res.clearCookie('session')
@@ -21,12 +21,12 @@ api.get(&#39;/session&#39;, function (req, res) { // Informations sur la session
 });
  
 api.post('/session', function (req, res) { // Se connecter
-    sessions.open(req.body, function (err, session) {
+    sessions.open(req.body, function (err) {
         if (err) {
             res.send(err)
         } else {
-            res.cookie('session', session._id);
-            res.send(session)
+            res.cookie('session', sessions.cur._id);
+            res.send(sessions.cur)
         }
     })
 })
@@ -42,6 +42,20 @@ api.delete(&#39;/session&#39;, function (req, res) { // Se déconnecter
     }
 })
  
+ifPermission = function (req, res, perm, cb) {
+    sessions.use(req.cookies.session, function (err) {
+        if (err) {
+            res.status(403).end()
+        } else {
+            if (sessions.cur[perm]) {
+                cb()
+            } else {
+                res.status(403).end()
+            }
+        }
+    })
+}
+
  
 // Membres
 api.get('/membres', function (req, res) { // Liste des membres
@@ -53,27 +67,31 @@ api.get(&#39;/membres&#39;, function (req, res) { // Liste des membres
 });
  
 api.post('/membres', function (req, res) { // Ajout d'un membre
-    membres.add(req.body, function (err, membre) {
-        if (err)
-            res.send(err);
-        membres.list(function (err, membres) {
+    ifPermission(req, res, 'canAddMembre', function () {
+        membres.add(req.body, function (err, membre) {
             if (err)
                 res.send(err);
-            res.json(membres);
+            membres.list(function (err, membres) {
+                if (err)
+                    res.send(err);
+                res.json(membres);
+            });
         });
-    });
+    })
 });
  
 api.delete('/membres/:membre_id', function (req, res) { // Supression d'un membre
-    membres.remove(req.params.membre_id, function (err, membre) {
-        if (err)
-            res.send(err);
-        membres.list(function (err, membres) {
+    ifPermission(req, res, 'canDelMembre', function () {
+        membres.remove(req.params.membre_id, function (err, membre) {
             if (err)
                 res.send(err);
-            res.json(membres);
+            membres.list(function (err, membres) {
+                if (err)
+                    res.send(err);
+                res.json(membres);
+            });
         });
-    });
+    })
 })
  
 module.exports = api;
 \ No newline at end of file
 angular.module('MembreCtrl', []).controller('MembreController', ['$scope', '$http', 'SessionService',
     function ($scope, $http, SessionService) {
         $scope.formData = {};
-        $scope.canAdd = SessionService.logged
-        $scope.canDel = SessionService.logged
  
-        // when landing on the page, get all Membres and show them
+        $scope.session = SessionService.cur
+        SessionService.onChange(function () {
+            $scope.session = SessionService.cur
+        })
+
         $http.get('/api/membres')
             .success(function (data) {
                 $scope.membres = data;
@@ -14,12 +16,11 @@ angular.module(&#39;MembreCtrl&#39;, []).controller(&#39;MembreController&#39;, [&#39;$scope&#39;, &#39;$htt
                 console.log('Error: ' + data);
             });
  
-        // when submitting the add form, send the text to the node API
         $scope.createMembre = function () {
             console.log('Adding', $scope.formData);
             $http.post('/api/membres', $scope.formData)
                 .success(function (data) {
-                    $scope.formData = {}; // clear the form so our user is ready to enter another
+                    $scope.formData = {};
                     $scope.membres = data;
                 })
                 .error(function (data) {
@@ -27,7 +28,6 @@ angular.module(&#39;MembreCtrl&#39;, []).controller(&#39;MembreController&#39;, [&#39;$scope&#39;, &#39;$htt
                 });
         };
  
-        // delete a Membre after checking it
         $scope.deleteMembre = function (id) {
             $http.delete('/api/membres/' + id)
                 .success(function (data) {
 angular.module('SessionsCtrl', []).controller('SessionController', ['$scope', 'SessionService',
     function ($scope, SessionService) {
-        $scope.session = SessionService
-        // $scope.session.onChange(function () {
-        //     // TODO
-        // })
+        $scope.session = SessionService.cur
+        $scope.disconnect = function () {
+            SessionService.disconnect()
+        }
+        SessionService.onChange(function () {
+            $scope.session = SessionService.cur
+        })
         // $scope.$on("$destroy", function () {
         //     // TODO
         // })
 angular.module('SessionsServ', []).service('SessionService', ['$http',
     function ($http) {
         a = {
-            nom: "Invité",
-            logged: false,
+            cur: false,
             status: 0,
             changeHandlers: [],
             onChange: function (fun) {
@@ -14,18 +13,15 @@ angular.module(&#39;SessionsServ&#39;, []).service(&#39;SessionService&#39;, [&#39;$http&#39;,
                 }
             },
             updateSessionInfos: function (data) {
+                console.log("Connection:", data)
                 if (typeof data === 'object') {
-                    console.log("Connected")
-                    this.logged = true
-                    this.nom = data.nom
+                    this.cur = data
                 } else {
-
-                    this.logged = false
+                    this.cur = false
                 }
                 this.triggerChange()
             },
             get: function (cb) { // Fetch infos if needed
-                console.log("Session: get")
                 if (status == 0) {
                     this.status = 1 // Fetching
                     _this = this
@@ -41,11 +37,10 @@ angular.module(&#39;SessionsServ&#39;, []).service(&#39;SessionService&#39;, [&#39;$http&#39;,
                         }
                     })
                 } else {
-                    console.warn("get multiple times")
+                    console.warn("Unnecessary get() call")
                 }
             },
             connect: function (login, pass, cb) {
-                console.log("Session: connecting with login:", login)
                 _this = this
                 $http.post('/api/session', {
                     login: login,
@@ -62,9 +57,8 @@ angular.module(&#39;SessionsServ&#39;, []).service(&#39;SessionService&#39;, [&#39;$http&#39;,
                 })
             },
             disconnect: function () {
-                console.log("Session: disconnect", this.name)
+                this.updateSessionInfos(false)
                 $http.delete('/api/session')
-                this.logged = false
             }
         }
         a.get()
@@ -31,8 +31,8 @@
                         </ul>
                         <ul class="nav navbar-nav navbar-right" ng-controller="SessionController">
                             <li>
-                                <a ng-hide="session.logged" href="/connect">Se connecter</a>
-                                <a ng-show="session.logged" href="#" ng-click="session.disconnect()">{{ session.nom }} <span class="glyphicon glyphicon-off"></span></a>
+                                <a ng-hide="session" href="/connect">Se connecter</a>
+                                <a ng-show="session" href="#" ng-click="disconnect()">{{ session.nom }} <span class="glyphicon glyphicon-off"></span></a>
                             </li>
                         </ul>
                     </div>
@@ -10,7 +10,7 @@
                 <th>Nom</th>
                 <th>Section</th>
                 <th>Rôle</th>
-                <th ng-if="canAdd || canDel">Action</th>
+                <th ng-if="session.canAddMembre || session.canDelMembre">Action</th>
             </tr>
         </thead>
         <tbody ng-repeat="membre in membres">
@@ -18,13 +18,13 @@
                 <td>{{ membre.login }}</td>
                 <td>{{ membre.section }}</td>
                 <td>{{ membre.role }}</td>
-                <td ng-if="canDel"><button type="button" class="btn btn-danger" aria-label="Expulser" ng-click="deleteMembre(membre._id)">
+                <td ng-if="session.canDelMembre"><button type="button" class="btn btn-danger" aria-label="Expulser" ng-click="deleteMembre(membre._id)">
                     <span class="glyphicon glyphicon-remove" aria-hidden="true"></span>
                     </button>
                 </td>
             </tr>
         </tbody>
-        <tfoot id="membre-form" ng-if="canAdd">
+        <tfoot id="membre-form" ng-if="session.canAddMembre">
         <tr class="form-group">
             <td>
                 <input type="text" class="form-control input-lg" placeholder="Login" ng-model="formData.login">