Update node version to 22 in tests.yaml and package.json and update dependencies

.github/workflows/tests.yaml

@@ -15,7 +15,7 @@ jobs:
 
       - uses: actions/setup-node@v4
         with:
-          node-version: '16'
+          node-version: '22'
           cache: yarn
 
       - name: install dependencies

eslint.config.mjs

@@ -0,0 +1,13 @@
+import path from "node:path";
+import { fileURLToPath } from "node:url";
+import js from "@eslint/js";
+import { FlatCompat } from "@eslint/eslintrc";
+
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = path.dirname(__filename);
+const compat = new FlatCompat({
+    baseDirectory: __dirname,
+    recommendedConfig: js.configs.recommended,
+    allConfig: js.configs.all
+});
+export default [...compat.extends("scality")];

lib/RESTClient.js

@@ -1,4 +1,4 @@
-'use strict'; // eslint-disable-line strict
+'use strict';
 
 const assert = require('assert');
 const http = require('http');
@@ -351,9 +351,9 @@ class RESTClient {
                     ret.push(data);
                     retLen += data.length;
                 }).on('error', callback)
-                  .on('end', () => {
-                      callback(null, Buffer.concat(ret, retLen).toString());
-                  });
+                    .on('end', () => {
+                        callback(null, Buffer.concat(ret, retLen).toString());
+                    });
                 return undefined;
             });
     }
@@ -421,11 +421,11 @@ class RESTClient {
         }
 
         req.on('response', res => this.endRespond(res, res, log, callback))
