kuzzle 2.19.6 → 2.19.8

package/lib/api/controllers/baseController.js

@@ -85,12 +85,6 @@ class NativeController extends BaseController {
       throw assertionError.get("invalid_type", "body.query", "object");
     }
 
-    try {
-      global.kuzzle.koncorde.validate(koncordeFilters);
-    } catch (error) {
-      throw assertionError.getFrom(error, "koncorde_dsl_error", error.message);
-    }
-
     try {
       return await this.ask("core:storage:public:translate", koncordeFilters);
     } catch (error) {

package/lib/config/default.config.js

@@ -290,6 +290,10 @@ const defaultConfig = {
                     },
                 },
             },
+            defaultSettings: {
+                number_of_replicas: 1,
+                number_of_shards: 1,
+            },
             internalIndex: {
                 name: "kuzzle",
                 collections: {

package/lib/core/shared/sdk/embeddedSdk.d.ts

@@ -69,7 +69,7 @@ export declare class EmbeddedSDK extends Kuzzle {
      * @param request - API request (https://docs.kuzzle.io/core/2/guides/main-concepts/1-api#other-protocols)
      * @param options - Optional arguments
      */
-    query<TRequest extends BaseRequest, TResult>(request: TRequest, options?: {
+    query<TRequest extends BaseRequest = BaseRequest, TResult extends JSONObject = JSONObject>(request: TRequest, options?: {
         propagate?: boolean;
     }): Promise<ResponsePayload<TResult>>;
 }

package/lib/kerror/index.js

@@ -169,7 +169,7 @@ function rawGetFrom(domains, source, domain, subdomain, error, ...placeholders)
     const derivedError = rawGet(domains, domain, subdomain, error, ...placeholders);
     // If a stacktrace is present, we need to modify the first line because it
     // still contains the original error message
-    if (derivedError.stack && derivedError.stack.length) {
+    if (derivedError.stack && derivedError.stack.length && source.stack) {
         const stackArray = source.stack.split("\n");
         stackArray.shift();
         derivedError.stack = [

package/lib/service/storage/elasticsearch.js

@@ -1395,6 +1395,14 @@ class ElasticSearch extends Service {
       ),
     };
 
+    esRequest.body.settings.number_of_replicas =
+      esRequest.body.settings.number_of_replicas ||
+      this.config.defaultSettings.number_of_replicas;
+
+    esRequest.body.settings.number_of_shards =
+      esRequest.body.settings.number_of_shards ||
+      this.config.defaultSettings.number_of_shards;
+
     try {
       await this._client.indices.create(esRequest);
     } catch (error) {
@@ -3141,8 +3149,8 @@ class ElasticSearch extends Service {
             [this._getAlias(index, HIDDEN_COLLECTION)]: {},
           },
           settings: {
-            number_of_replicas: 1,
-            number_of_shards: 1,
+            number_of_replicas: this.config.defaultSettings.number_of_replicas,
+            number_of_shards: this.config.defaultSettings.number_of_shards,
           },
         },
         index: await this._getAvailableIndice(index, HIDDEN_COLLECTION),

package/lib/service/storage/esWrapper.js

@@ -232,6 +232,10 @@ class ESWrapper {
   _handleNotFoundError(error, message) {
     let errorMessage = message;
 
+    if (!error.body._index) {
+      return kerror.get("unexpected_not_found", errorMessage);
+    }
+
     // _index= "&nyc-open-data.yellow-taxi"
     const index = error.body._index.split(".")[0].slice(1);
     const collection = error.body._index.split(".")[1];

package/lib/service/storage/queryTranslator.js

@@ -84,7 +84,6 @@ const KONCORDE_CLAUSES_TO_ES = {
       ...content,
     },
   }),
-  regexp: undefined,
 };
 
 const KONCORDE_OPERATORS_TO_ES = {
@@ -145,7 +144,9 @@ class QueryTranslator {
     const converter = KONCORDE_CLAUSES_TO_ES[clause];
 
     if (converter === undefined) {
-      throw new KeywordError("clause", clause);
+      return {
+        [clause]: content,
+      };
     }
 
     return converter(content);

package/lib/types/config/storageEngine/StorageEngineElasticsearchConfiguration.d.ts

@@ -95,6 +95,24 @@ export declare type StorageEngineElasticsearch = {
             };
         };
     };
+    /**
+     * Global settings to apply by default (if not overridden by user request ones)
+     * @default
+     * {
+     *   number_of_shards: 1,
+     *   number_of_replicas: 1
+     * }
+     */
+    defaultSettings: {
+        /**
+         * @default 1
+         */
+        number_of_shards: number;
+        /**
+         * @default 1
+         */
+        number_of_replicas: number;
+    };
     internalIndex: {
         /**
          * @default "kuzzle"

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "kuzzle",
   "author": "The Kuzzle Team <support@kuzzle.io>",
-  "version": "2.19.6",
+  "version": "2.19.8",
   "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": {
@@ -26,6 +26,7 @@
     "test:functional:legacy:mqtt": "npx cucumber-js --format progress-bar --profile mqtt ./features-legacy",
     "cucumber": "cucumber.js --fail-fast",
     "codecov": "codecov",
+    "prettier": "npx prettier ./lib ./test ./bin ./features ./plugins/available/functional-test-plugin --write",
     "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",