kuzzle 2.59.0 → 2.60.0-beta.2

package/dist/lib/model/security/role.js

@@ -35,13 +35,23 @@ var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (
 }) : function(o, v) {
     o["default"] = v;
 });
-var __importStar = (this && this.__importStar) || function (mod) {
-    if (mod && mod.__esModule) return mod;
-    var result = {};
-    if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
-    __setModuleDefault(result, mod);
-    return result;
-};
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.Role = void 0;
 const kerror = __importStar(require("../../kerror"));

package/dist/lib/model/security/user.js

@@ -35,13 +35,23 @@ var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (
 }) : function(o, v) {
     o["default"] = v;
 });
-var __importStar = (this && this.__importStar) || function (mod) {
-    if (mod && mod.__esModule) return mod;
-    var result = {};
-    if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
-    __setModuleDefault(result, mod);
-    return result;
-};
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
 var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };

package/dist/lib/model/storage/apiKey.d.ts

@@ -12,7 +12,7 @@ declare class ApiKey extends BaseModel {
     userId: string;
     description: string;
     expiresAt: number;
-    ttl: number | string;
+    ttl: number;
     token: string;
     fingerprint: string;
     constructor(_source: JSONObject, _id?: string | null);

package/dist/lib/model/storage/apiKey.js

@@ -35,13 +35,23 @@ var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (
 }) : function(o, v) {
     o["default"] = v;
 });
-var __importStar = (this && this.__importStar) || function (mod) {
-    if (mod && mod.__esModule) return mod;
-    var result = {};
-    if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
-    __setModuleDefault(result, mod);
-    return result;
-};
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
 var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };

package/dist/lib/service/cache/redis.js

@@ -35,13 +35,23 @@ var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (
 }) : function(o, v) {
     o["default"] = v;
 });
-var __importStar = (this && this.__importStar) || function (mod) {
-    if (mod && mod.__esModule) return mod;
-    var result = {};
-    if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
-    __setModuleDefault(result, mod);
-    return result;
-};
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
 var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
@@ -53,6 +63,23 @@ const service_1 = __importDefault(require("../service"));
 const kerrorLib = __importStar(require("../../kerror"));
 require("../../types/Global");
 const kerror = kerrorLib.wrap("services", "cache");
