not-node 6.3.73 → 6.3.75

package/index.js

@@ -1,8 +1,12 @@
 /**
  * @module not-node
  */
-
+/** Environment variables */
 module.exports.Env = require("./src/env.js");
+/** getter/setter shortcuts */
+module.exports.getEnv = module.exports.Env.get;
+module.exports.setEnv = module.exports.Env.set;
+
 /** Error module
  * @type {not-node/Error}
  */

package/package.json

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

package/src/cluster/cluster.redis.js

@@ -49,7 +49,7 @@ module.exports = class notClusterRedisProvider {
             if (this.#clientGetter) {
                 return this.#clientGetter();
             } else {
-                return notEnv.getEnv(this.#clientName);
+                return notEnv.get(this.#clientName);
             }
         } catch (e) {
             error(e);

package/src/domain.js

@@ -466,7 +466,7 @@ class notDomain extends EventEmitter {
     }
 
     getEnv(key) {
-        return Env.getEnv(key);
+        return Env.get(key);
     }
 
     /**
@@ -476,7 +476,7 @@ class notDomain extends EventEmitter {
      *  @return {notDomain}      chainable
      */
     static setEnv(key, val) {
-        Env.setEnv(key, val);
+        Env.set(key, val);
         return this;
     }
 

package/src/env.js

@@ -5,12 +5,26 @@ const ENVS = {
 };
 
 module.exports = class notEnv {
+    static LOG_STRING_ASSIGNMENTS = true;
     /**
-     *  Returns application environment variable
+     * Obsolete!
+     * Wrapper for 'get'
+     * @static
      *  @param   {string}      key  name of var
      *  @return {object|undefined}    value or undefined
      */
     static getEnv(key) {
+        console.error("Obsolete: use Env.get instead!");
+        return this.get(key);
+    }
+
+    /**
+     *  Returns application environment variable
+     * @static
+     *  @param   {string}      key  name of var
+     *  @return {object|undefined}    value or undefined
+     */
+    static get(key) {
         if (objHas(ENVS, key)) {
             return ENVS[key];
         } else {
@@ -19,12 +33,29 @@ module.exports = class notEnv {
     }
 
     /**
-     *  Setting application environment variable
+     * Obsolete!
+     * Wrapper for 'set'
+     * @static
      *  @param   {string}      key  name of var
      *  @param   {object}      val  value
      *  @return     {notEnv}      chainable
      */
     static setEnv(key, val) {
+        console.error("Obsolete: use Env.set instead!");
+        return this.set(key, val);
+    }
+
+    /**
+     *  Setting application environment variable
+     * @static
+     *  @param   {string}      key  name of var
+     *  @param   {object}      val  value
+     *  @return     {notEnv}      chainable
+     */
+    static set(key, val) {
+        if (notEnv.LOG_STRING_ASSIGNMENTS && typeof val === "string") {
+            console.log("ENV SET ", key, " = ", val);
+        }
         ENVS[key] = val;
         return notEnv;
     }

package/src/init/index.js

@@ -76,11 +76,11 @@ class Init {
     }
 
     static setEnv(key, val) {
-        Env.setEnv(key, val);
+        Env.set(key, val);
     }
 
     static getEnv(key) {
-        return Env.getEnv(key);
+        return Env.get(key);
     }
 
     static getApp() {

package/src/init/lib/app.js

@@ -3,7 +3,7 @@ const ENV = process.env.NODE_ENV || "development";
 const path = require("path");
 const logger = require("not-log");
 const log = logger(module, "not-node//init//app");
-const { notErrorReporter } = require("not-error");
+const { notErrorReporter } = require("not-error/src/index.cjs");
 
 const CONST_CORE_PATH = path.join(__dirname, "../../core");
 
@@ -21,23 +21,7 @@ module.exports = class InitApp {
 
     static async setAppEnvs({ config, options, master, emit }) {
         await emit("app.setEnv.pre", { config, options, master });
-        master.setEnv("validationEnv", options.validationEnv);
-        master.setEnv("hostname", config.get("hostname"));
-        master.setEnv("server", `https://` + config.get("host"));
-        master.setEnv(
-            "appPath",
-            master.getAbsolutePath(config.get("path.app"))
-        );
-        master.setEnv(
-            "dbDumpsPath",
-            master.getAbsolutePath(config.get("path.dbDumps"))
-        );
-        master.setEnv(
-            "tmpPath",
-            master.getAbsolutePath(config.get("path.tmp"))
-        );
         master.setEnv("name", master.getManifest().name);
-        master.setEnv("fullServerName", config.get("fullServerName"));
         master.setEnv(
             "rolesPriority",
             master.getManifest().targets.server.roles

package/src/init/lib/db/index.js

@@ -28,7 +28,8 @@ module.exports = class InitDB {
      *  [db_driver_name_1]: options,
      *  [db_driver_name_2]: options,
      * }
-     * to get driver require('not-node').getEnv(db_driver_name_2)
+     * to get driver
+     * require('not-node').getEnv(db_driver_name_2)
      * require('not-node').getEnv('mongoose')
      * require('not-node').getEnv('redis')
      **/

package/src/init/lib/env.js

@@ -71,7 +71,7 @@ module.exports = class InitENV {
     }
 
     static addToEnv(key, val) {
-        notEnv.setEnv(key, val);
+        notEnv.set(key, val);
         return val;
     }
 
@@ -79,7 +79,7 @@ module.exports = class InitENV {
         config.set(
             to,
             InitENV.addToEnv(to, master.getAbsolutePath(config.get(from, def))),
-            `obsolete: use notEnv.getEnv('${to}') instead`
+            `obsolete: use notEnv.get('${to}') instead`
         );
     }
 
@@ -97,18 +97,22 @@ module.exports = class InitENV {
         log?.info("Setting up server environment variables...");
         await emit("env.pre", { config, options, master });
 
+        master.setEnv("validationEnv", options.validationEnv);
+        master.setEnv("hostname", config.get("hostname"));
+        master.setEnv("server", `https://` + config.get("host"));
+
         InitENV.initFromTemplate({ config, master });
 
         config.set(
             "appPath",
             InitENV.addToEnv("appPath", options.pathToApp),
-            "obsolete: use notEnv.getEnv('appPath')"
+            "obsolete: use notEnv.get('appPath')"
         );
 
         config.set(
             "npmPath",
             InitENV.addToEnv("npmPath", options.pathToNPM),
-            "obsolete: use notEnv.getEnv('npmPath')"
+            "obsolete: use notEnv.get('npmPath')"
         );
 
         config.set(
@@ -117,7 +121,7 @@ module.exports = class InitENV {
                 "fullServerName",
                 InitENV.getFullServerName(config)
             ),
-            "obsolete: use notEnv.getEnv('fullServerName')"
+            "obsolete: use notEnv.get('fullServerName')"
         );
 
         await InitENV.checkPaths(master, config);

package/src/init/lib/rateLimiter.js

@@ -71,7 +71,7 @@ module.exports = class InitRateLimiter {
     }
 
     static getClient(storeClient) {
-        return notEnv.getEnv(`db.${storeClient}`);
+        return notEnv.get(`db.${storeClient}`);
     }
 
     static initCustom(

package/src/init/lib/routes.js

@@ -2,7 +2,11 @@
 
 const serveStatic = require("serve-static");
 const log = require("not-log")(module, "not-node//init//routes");
-const { notError, notValidationError, notRequestError } = require("not-error");
+const {
+    notError,
+    notValidationError,
+    notRequestError,
+} = require("not-error/src/index.cjs");
 
 module.exports = class InitRoutes {
     static finalError({ master }) {
@@ -57,7 +61,7 @@ module.exports = class InitRoutes {
         };
     }
 
-    async run({ master, config, options }) {
+    async run({ master, options }) {
         log?.info("Setting up routes...");
         //pages rendering
         await master.getApp().execInModules("registerPagesRoutes", master);
@@ -65,7 +69,7 @@ module.exports = class InitRoutes {
         master.getApp().expose(master.getServer());
         //user defined pages
         require(options.routesPath)(master.getServer(), master.getApp());
-        master.getServer().use(serveStatic(config.get("staticPath")));
+        master.getServer().use(serveStatic(master.getEnv("staticPath")));
         master.getServer().use(options.indexRoute);
         master.getServer().use(
             InitRoutes.finalError({

package/src/model/buildValidator.js

@@ -11,7 +11,7 @@ function extractValidationEnvGetter(options) {
     ) {
         return options.validationEnv;
     } else {
-        const globalValidationEnvGetter = notEnv.getEnv("validationEnv");
+        const globalValidationEnvGetter = notEnv.get("validationEnv");
         if (globalValidationEnvGetter && isFunc(globalValidationEnvGetter)) {
             return globalValidationEnvGetter;
         } else {

package/test/init/env.js

@@ -78,6 +78,7 @@ module.exports = ({ expect }) => {
                     getAbsolutePath(str) {
                         return str + "_fake_absolute";
                     },
+                    setEnv() {},
                 };
                 const options = {
                     pathToApp: "pathToApp__fake",
@@ -108,6 +109,7 @@ module.exports = ({ expect }) => {
                     getAbsolutePath(str) {
                         return str + "_fake_absolute";
                     },
+                    setEnv() {},
                 };
                 const options = {
                     pathToApp: "pathToApp__fake",

package/test/init/routes.js

@@ -182,16 +182,14 @@ module.exports = ({ expect }) => {
                     getServer() {
                         return fakeServer;
                     },
+                    getEnv(str) {
+                        return str + "__fake";
+                    },
                     getApp() {
                         return fakeApp;
                     },
                 };
-                const config = {
-                    get(str) {
-                        return str + "__fake";
-                    },
-                };
-                await new InitRoutes().run({ master, config, options });
+                await new InitRoutes().run({ master, options });
             });
         });
     });