Global refactor

server/authorization.js

@@ -20,7 +20,7 @@ function getUserRole(found) {
         // Mail contains no number
         roles += ',"author"';
     }
-    return roles;
+    return JSON.stringify('{'+roles+'}');
 }
 
 // Fill in session object
@@ -37,6 +37,7 @@ function fillSession (req, user, roles, cb) {
                     req.session.user = user._id;
                 }
                 req.session.roles = roles;
+                console.info(req.session);
             }
             return cb (err);
         });
@@ -128,7 +129,7 @@ const authorization = {
             if (entry != null && entry.pwd) {
                 if (crypto.checkLocalAuth (entry, req.body.pwd)) {
                     console.info(entry.roles);
-                    return fillSession (req, entry, JSON.stringify(entry.roles), returnSession);
+                    return fillSession (req, entry, JSON.parse(entry.roles), returnSession);
                 }
                 return returnError ();
             }

server/dbs.js

@@ -81,35 +81,6 @@ const dbs = {
                     });
             },
         },
-
-        "usr/bookmark/ids": {
-            /* GET /api/usr/bookmark/ids [no args]
-             * -> Array of message schema object ids
-             * Get bookmark message ids
-             */
-
-            get: function (req, res) {
-
-                model.Users.find({
-
-                    },
-                    function (err, result) {
-                        if (err) {
-                            return res.status(401).json(err.message);
-                        } else {
-                            var parsed = [];
-                            for (var i in result) {
-                                parsed.push(result[i].bookmarks);
-                            }
-                            console.log("bookmarks:" + JSON.stringify(parsed));
-                            res.send(parsed);
-                        }
-                        if (result == null) {
-                            return res.status(500).json("Can save bookmark.")
-                        }
-                    });
-            },
-        },
         "msg": {
             /* POST /api/msg
              * <- Message schema
@@ -190,6 +161,32 @@ const dbs = {
                     });
             },
         },
+        "usr/bookmark/ids": {
+            /* GET /api/usr/bookmark/ids [no args]
+             * -> Array of message schema object ids
+             * Get bookmark message ids
+             */
+
+            get: function (req, res) {
+                model.Users.find({
+                    },
+                    function (err, result) {
+                        if (err) {
+                            return res.status(401).json(err.message);
+                        } else {
+                            var parsed = [];
+                            for (var i in result) {
+                                parsed.push(result[i].bookmarks);
+                            }
+                            console.log("bookmarks:" + JSON.stringify(parsed));
+                            res.send(parsed);
+                        }
+                        if (result == null) {
+                            return res.status(500).json("Can save bookmark.")
+                        }
+                    });
+            },
+        },
         "usr/id": {
             params: ":id",
             /* GET /api/usr/[usr-id]
@@ -380,7 +377,7 @@ const dbs = {
             bookmarks: { type: [String],
                       _comment: "" },
             sessionid: { type: String },
-            roles:   { type: [String], required: true,
+            roles:   { type: String, required: true,
                       _comment: "" },
         });
         model.Users = common.mongoose.model('users', userSchema);