+/**
+ * ioredis 6 changed three connection defaults. Kuzzle 2 pins them back to
+ * what ioredis 5 used, so upgrading the library changes nothing on the wire
+ * or in how a lost connection is retried. Anything set in `.kuzzlerc` (or
+ * from code) still wins: these are only the fallbacks.
+ *
+ * - `protocol`: ioredis 6 speaks RESP3 by default; RESP2 is what Kuzzle has
+ *   always used, and what Redis servers older than 6 only understand.
+ * - `keepAlive`: ioredis 6 enables TCP keep-alive probes after 30s.
+ * - `retryStrategy`: ioredis 6 uses an exponential backoff capped at 5s,
+ *   plus jitter; ioredis 5 waited 50ms more per attempt, capped at 2s.
+ */
+const IOREDIS_V5_DEFAULTS = {
+    keepAlive: 0,
+    protocol: 2,
+    retryStrategy: (times) => Math.min(times * 50, 2000),
+};
 /**
  * @class Redis
  * @extends Service
@@ -106,7 +133,7 @@ class Redis extends service_1.default {
         const client = config.nodes
             ? this._buildClusterClient({
                 ...config.clusterOptions,
-                redisOptions: config.options,
+                redisOptions: { ...IOREDIS_V5_DEFAULTS, ...config.options },
             })
             : this._buildClient(config.options ?? {});
         this.client = client;
@@ -266,7 +293,11 @@ class Redis extends service_1.default {
         });
     }
     _buildClient(options) {
-        return new ioredis_1.default({ ...this._config.node, ...options });
+        return new ioredis_1.default({
+            ...IOREDIS_V5_DEFAULTS,
+            ...this._config.node,
+            ...options,
+        });
     }
     _buildClusterClient(options) {
         // The caller only reaches here when `nodes` is set; `?? []` is what the

package/dist/lib/service/service.js

@@ -35,13 +35,23 @@ var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (
 }) : function(o, v) {
     o["default"] = v;
 });
-var __importStar = (this && this.__importStar) || function (mod) {
-    if (mod && mod.__esModule) return mod;
-    var result = {};
-    if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
-    __setModuleDefault(result, mod);
-    return result;
-};
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
 var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };

package/dist/lib/service/storage/7/elasticsearch.d.ts

@@ -1,10 +1,11 @@
 import _ from "lodash";
 import type { RequestParams } from "sdk-es7";
 import { Client } from "sdk-es7";
-import type { TypeMapping } from "sdk-es7/api/types";
+import type { ElasticsearchVersionInfo, ShardStatistics, TypeMapping } from "sdk-es7/api/types";
 import type { AliasToTargets, CatAliasRecord, InfoResult, JSONObject, KImportError, KImportResult, KMExecuteResult, KRequestBody, KRequestParams } from "../../../types/storage/7/Elasticsearch";
 import type { ValidatedTarget } from "../../../types/Target";
 import { storeScopeEnum } from "../../../core/storage/storeScopeEnum";
+import type { StorageEngineElasticsearch } from "../../../types/config/storageEngine/StorageEngineElasticsearchConfiguration";
 import { QueryTranslator } from "../commons/queryTranslator";
 import ESWrapper from "./esWrapper";
 /**
@@ -22,16 +23,16 @@ export declare class ES7 {
     _scope: storeScopeEnum;
     _indexPrefix: string;
     _esWrapper: ESWrapper;
-    _esVersion: any;
+    _esVersion: ElasticsearchVersionInfo | null;
     _translator: QueryTranslator;
     searchBodyKeys: string[];
     scriptKeys: string[];
     scriptAllowedArgs: string[];
     maxScrollDuration: number;
     scrollTTL: number;
-    _config: any;
+    _config: StorageEngineElasticsearch;
     private readonly logger;
-    constructor(config: any, scope?: storeScopeEnum);
+    constructor(config: StorageEngineElasticsearch, scope?: storeScopeEnum);
     get scope(): storeScopeEnum;
     /**
      * Initializes the elasticsearch client
@@ -85,10 +86,12 @@ export declare class ES7 {
             _id: any;
             _score: any;
             _source: any;
-            collection: any;
+            collection: string | undefined;
             highlight: any;
-            index: any;
-            inner_hits: JSONObject | undefined;
+            index: string | undefined;
+            inner_hits: {
+                [k: string]: unknown[];
+            } | undefined;
         }[];
         remaining: any;
         scrollId: any;
@@ -109,7 +112,7 @@ export declare class ES7 {
         index?: string;
         collection?: string;
         searchBody?: JSONObject;
-        targets?: any[];
+        targets?: ValidatedTarget[];
     }, { from, size, scroll, }?: {
         from?: number;
         size?: number;
@@ -120,10 +123,12 @@ export declare class ES7 {
             _id: any;
             _score: any;
             _source: any;
-            collection: any;
+            collection: string | undefined;
             highlight: any;
-            index: any;
-            inner_hits: JSONObject | undefined;
+            index: string | undefined;
+            inner_hits: {
+                [k: string]: unknown[];
+            } | undefined;
         }[];
         remaining: any;
         scrollId: any;
@@ -143,16 +148,22 @@ export declare class ES7 {
      * @returns
      */
     _mapTargetsToAlias(targets: ValidatedTarget[]): AliasToTargets;
