kuzzle 2.19.2 → 2.19.3

package/lib/util/promback.js

@@ -18,12 +18,12 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-'use strict';
+"use strict";
 
-const Bluebird = require('bluebird');
+const Bluebird = require("bluebird");
 
 class Promback {
-  constructor (callback = null) {
+  constructor(callback = null) {
     this._callback = callback;
     this._resolve = null;
     this._reject = null;
@@ -38,29 +38,27 @@ class Promback {
     }
   }
 
-  resolve (result) {
+  resolve(result) {
     if (this.isPromise) {
       this._resolve(result);
-    }
-    else {
+    } else {
       this._callback(null, result);
     }
 
     return this.deferred;
   }
 
-  reject (error) {
+  reject(error) {
     if (this.isPromise) {
       this._reject(error);
-    }
-    else {
+    } else {
       this._callback(error);
     }
 
     return this.deferred;
   }
 
-  get promise () {
+  get promise() {
     return this.deferred;
   }
 }

package/lib/util/readYamlFile.d.ts

@@ -1,2 +1,2 @@
-import { JSONObject } from '../../index';
+import { JSONObject } from "../../index";
 export declare function readYamlFile(path: string): JSONObject;

package/lib/util/readYamlFile.js

@@ -4,7 +4,7 @@ exports.readYamlFile = void 0;
 const fs_1 = require("fs");
 const js_yaml_1 = require("js-yaml");
 function readYamlFile(path) {
-    return (0, js_yaml_1.load)((0, fs_1.readFileSync)(path, 'utf-8'));
+    return (0, js_yaml_1.load)((0, fs_1.readFileSync)(path, "utf-8"));
 }
 exports.readYamlFile = readYamlFile;
 //# sourceMappingURL=readYamlFile.js.map

package/lib/util/requestAssertions.js

@@ -19,13 +19,13 @@
  * limitations under the License.
  */
 
-'use strict';
+"use strict";
 
-const _ = require('lodash');
-const { get, isPlainObject } = require('./safeObject');
-const kerror = require('../kerror');
+const _ = require("lodash");
+const { get, isPlainObject } = require("./safeObject");
+const kerror = require("../kerror");
 
-const assertionError = kerror.wrap('api', 'assert');
+const assertionError = kerror.wrap("api", "assert");
 
 module.exports = {
   /**
@@ -33,7 +33,7 @@ module.exports = {
    */
   assertArgsHasAttribute: (request, attribute) => {
     if (_.isNil(get(request.input.args, attribute))) {
-      throw assertionError.get('missing_argument', attribute);
+      throw assertionError.get("missing_argument", attribute);
     }
   },
 
@@ -45,29 +45,29 @@ module.exports = {
    */
   assertBodyAttributeType: (request, attribute, type) => {
     switch (type) {
-      case 'number':
-      case 'boolean':
-      case 'string':
+      case "number":
+      case "boolean":
+      case "string":
         // The body should always be passed as JSON, we don't consider type conversion possibilities
         if (typeof get(request.input.body, attribute) === type) {
           return true;
         }
         break;
-      case 'array':
+      case "array":
         if (Array.isArray(get(request.input.body, attribute))) {
           return true;
         }
         break;
-      case 'object':
+      case "object":
         if (isPlainObject(get(request.input.body, attribute))) {
           return true;
         }
         break;
       default:
-        throw assertionError.get('unexpected_type_assertion', type, attribute);
+        throw assertionError.get("unexpected_type_assertion", type, attribute);
     }
 
-    throw assertionError.get('invalid_type', `body.${attribute}`, type);
+    throw assertionError.get("invalid_type", `body.${attribute}`, type);
   },
 
   /**
@@ -77,7 +77,7 @@ module.exports = {
    */
   assertBodyHasAttribute: (request, attribute) => {
     if (_.isNil(get(request.input.body, attribute))) {
-      throw assertionError.get('missing_argument', `body.${attribute}`);
+      throw assertionError.get("missing_argument", `body.${attribute}`);
     }
   },
 
@@ -86,8 +86,8 @@ module.exports = {
    * @param {string} attribute
    */
   assertBodyHasNotAttribute: (request, attribute) => {
-    if (! _.isNil(get(request.input.body, attribute))) {
-      throw assertionError.get('forbidden_argument', `body.${attribute}`);
+    if (!_.isNil(get(request.input.body, attribute))) {
+      throw assertionError.get("forbidden_argument", `body.${attribute}`);
     }
   },
 
@@ -98,40 +98,40 @@ module.exports = {
    * @param {string} attribute
    * @param {string} type
    */
-  assertHasBody: request => {
+  assertHasBody: (request) => {
     if (_.isNil(request.input.body)) {
-      throw assertionError.get('body_required');
+      throw assertionError.get("body_required");
     }
   },
 
   /**
    * @param {Request} request
    */
-  assertHasId: request => {
-    if (! request.input.args._id) {
-      throw assertionError.get('missing_argument', '_id');
+  assertHasId: (request) => {
+    if (!request.input.args._id) {
+      throw assertionError.get("missing_argument", "_id");
     }
   },
 
   /**
    * @param {Request} request
    */
-  assertHasIndex: request => {
-    if (! request.input.args.index) {
-      throw assertionError.get('missing_argument', 'index');
+  assertHasIndex: (request) => {
+    if (!request.input.args.index) {
+      throw assertionError.get("missing_argument", "index");
     }
   },
 
   /**
    * @param {Request} request
    */
-  assertHasIndexAndCollection: request => {
-    if (! request.input.args.index) {
-      throw assertionError.get('missing_argument', 'index');
+  assertHasIndexAndCollection: (request) => {
+    if (!request.input.args.index) {
+      throw assertionError.get("missing_argument", "index");
     }
 
-    if (! request.input.args.collection) {
-      throw assertionError.get('missing_argument', 'collection');
+    if (!request.input.args.collection) {
+      throw assertionError.get("missing_argument", "collection");
     }
   },
 
@@ -141,7 +141,7 @@ module.exports = {
    */
   assertIsAuthenticated: (anonymousId, request) => {
     if (request.context.user._id === anonymousId) {
-      throw kerror.get('security', 'rights', 'unauthorized');
+      throw kerror.get("security", "rights", "unauthorized");
     }
   },
 
@@ -149,9 +149,9 @@ module.exports = {
    * @param {Kuzzle} kuzzle
    * @param {Request} request
    */
-  assertIsObject: value => {
-    if (! isPlainObject(value)) {
-      throw assertionError.get('invalid_argument', value, 'object');
+  assertIsObject: (value) => {
+    if (!isPlainObject(value)) {
+      throw assertionError.get("invalid_argument", value, "object");
     }
-  }
+  },
 };

package/lib/util/safeObject.js

@@ -19,15 +19,15 @@
  * limitations under the License.
  */
 
-'use strict';
+"use strict";
 
 // Simple utility functions for safe (and fast) object manipulations
 
-function has (o, prop) {
+function has(o, prop) {
   return Object.prototype.hasOwnProperty.call(o, prop);
 }
 
-function get (o, prop) {
+function get(o, prop) {
   if (has(o, prop)) {
     return o[prop];
   }
@@ -35,8 +35,8 @@ function get (o, prop) {
   return undefined;
 }
 
-function isPlainObject (o) {
-  return Object.prototype.toString.call(o) === '[object Object]';
+function isPlainObject(o) {
+  return Object.prototype.toString.call(o) === "[object Object]";
 }
 
 module.exports = { get, has, isPlainObject };

package/lib/util/stackTrace.js

@@ -19,12 +19,12 @@
  * limitations under the License.
  */
 
-'use strict';
+"use strict";
 
-const util = require('util');
+const util = require("util");
 
-const MARKER = '>';
-const PADDING = '  ';
+const MARKER = ">";
+const PADDING = "  ";
 
 /**
  * Hilight user code
@@ -36,17 +36,18 @@ const PADDING = '  ';
  * >>>>  at init (/home/aschen/projets/app/test.ts:8:3)
  *       at Module._compile (internal/modules/cjs/loader.js:1133:30)
  */
-function hilightUserCode (line) {
+function hilightUserCode(line) {
   // ignore first line (error message) or already enhanced
-  if (! line.includes(' at ') || line.startsWith(MARKER)) {
+  if (!line.includes(" at ") || line.startsWith(MARKER)) {
     return line;
   }
 
-  const isKuzzleCode = line.includes('kuzzle/lib/');
-  const isNodeCode = ! line.includes('at /')
-                  && ! line.includes('at async /')
-                  && line.charAt(line.indexOf('(') + 1) !== '/';
-  const isModuleCode = line.includes('node_modules');
+  const isKuzzleCode = line.includes("kuzzle/lib/");
+  const isNodeCode =
+    !line.includes("at /") &&
+    !line.includes("at async /") &&
+    line.charAt(line.indexOf("(") + 1) !== "/";
+  const isModuleCode = line.includes("node_modules");
   if (isKuzzleCode || isNodeCode || isModuleCode) {
     return PADDING + line;
   }
@@ -62,23 +63,20 @@ function hilightUserCode (line) {
  *                               request response
  * @returns {*} return the data minus the stack trace
  */
-function removeStacktrace (data) {
+function removeStacktrace(data) {
   if (util.types.isNativeError(data)) {
-    if (global.NODE_ENV !== 'development') {
+    if (global.NODE_ENV !== "development") {
       data.stack = undefined;
+    } else {
+      data.stack = data.stack.split("\n").map(hilightUserCode).join("\n");
     }
-    else {
-      data.stack = data.stack.split('\n').map(hilightUserCode).join('\n');
-    }
-  }
-  else if (data && data.content && data.content.error) {
+  } else if (data && data.content && data.content.error) {
     // @todo v3: stack should be removed only for "production" env
-    if (global.NODE_ENV !== 'development') {
+    if (global.NODE_ENV !== "development") {
       data.content.error.stack = undefined;
-    }
-    else {
+    } else {
       data.content.error.stack = data.content.error.stack
-        ? data.content.error.stack.split('\n').map(hilightUserCode).join('\n')
+        ? data.content.error.stack.split("\n").map(hilightUserCode).join("\n")
         : undefined;
     }
   }

package/lib/util/wildcard.js

@@ -19,37 +19,37 @@
  * limitations under the License.
  */
 
-'use strict';
+"use strict";
 
-const _ = require('lodash');
+const _ = require("lodash");
 
-function match (pattern, list) {
+function match(pattern, list) {
   // Match everything
-  if (pattern === '*') {
+  if (pattern === "*") {
     return list;
   }
   /**
    * Reduces repeating "*" to one ".*"
    * One of the fastest and most readable way to do this
    */
-  const wildCardPattern = pattern.split('*')
-    .filter((patternPart, index, array) => 
-      patternPart !== ''
-      || index === 0
-      || index === array.length - 1
+  const wildCardPattern = pattern
+    .split("*")
+    .filter(
+      (patternPart, index, array) =>
+        patternPart !== "" || index === 0 || index === array.length - 1
     )
-    .map(patternPart => _.escapeRegExp(patternPart)) // escape special regex characters
-    .join('.*');
-  
+    .map((patternPart) => _.escapeRegExp(patternPart)) // escape special regex characters
+    .join(".*");
+
   // Match everything
-  if (wildCardPattern === '.*') {
+  if (wildCardPattern === ".*") {
     return list;
   }
-  
+
   const regex = new RegExp(`^${wildCardPattern}$`);
 
   // Keep only matching elements
-  return list.filter(item => ! regex.test(item));
+  return list.filter((item) => !regex.test(item));
 }
 
 module.exports = { match };

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "kuzzle",
   "author": "The Kuzzle Team <support@kuzzle.io>",
-  "version": "2.19.2",
+  "version": "2.19.3",
   "description": "Kuzzle is an open-source solution that handles all the data management through a secured API, with a large choice of protocols.",
   "bin": "bin/start-kuzzle-server",
   "scripts": {
@@ -29,8 +29,8 @@
     "test:lint": "npm run test:lint:js && npm run test:lint:ts",
     "test:lint:ts": "eslint --max-warnings=0 ./lib --ext .ts --config .eslintc-ts.json",
     "test:lint:ts:fix": "eslint --max-warnings=0 --fix ./lib --ext .ts --config .eslintc-ts.json",
-    "test:lint:js": "eslint --max-warnings=0 ./lib ./test ./features-legacy ./bin ./features ./doc/build-error-codes.js ./plugins/available/functional-test-plugin",
-    "test:lint:js:fix": "eslint --max-warnings=0 --fix ./lib ./test ./features-legacy ./bin ./features ./doc/build-error-codes.js",
+    "test:lint:js": "eslint --max-warnings=0 ./lib ./test ./bin ./features ./plugins/available/functional-test-plugin",
+    "test:lint:js:fix": "eslint --max-warnings=0 --fix ./lib ./test ./bin ./features",
     "doc-error-codes": "node -r ts-node/register doc/build-error-codes",
     "docker:install": "docker-compose run kuzzle_node_1 npm install",
     "docker:test:unit": "docker-compose run kuzzle_node_1 npm run test:unit",
@@ -71,12 +71,13 @@
     "ndjson": "^2.0.0",
     "node-segfault-handler": "^1.0.4",
     "passport": "^0.5.2",
+    "prettier": "^2.7.1",
     "protobufjs": "~6.11.2",
     "rc": "1.2.8",
     "semver": "^7.3.7",
     "sorted-array": "^2.0.4",
-    "uuid": "^8.3.2",
     "uWebSockets.js": "https://github.com/uNetworking/uWebSockets.js/archive/refs/tags/v20.0.0.tar.gz",
+    "uuid": "^8.3.2",
     "validator": "^13.7.0",
     "winston": "^3.7.2",
     "winston-elasticsearch": "0.16.1",
@@ -99,7 +100,7 @@
     "cucumber": "^6.0.5",
     "ergol": "^1.0.2",
     "eslint": "^8.13.0",
-    "eslint-plugin-no-only-tests": "^2.6.0",
+    "eslint-plugin-kuzzle": "0.0.1",
     "mocha": "^9.2.2",
     "mock-require": "^3.0.3",
     "mqtt": "^4.3.7",
@@ -126,7 +127,6 @@
     "lib/**/*.proto",
     "lib/**/*.yaml",
     "package.json",
-    "package-lock.json",
     "index.js",
     "index.d.ts",
     "LICENSE.md",