not-node 6.5.45 → 6.5.47

package/bin/not-builder.js

@@ -9,7 +9,7 @@
 
 */
 
-var argv = require("yargs").argv,
+let argv = require("yargs"),
     fs = require("fs"),
     path = require("path"),
     child_process = require("child_process"),

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "not-node",
-	"version": "6.5.45",
+	"version": "6.5.47",
 	"description": "node complimentary part for client side notFramework.",
 	"main": "index.js",
 	"scripts": {

package/src/auth/exceptions.js

@@ -1,5 +1,15 @@
 const { notError } = require("not-error");
 
+class UserIdentityExceptionUserIdIsNotString extends notError {
+    constructor(typeofUserId) {
+        super("not-node:auth_user_identity_user_id_is_not_string", {
+            typeofUserId,
+        });
+    }
+}
+module.exports.UserIdentityExceptionUserIdIsNotString =
+    UserIdentityExceptionUserIdIsNotString;
+
 class RolesExceptionRoleSetIsNotValid extends notError {
     constructor(name) {
         super("not-node:auth_roles_role_set_is_not_valid", { name });

package/src/identity/providers/session.js

@@ -1,11 +1,12 @@
 const CONST = require("../../auth/const");
 const ROLES = require("../../auth/roles");
+const EXCEPTIONS = require("../../auth/exceptions");
 
 module.exports = class IdentityProviderSession {
     static #options = {};
 
     static setOptions(options = {}) {
-        this.#options = {...this.#options, ...options};
+        this.#options = { ...this.#options, ...options };
     }
 
     static #getOptions() {
@@ -20,6 +21,15 @@ module.exports = class IdentityProviderSession {
         IdentityProviderSession.#getOptions().secondaryRoles = [...list];
     }
 
+    _idToString(_id) {
+        if (typeof _id === "string") {
+            return _id;
+        } else if (_id.toString && typeof _id.toString === "function") {
+            return _id.toString();
+        }
+        throw new EXCEPTIONS.UserIdentityExceptionUserIdIsNotString(typeof _id);
+    }
+
     constructor(req) {
         this.req = req;
         return this;
@@ -93,7 +103,8 @@ module.exports = class IdentityProviderSession {
     setUserId(_id) {
         const req = this.req;
         if (req && req.session) {
-            req.session.user = _id;
+            const stringId = this._idToString(_id);
+            req.session.user = stringId;
             req.session.save();
         }
     }

package/src/init/lib/static.js

@@ -30,7 +30,7 @@ module.exports = class InitStatic {
         return (req, res, next) => {
             try {
                 const frontApp = InitStatic.selectVersion(config, req, ext);
-                let pathTo = path.resolve(options.pathToApp, frontApp);
+                const pathTo = path.resolve(options.pathToApp, frontApp);
                 return InitStatic.serveStatic(pathTo)(req, res, next);
             } catch (e) {
                 next(e);
@@ -41,7 +41,7 @@ module.exports = class InitStatic {
     async run({ config, options, master, emit }) {
         await emit("static.pre", { config, options, master });
         master.getServer().use(
-            "/front/(:role).js",
+            '/front/:"role".js',
             InitStatic.createStaticFrontServer("js", {
                 config,
                 options,
@@ -49,7 +49,7 @@ module.exports = class InitStatic {
             })
         );
         master.getServer().use(
-            "/front/(:role).css",
+            '/front/:"role".css',
             InitStatic.createStaticFrontServer("css", {
                 config,
                 options,

package/test/identity/providers/session.js

@@ -110,7 +110,7 @@ module.exports = ({ expect }) => {
                 };
                 const id = new mongoose.Types.ObjectId();
                 new Provider(t).setUserId(id);
-                expect(t.session.user).to.eql(id);
+                expect(t.session.user).to.eql(id.toString());
             });
 
             it("session not exist, set _id", function () {

package/tmpl/files/app/config/common.ejs

@@ -14,8 +14,6 @@
       "mongoose": {
         "uri": "mongodb://<%- db.mongoose.hostname %>/<%- db.mongoose.db %>?authSource=<%- db.mongoose.authSource %>",
         "options": {
-          "useNewUrlParser": true,
-          "useUnifiedTopology": true,
           "db": "<%- db.mongoose.db %>",
           "host": "<%- db.mongoose.hostname %>",          
           "autoIndex": false