-    _formatSearchResult(body: any, searchInfo?: any): Promise<{
+    _formatSearchResult(body: JSONObject, searchInfo?: {
+        collection?: string;
+        index?: string;
+        targets?: ValidatedTarget[];
+    }): Promise<{
         aggregations: any;
         hits: {
             _id: any;
             _score: any;
             _source: any;
-            collection: any;
+            collection: string | undefined;
             highlight: any;
-            index: any;
-            inner_hits: JSONObject | undefined;
+            index: string | undefined;
+            inner_hits: {
+                [k: string]: unknown[];
+            } | undefined;
         }[];
         remaining: any;
         scrollId: any;
@@ -355,7 +366,7 @@ export declare class ES7 {
      *
      * @returns {Promise.<{ _id, _version, _source }>}
      */
-    deleteFields(index: string, collection: string, id: string, fields: string, { refresh, userId, }?: {
+    deleteFields(index: string, collection: string, id: string, fields: string[], { refresh, userId, }?: {
         refresh?: boolean | "wait_for";
         userId?: string | null;
     }): Promise<{
@@ -410,10 +421,10 @@ export declare class ES7 {
      *
      * @returns {Promise.<any[]>} Array of results returned by the callback
      */
-    mExecute(index: string, collection: string, query: JSONObject, callback: any, { size, scrollTTl, }?: {
+    mExecute(index: string, collection: string, query: JSONObject, callback: (hits: JSONObject[]) => unknown, { size, scrollTTl, }?: {
         size?: number;
         scrollTTl?: string;
-    }): Promise<any>;
+    }): Promise<unknown[]>;
     /**
      * Creates a new index.
      *
@@ -439,7 +450,7 @@ export declare class ES7 {
      */
     createCollection(index: string, collection: string, { mappings, settings, }?: {
         mappings?: TypeMapping;
-        settings?: Record<string, any>;
+        settings?: JSONObject;
     }): Promise<null>;
     /**
      * Retrieves settings definition for index/type
@@ -478,7 +489,7 @@ export declare class ES7 {
     updateCollection(index: string, collection: string, { mappings, reindexCollection, settings, }?: {
         mappings?: TypeMapping;
         reindexCollection?: boolean;
-        settings?: Record<string, any>;
+        settings?: JSONObject;
     }): Promise<null>;
     /**
      * Given index settings we return a new version of index settings
@@ -603,7 +614,7 @@ export declare class ES7 {
      *
      * @returns {Promise.<String[]>}
      */
-    deleteIndexes(indexes?: string[]): Promise<unknown[]>;
+    deleteIndexes(indexes?: string[]): Promise<string[]>;
     /**
      * Deletes an index
      *
@@ -624,7 +635,7 @@ export declare class ES7 {
      * @returns {Promise.<Object>} { _shards }
      */
     refreshCollection(index: string, collection: string): Promise<{
-        _shards: any;
+        _shards: ShardStatistics;
     }>;
     /**
      * Returns true if the document exists
@@ -688,7 +699,7 @@ export declare class ES7 {
      *
      * @returns {Promise.<Object>} { items, errors }
      */
-    mCreate(index: string, collection: string, documents: JSON[], { refresh, timeout, userId, }?: {
+    mCreate(index: string, collection: string, documents: JSONObject[], { refresh, timeout, userId, }?: {
         refresh?: boolean | "wait_for";
         timeout?: string;
         userId?: string | null;
@@ -865,7 +876,7 @@ export declare class ES7 {
      * @return {Promise<*>} the settings of the indice.
      * @private
      */
-    _getSettings(esRequest: RequestParams.IndicesGetSettings): Promise<any>;
+    _getSettings(esRequest: RequestParams.IndicesGetSettings): Promise<JSONObject>;
     /**
      * Given index + collection, returns an available indice name.
      * Use this function when creating the associated indice. Otherwise use `_getAlias`.
@@ -960,7 +971,7 @@ export declare class ES7 {
      *
      * @returns {Promise.<Array>} resolve to an array of documents
      */
-    _getAllDocumentsFromQuery(esRequest: RequestParams.Search<Record<string, any>>): Promise<any>;
+    _getAllDocumentsFromQuery(esRequest: RequestParams.Search<JSONObject>): Promise<any>;
     /**
      * Clean and normalize the searchBody
      * Ensure only allowed parameters are passed to ES

package/dist/lib/service/storage/7/elasticsearch.js

@@ -35,13 +35,23 @@ var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (
 }) : function(o, v) {
     o["default"] = v;
 });
-var __importStar = (this && this.__importStar) || function (mod) {
-    if (mod && mod.__esModule) return mod;
-    var result = {};
-    if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
-    __setModuleDefault(result, mod);
-    return result;
-};
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
 var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
@@ -57,7 +67,7 @@ const storeScopeEnum_1 = require("../../../core/storage/storeScopeEnum");
 const kerror = __importStar(require("../../../kerror"));
 const didYouMean_1 = __importDefault(require("../../../util/didYouMean"));
 const extractFields_1 = __importDefault(require("../../../util/extractFields"));
-const mutex_1 = require("../../../util/mutex");
+const distributedLock_1 = require("../../../util/distributedLock");
 const name_generator_1 = require("../../../util/name-generator");
 const requestAssertions_1 = require("../../../util/requestAssertions");
 const safeObject_1 = require("../../../util/safeObject");
@@ -73,6 +83,12 @@ const ROOT_MAPPING_PROPERTIES = [
 const CHILD_MAPPING_PROPERTIES = ["type"];
 // Used for collection emulation
 const HIDDEN_COLLECTION = "_kuzzle_keep";
+// The hidden collection locks wait for as long as it takes, as the `Mutex`
+// they replace did: they guard a collection creation, which must not fail
+// only because another request on the same index got there first. The lock
+// is refreshed while held, so a live holder is never overtaken; a crashed
+// one leaves it for at most `ttl` (30 s).
+const HIDDEN_COLLECTION_LOCK = { retryAttempts: Number.POSITIVE_INFINITY };
 const ALIAS_PREFIX = "@"; // @todo next major release: Add ALIAS_PREFIX in FORBIDDEN_CHARS
 const PRIVATE_PREFIX = "%";
 const PUBLIC_PREFIX = "&";
@@ -184,7 +200,7 @@ class ES7 {
     info() {
         const result = {
             type: "elasticsearch",
-            version: this._esVersion,
+            version: this._esVersion?.number ?? "",
         };
         return this._client
             .info()
@@ -217,7 +233,6 @@ class ES7 {
         const indexes = {};
         let size = 0;
         for (const [indice, indiceInfo] of Object.entries(body.indices ?? {})) {
-            const infos = indiceInfo;
             // Ignore non-Kuzzle indices
             if (!indice.startsWith(PRIVATE_PREFIX) &&
                 !indice.startsWith(PUBLIC_PREFIX)) {
@@ -237,12 +252,12 @@ class ES7 {
                 size: 0,
             }));
             indexStats.collections.push({
-                documentCount: infos.total.docs.count,
+                documentCount: indiceInfo.total.docs.count,
                 name: collectionName,
-                size: infos.total.store.size_in_bytes,
+                size: indiceInfo.total.store.size_in_bytes,
             });
-            indexStats.size += infos.total.store.size_in_bytes;
-            size += infos.total.store.size_in_bytes;
+            indexStats.size += indiceInfo.total.store.size_in_bytes;
+            size += indiceInfo.total.store.size_in_bytes;
         }
         return {
             indexes: Object.values(indexes),
@@ -392,7 +407,9 @@ class ES7 {
         }
         return aliasToTargets;
     }
-    async _formatSearchResult(body, searchInfo = {}) {
+    async _formatSearchResult(
+    // The ES 7 client does not type its answers.
+    body, searchInfo = {}) {
         let aliasToTargets = {};
         const aliasCache = new Map();
         if (searchInfo.targets) {
@@ -446,11 +463,11 @@ class ES7 {
             if (!innerHits) {
                 return undefined;
             }
-            const formattedInnerHits = {};
-            for (const [name, innerHit] of Object.entries(innerHits)) {
-                formattedInnerHits[name] = await bluebird_1.default.map(innerHit.hits.hits, formatHit);
-            }
-            return formattedInnerHits;
+            const formatted = await Promise.all(Object.entries(innerHits).map(async ([name, innerHit]) => [
+                name,
+                await bluebird_1.default.map(innerHit.hits.hits, formatHit),
+            ]));
+            return Object.fromEntries(formatted);
         }
         const hits = await bluebird_1.default.map(body.hits.hits, async (hit) => ({
             inner_hits: await formatInnerHits(hit.inner_hits),
@@ -1114,19 +1131,6 @@ class ES7 {
         if (collection === HIDDEN_COLLECTION) {
             throw kerror.get("services", "storage", "collection_reserved", HIDDEN_COLLECTION);
         }
-        const mutex = new mutex_1.Mutex(`hiddenCollection/create/${index}`);
-        try {
-            await mutex.lock();
-            if (await this._hasHiddenCollection(index)) {
-                await this.deleteCollection(index, HIDDEN_COLLECTION);
-            }
-        }
-        catch (error) {
-            throw this._esWrapper.formatESError(error);
-        }
-        finally {
-            await mutex.unlock();
-        }
         const requestBody = {
             aliases: {
                 [this._getAlias(index, collection)]: {},
@@ -1168,6 +1172,19 @@ class ES7 {
             }
             throw this._esWrapper.formatESError(error);
         }
+        // Only once the real collection exists: dropping the placeholder first
+        // would leave the index with no indice at all if the creation failed.
+        // Lock and ES errors alike reach the caller through `formatESError`
+        try {
+            await (0, distributedLock_1.withLock)(`hiddenCollection/create/${index}`, async () => {
+                if (await this._hasHiddenCollection(index)) {
+                    await this.deleteCollection(index, HIDDEN_COLLECTION);
+                }
+            }, HIDDEN_COLLECTION_LOCK);
+        }
+        catch (error) {
+            throw this._esWrapper.formatESError(error);
+        }
         return null;
     }
     /**
@@ -1630,7 +1647,11 @@ class ES7 {
                     name: alias,
                 });
             }
-            await this._createHiddenCollection(index);
+            // `createCollection` drops the placeholder through here: recreating it
+            // would undo that.
+            if (collection !== HIDDEN_COLLECTION) {
+                await this._createHiddenCollection(index);
+            }
         }
         catch (e) {
             throw this._esWrapper.formatESError(e);
@@ -2635,33 +2656,30 @@ class ES7 {
      * @param {String} index Index name
      */
     async _createHiddenCollection(index) {
-        const mutex = new mutex_1.Mutex(`hiddenCollection/${index}`);
         try {
-            await mutex.lock();
-            if (await this._hasHiddenCollection(index)) {
-                return;
-            }
-            const esRequest = {
-                body: {
-                    aliases: {
-                        [this._getAlias(index, HIDDEN_COLLECTION)]: {},
-                    },
-                    settings: {
-                        number_of_replicas: this._config.defaultSettings.number_of_replicas,
-                        number_of_shards: this._config.defaultSettings.number_of_shards,
+            await (0, distributedLock_1.withLock)(`hiddenCollection/${index}`, async () => {
+                if (await this._hasHiddenCollection(index)) {
+                    return;
+                }
+                const esRequest = {
+                    body: {
+                        aliases: {
+                            [this._getAlias(index, HIDDEN_COLLECTION)]: {},
+                        },
+                        settings: {
+                            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),
-                wait_for_active_shards: await this._getWaitForActiveShards(),
-            };
-            await this._client.indices.create(esRequest);
+                    index: await this._getAvailableIndice(index, HIDDEN_COLLECTION),
+                    wait_for_active_shards: await this._getWaitForActiveShards(),
+                };
+                await this._client.indices.create(esRequest);
+            }, HIDDEN_COLLECTION_LOCK);
         }
         catch (e) {
             throw this._esWrapper.formatESError(e);
         }
-        finally {
-            await mutex.unlock();
-        }
     }
     /**
      * We need to always wait for a minimal number of shards to be available