fix: Add version check in docker images

kuzzleio · rolljee · Jan 8, 2025 · Jan 7, 2025 · Jan 7, 2025 · Jan 7, 2025
commit 32a2efd896d4f5cf3d010cc7710ca8d523389ef3
diff --git a/docker/images/kuzzle/Dockerfile b/docker/images/kuzzle/Dockerfile
@@ -32,6 +32,7 @@
 ADD ./package-lock.json package-lock.json
 ADD ./index.ts index.ts
 ADD ./tsconfig.json tsconfig.json
+ADD ./check-node-version.js check-node-version.js
 
 RUN npm ci
 RUN npm run build

diff --git a/docker/images/plugin-dev/Dockerfile b/docker/images/plugin-dev/Dockerfile
@@ -20,6 +20,7 @@
 ADD ./package-lock.json package-lock.json
 ADD ./index.ts index.ts
 ADD ./tsconfig.json tsconfig.json
+ADD ./check-node-version.js check-node-version.js
 
 RUN npm ci
 RUN npm run build

diff --git a/package.json b/package.json
@@ -5,7 +5,7 @@
   "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": {
-    "preinstall": "node ./check-node-version",
+    "preinstall": "node ./check-node-version.js",
     "build": "tsc",
     "clean": "touch index.ts && npm run build | grep TSFILE | cut -d' ' -f 2 | xargs rm",
     "cucumber": "cucumber.js --fail-fast",

diff --git a/bin/.upgrades/lib/context.js b/bin/.upgrades/lib/context.js

    if (this.config.configs) {
      this.log.ok("Configuration files loaded:");
      this.config.configs.forEach((f) => this.log.print(`\t- ${f}`));
    }

    this.version = await this.getVersions();
diff --git a/features/fixtures/mappings.js b/features/fixtures/mappings.js
      properties: {
        job: { type: "keyword" },
        name: { type: "keyword" },
        age: { type: "integer" },
        city: { type: "keyword" },
      },
    },
  },
  "mtp-open-data": {
    "green-taxi": {
      properties: {
        job: { type: "keyword" },
        name: { type: "keyword" },
        age: { type: "integer" },
        city: { type: "keyword" },
      },
    },
diff --git a/features/fixtures/permissions.js b/features/fixtures/permissions.js
      credentials: {
        local: {
          username: "test-admin",
          password: "password",
        },
      },
    },
    "default-user": {
      content: {
        profileIds: ["default"],
      },
      credentials: {
        local: {
          username: "default-user",
          password: "password",
        },
      },
    },
diff --git a/features/step_definitions/auth-steps.js b/features/step_definitions/auth-steps.js
  async function (username, password) {
    this.props.result = await this.sdk.auth.login("local", {
      username,
      password,
    });
  },
 );