-           .on('error', error => {
-               // covers system errors like ECONNREFUSED, ECONNRESET etc.
-               log.error('error sending request to metadata', { error });
-               callback(errors.InternalError);
-           }).end();
+            .on('error', error => {
+                // covers system errors like ECONNREFUSED, ECONNRESET etc.
+                log.error('error sending request to metadata', { error });
+                callback(errors.InternalError);
+            }).end();
     }
 
     /**

lib/shell.js

@@ -1,4 +1,4 @@
-'use strict'; // eslint-disable-line strict
+'use strict';
 
 const readline = require('readline');
 

package.json

@@ -1,18 +1,18 @@
 {
   "name": "bucketclient",
   "engines": {
-    "node": ">=16"
+    "node": ">=20"
   },
-  "version": "8.1.10",
+  "version": "8.1.11",
   "description": "Bucket Client",
   "main": "index.js",
   "files": [
-      "CONTRIBUTING.md",
-      "config.json",
-      "index.d.ts",
-      "lib/*",
-      "licenses/*",
-      "yarn.lock"
+    "CONTRIBUTING.md",
+    "config.json",
+    "index.d.ts",
+    "lib/*",
+    "licenses/*",
+    "yarn.lock"
   ],
   "scripts": {
     "ft_test": "mocha tests/functional",
@@ -34,15 +34,16 @@
   },
   "homepage": "https://github.com/scality/bucketclient#readme",
   "devDependencies": {
-    "eslint": "^2.4.0",
-    "eslint-config-airbnb": "^6.0.0",
-    "eslint-config-scality": "scality/Guidelines#ec33dfb",
-    "eslint-plugin-react": "4.2.3",
-    "mocha": ""
+    "eslint": "^9.14.0",
+    "eslint-config-airbnb": "^19.0.4",
+    "eslint-config-scality": "scality/Guidelines#8.3.0",
+    "eslint-plugin-import": "^2.31.0",
+    "eslint-plugin-react": "7.37.2",
+    "mocha": "10.8.2"
   },
   "dependencies": {
-    "arsenal": "git+https://github.com/scality/Arsenal#8.1.87",
-    "httpagent": "git+https://github.com/scality/httpagent#1.0.5",
-    "werelogs": "scality/werelogs#8.1.2"
+    "arsenal": "git+https://github.com/scality/Arsenal#7e142a1bfca7f516c107d2072baeae54cbbe379c",
+    "httpagent": "git+https://github.com/scality/httpagent#1.0.6",
+    "werelogs": "scality/werelogs#8.1.5"
   }
 }

tests/functional/client.js

@@ -1,4 +1,4 @@
-'use strict'; // eslint-disable-line strict
+'use strict';  
 
 const assert = require('assert');
 
@@ -18,7 +18,7 @@ describe('Bucket Client tests', function testClient() {
 
     it('should create a new bucket', done => {
         client.createBucket(bucketName, reqUids,
-                            JSON.stringify(bucketAttributes), done);
+            JSON.stringify(bucketAttributes), done);
     });
 
     it('should try to create the same bucket and fail', done => {
@@ -90,31 +90,31 @@ describe('Bucket Client tests', function testClient() {
                 dataBufs.push(data);
                 dataLen += data.length;
             }).on('error', done)
-              .on('end', () => {
-                  const data = Buffer.concat(dataBufs, dataLen).toString();
-                  const obj = JSON.parse(data);
-                  assert.strictEqual(Object.prototype.hasOwnProperty.call(
-                      obj, 'info'), true);
-                  const info = obj.info;
-                  assert.strictEqual(['start', 'end', 'cseq', 'prune']
-                      .every(key => Object.prototype.hasOwnProperty.call(
-                          obj.info, key)), true);
-                  assert.strictEqual(info.start >= start, true);
-                  assert.strictEqual(info.end <= end, true);
-                  // NOTE: this check will be removed when pruned logs are
-                  // retrieved also
-                  assert.strictEqual(info.prune <= info.start, true);
-                  assert.strictEqual(info.cseq >= info.end, true);
-                  assert.strictEqual(Object.prototype.hasOwnProperty.call(
-                      obj, 'log'), true);
-                  const logs = obj.log;
-                  assert.strictEqual(Array.isArray(logs), true);
-                  assert.strictEqual(logs.length >= 1, true);
-                  assert.strictEqual(logs.every(log =>
-                      (typeof log === 'object') && (Object.keys(log).length > 0)
-                  ), true);
-                  return done();
-              });
+                .on('end', () => {
+                    const data = Buffer.concat(dataBufs, dataLen).toString();
+                    const obj = JSON.parse(data);
+                    assert.strictEqual(Object.prototype.hasOwnProperty.call(
+                        obj, 'info'), true);
+                    const info = obj.info;
+                    assert.strictEqual(['start', 'end', 'cseq', 'prune']
+                        .every(key => Object.prototype.hasOwnProperty.call(
+                            obj.info, key)), true);
+                    assert.strictEqual(info.start >= start, true);
+                    assert.strictEqual(info.end <= end, true);
+                    // NOTE: this check will be removed when pruned logs are
+                    // retrieved also
+                    assert.strictEqual(info.prune <= info.start, true);
+                    assert.strictEqual(info.cseq >= info.end, true);
+                    assert.strictEqual(Object.prototype.hasOwnProperty.call(
+                        obj, 'log'), true);
+                    const logs = obj.log;
+                    assert.strictEqual(Array.isArray(logs), true);
+                    assert.strictEqual(logs.length >= 1, true);
+                    assert.strictEqual(logs.every(log =>
+                        (typeof log === 'object') && (Object.keys(log).length > 0)
+                    ), true);
+                    return done();
+                });
             return undefined;
         });
     });

tests/unit/simple_tests.js

@@ -1,4 +1,4 @@
-'use strict'; // eslint-disable-line strict
+'use strict';  
 
 const assert = require('assert');
 const fs = require('fs');
@@ -54,11 +54,11 @@ const env = {
     https: {
         s: handler => https.createServer(httpsOptions, handler),
         c: new RESTClient(['bucketclient.testing.local:9000'],
-                          undefined,
-                          true,
-                          httpsOptions.key,
-                          httpsOptions.cert,
-                          httpsOptions.ca[0]),
+            undefined,
+            true,
+            httpsOptions.key,
+            httpsOptions.cert,
+            httpsOptions.ca[0]),
     },
 };