nodester 0.4.9 → 0.5.0

package/lib/application/index.js

@@ -478,17 +478,26 @@ module.exports = class NodesterApplication extends Emitter {
 	/**
 	 * Stops server
 	 *
+	 * @param {Function} callback
+	 *
 	 * @access public
 	 */
-	stop() {
+	stop(callback) {
 		if (this._isListening !== true) {
-			console.warn('Nothing to stop. Server is not listening.');
+			consl.warn('nothing to stop. Server is not listening.');
 			return;
 		}
 
-		this.server.close();
-		this._isListening = false;
+		this.server.close(() => {
+			this._isListening = false;
+			this._router.unlock();
 
-		this._router.unlock();
+			if (typeof callback === 'function') {
+				callback();
+			}
+			else if (!!callback) {
+				consl.warn(`argument 'callback' in stop() must be of type: function.`);
+			}
+		});
 	}
 }

package/lib/http/request/index.js

@@ -8,11 +8,8 @@
 const { IncomingMessage } = require('http');
 
 const isIP = require('net').isIP;
-const typeis = require('type-is');
 const fresh = require('fresh');
-const parseRange = require('range-parser');
 const parse = require('parseurl');
-const proxyaddr = require('proxy-addr');
 
 const { defineGetter } = require('./utils');
 

package/lib/loggers/console.js

@@ -7,7 +7,8 @@
 
 
 exports = module.exports = {
-	error: _error
+	error: _error,
+	warn: _warn
 }
 
 /**
@@ -19,5 +20,19 @@ exports = module.exports = {
  * @access public
  */
 function _error(err) {
-  console.error(err.stack || err.toString());
+	console.error(err.stack || err.toString());
+}
+
+
+/**
+ * Log warning with nodester prefix using console.warning.
+ *
+ * @param {Error} err
+ *
+ * @alias error
+ * @access public
+ */
+function _warn(...args) {
+	const prefix = '[nodester] warning:';
+	console.warn(prefix, ...args);
 }

package/lib/structures/Filter.js

@@ -11,6 +11,9 @@ const CLAUSES = require('../constants/Clauses');
 const { isModel } = require('../utils/models');
 const { ensure } = require('nodester/validators/arguments');
 
+// Console:
+const consl = require('nodester/loggers/console');
+
 
 /**
  * @class
@@ -107,12 +110,13 @@ module.exports = class NodesterFilter {
 					// Empty bounds:
 					if (!!includeFilter.statics.clauses.limit) {
 						const msg = [
-							`(nodester) warning: include "${ includeName }" has`,
-							`association type of "${ associationType }", but has a filter clause "limit",`,
+							`include "${ includeName }" has association type of`,
+							`"${ associationType }", but has a filter clause "limit",`,
 							`which is forbidden on any association type except for "HasMany".`,
-							`It was automatically removed from clauses.`
+							`It was automatically removed from clauses.`,
+							`Consider also removing it from your code.`
 						].join(' ');
-						console.warn(msg);
+						consl.warn(msg);
 					}
 					delete includeFilter.statics.clauses.limit;
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodester",
-  "version": "0.4.9",
+  "version": "0.5.0",
   "description": "A versatile REST framework for Node.js",
   "directories": {
     "docs": "docs",
@@ -79,7 +79,7 @@
     "@js-temporal/polyfill": "^0.4.3",
     "body-parser": "^1.20.2",
     "common-js-file-extensions": "^1.0.4",
-    "cookie": "^0.5.0",
+    "cookie": "^1.0.0",
     "cookie-signature": "^1.2.0",
     "debug": "^4.3.4",
     "etag": "^1.8.1",
@@ -92,17 +92,13 @@
     "mysql2": "^3.6.0",
     "pg": "^8.11.3",
     "pg-hstore": "^2.3.4",
-    "proxy-addr": "^2.0.7",
     "qs": "^6.11.0",
-    "range-parser": "^1.2.1",
-    "sequelize": "^6.6.5",
-    "type-is": "^1.6.18",
-    "vary": "^1.1.2"
+    "sequelize": "^6.6.5"
   },
   "devDependencies": {
     "eslint": "^8.55.0",
     "eslint-plugin-jsdoc": "^46.9.0",
-    "jest": "^29.4.2"
+    "jest": "^29.7.0"
   },
   "engines": {
     "node": ">= 12.17.0"

package/tests/index.test.js

@@ -31,10 +31,10 @@ describe('nodester application', () => {
 			expect(app.isListening).toBe(true);
 			expect(app.middlewaresStack.length).toBe(4);
 
-			app.stop();
-
-			expect(app.isLocked).toBe(false);
-			expect(app.isListening).toBe(false);
+			app.stop(() => {
+				expect(app.isLocked).toBe(false);
+				expect(app.isListening).toBe(false);
+			});
 		});
 	});
 });