Merge pull request #123 from bwyx/patch-1

better roles config
hagopj13 · Jun 6, 2021 · ca23415 · ca23415
2 parents d2aec4f + 19cda9c
commit ca23415
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/src/config/roles.js b/src/config/roles.js
@@ -1,8 +1,10 @@
-const roles = ['user', 'admin'];
+const allRoles = {
+  user: [],
+  admin: ['getUsers', 'manageUsers'],
+};
 
-const roleRights = new Map();
-roleRights.set(roles[0], []);
-roleRights.set(roles[1], ['getUsers', 'manageUsers']);
+const roles = Object.keys(allRoles);
+const roleRights = new Map(Object.entries(allRoles));
 
 module.exports = {
   roles,

diff --git a/src/services/token.service.js b/src/services/token.service.js
@@ -11,6 +11,7 @@ const { tokenTypes } = require('../config/tokens');
  * Generate token
  * @param {ObjectId} userId
  * @param {Moment} expires
+ * @param {string} type
  * @param {string} [secret]
  * @returns {string}
  */