diff --git a/features/step_definitions/bulk-steps.js b/features/step_definitions/bulk-steps.js

    const request = {
      controller: "bulk",
      action: "deleteByQuery",
      index: this.props.index,
      collection: this.props.collection,
      refresh: "wait_for",
      body: { query },
    };
	properties: {
	job: { type: "keyword" },
	name: { type: "keyword" },
	age: { type: "integer" },
Check warning on line 10 in features/fixtures/mappings.js GitHub Actions / Lint - Node.js (18) `Expected object keys to be in ascending order. 'age' should be before 'name'` Check warning on line 10 in features/fixtures/mappings.js GitHub Actions / Lint - Node.js (20) `Expected object keys to be in ascending order. 'age' should be before 'name'` Check warning on line 10 in features/fixtures/mappings.js GitHub Actions / Lint - Node.js (22) `Expected object keys to be in ascending order. 'age' should be before 'name'`
	city: { type: "keyword" },
	},
	},
	},
	"mtp-open-data": {
Check warning on line 15 in features/fixtures/mappings.js GitHub Actions / Lint - Node.js (18) `Expected object keys to be in ascending order. 'mtp-open-data' should be before 'nyc-open-data'` Check warning on line 15 in features/fixtures/mappings.js GitHub Actions / Lint - Node.js (20) `Expected object keys to be in ascending order. 'mtp-open-data' should be before 'nyc-open-data'` Check warning on line 15 in features/fixtures/mappings.js GitHub Actions / Lint - Node.js (22) `Expected object keys to be in ascending order. 'mtp-open-data' should be before 'nyc-open-data'`
	"green-taxi": {
	properties: {
	job: { type: "keyword" },
	name: { type: "keyword" },
	age: { type: "integer" },
Check warning on line 20 in features/fixtures/mappings.js GitHub Actions / Lint - Node.js (18) `Expected object keys to be in ascending order. 'age' should be before 'name'` Check warning on line 20 in features/fixtures/mappings.js GitHub Actions / Lint - Node.js (20) `Expected object keys to be in ascending order. 'age' should be before 'name'` Check warning on line 20 in features/fixtures/mappings.js GitHub Actions / Lint - Node.js (22) `Expected object keys to be in ascending order. 'age' should be before 'name'`
	city: { type: "keyword" },
	},
	},
	credentials: {
	local: {
	username: "test-admin",
	password: "password",
Check warning on line 14 in features/fixtures/permissions.js GitHub Actions / Lint - Node.js (18) `Expected object keys to be in ascending order. 'password' should be before 'username'` Check warning on line 14 in features/fixtures/permissions.js GitHub Actions / Lint - Node.js (20) `Expected object keys to be in ascending order. 'password' should be before 'username'` Check warning on line 14 in features/fixtures/permissions.js GitHub Actions / Lint - Node.js (22) `Expected object keys to be in ascending order. 'password' should be before 'username'`
	},
	},
	},
	"default-user": {
Check warning on line 18 in features/fixtures/permissions.js GitHub Actions / Lint - Node.js (18) `Expected object keys to be in ascending order. 'default-user' should be before 'test-admin'` Check warning on line 18 in features/fixtures/permissions.js GitHub Actions / Lint - Node.js (20) `Expected object keys to be in ascending order. 'default-user' should be before 'test-admin'` Check warning on line 18 in features/fixtures/permissions.js GitHub Actions / Lint - Node.js (22) `Expected object keys to be in ascending order. 'default-user' should be before 'test-admin'`
	content: {
	profileIds: ["default"],
	},
	credentials: {
	local: {
	username: "default-user",
	password: "password",
Check warning on line 25 in features/fixtures/permissions.js GitHub Actions / Lint - Node.js (18) `Expected object keys to be in ascending order. 'password' should be before 'username'` Check warning on line 25 in features/fixtures/permissions.js GitHub Actions / Lint - Node.js (20) `Expected object keys to be in ascending order. 'password' should be before 'username'` Check warning on line 25 in features/fixtures/permissions.js GitHub Actions / Lint - Node.js (22) `Expected object keys to be in ascending order. 'password' should be before 'username'`
	},
	},
	},
	async function (username, password) {
	this.props.result = await this.sdk.auth.login("local", {
	username,
	password,
Check warning on line 12 in features/step_definitions/auth-steps.js GitHub Actions / Lint - Node.js (18) `Expected object keys to be in ascending order. 'password' should be before 'username'` Check warning on line 12 in features/step_definitions/auth-steps.js GitHub Actions / Lint - Node.js (20) `Expected object keys to be in ascending order. 'password' should be before 'username'` Check warning on line 12 in features/step_definitions/auth-steps.js GitHub Actions / Lint - Node.js (22) `Expected object keys to be in ascending order. 'password' should be before 'username'`
	});
	},
	);

	const request = {
	controller: "bulk",
	action: "deleteByQuery",
Check warning on line 48 in features/step_definitions/bulk-steps.js GitHub Actions / Lint - Node.js (18) `Expected object keys to be in ascending order. 'action' should be before 'controller'` Check warning on line 48 in features/step_definitions/bulk-steps.js GitHub Actions / Lint - Node.js (20) `Expected object keys to be in ascending order. 'action' should be before 'controller'` Check warning on line 48 in features/step_definitions/bulk-steps.js GitHub Actions / Lint - Node.js (22) `Expected object keys to be in ascending order. 'action' should be before 'controller'`
	index: this.props.index,
	collection: this.props.collection,
Check warning on line 50 in features/step_definitions/bulk-steps.js GitHub Actions / Lint - Node.js (18) `Expected object keys to be in ascending order. 'collection' should be before 'index'` Check warning on line 50 in features/step_definitions/bulk-steps.js GitHub Actions / Lint - Node.js (20) `Expected object keys to be in ascending order. 'collection' should be before 'index'` Check warning on line 50 in features/step_definitions/bulk-steps.js GitHub Actions / Lint - Node.js (22) `Expected object keys to be in ascending order. 'collection' should be before 'index'`
	refresh: "wait_for",
	body: { query },
	};