@applitools/ec-client 1.2.34 → 1.3.0

package/CHANGELOG.md

@@ -1,5 +1,38 @@
 # Changelog
 
+## [1.3.0](https://github.com/applitools/eyes.sdk.javascript1/compare/js/ec-client@1.2.34...js/ec-client@1.3.0) (2023-06-21)
+
+
+### Features
+
+* put in a queue create tunnel requests which cannot succeed due to a limit ([3309147](https://github.com/applitools/eyes.sdk.javascript1/commit/33091473f3fcbc4dd5fc853624bbe441ce11ce87))
+
+
+### Dependencies
+
+* @applitools/core-base bumped from 1.1.58 to 1.2.0
+  #### Features
+
+  * **js/core-base:** new feature ([dd5705d](https://github.com/applitools/eyes.sdk.javascript1/commit/dd5705d5e99d34f9492e890a0b4af6c52d6b33e3))
+
+
+  #### Bug Fixes
+
+  * rerelease ([2d46d0c](https://github.com/applitools/eyes.sdk.javascript1/commit/2d46d0c9ee14a72406e60350d4cce92991272afd))
+
+
+
+* @applitools/driver bumped from 1.12.2 to 1.12.3
+
+* @applitools/logger bumped from 2.0.3 to 2.0.4
+  #### Bug Fixes
+
+  * fixed issue when extended logger didn't preserve base's handler ([7c5e029](https://github.com/applitools/eyes.sdk.javascript1/commit/7c5e0299522f792aad72b7b3827df31a1ab2d68f))
+* @applitools/socket bumped from 1.1.3 to 1.1.4
+
+* @applitools/spec-driver-webdriver bumped from 1.0.34 to 1.0.35
+
+
 ## [1.2.34](https://github.com/applitools/eyes.sdk.javascript1/compare/js/ec-client@1.2.33...js/ec-client@1.2.34) (2023-06-15)
 
 

package/dist/tunnels/manager.js

@@ -25,9 +25,15 @@ var __importStar = (this && this.__importStar) || function (mod) {
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.makeTunnelManager = void 0;
 const req_1 = require("@applitools/req");
+const queue_1 = require("../utils/queue");
 //@ts-ignore
 const execution_grid_tunnel_1 = require("@applitools/execution-grid-tunnel");
 const utils = __importStar(require("@applitools/utils"));
+const RETRY_BACKOFF = [
+    ...Array(5).fill(2000),
+    ...Array(4).fill(5000),
+    10000, // all next tries with delay 10s
+];
 async function makeTunnelManager({ settings, logger, }) {
     let server;
     const getTunnelServiceUrl = utils.general.cachify(async () => {
@@ -42,24 +48,8 @@ async function makeTunnelManager({ settings, logger, }) {
         };
         return `http://localhost:${port}`;
     });
-    const req = (0, req_1.makeReq)({
-        retry: {
-            validate: async ({ response }) => {
-                if (!response)
-                    return false;
-                const body = await response
-                    .clone()
-                    .json()
-                    .catch(() => null);
-                return ['CONCURRENCY_LIMIT_REACHED', 'NO_AVAILABLE_TUNNEL_PROXY'].includes(body === null || body === void 0 ? void 0 : body.message);
-            },
-            timeout: [
-                ...Array(5).fill(2000),
-                ...Array(4).fill(5000),
-                10000, // all next tries with delay 10s
-            ],
-        },
-    });
+    const req = (0, req_1.makeReq)({});
+    const queues = new Map();
     const pools = new Map();
     return { create, destroy, acquire, release, close: async () => server === null || server === void 0 ? void 0 : server.close() };
     async function acquire(credentials) {
@@ -83,24 +73,48 @@ async function makeTunnelManager({ settings, logger, }) {
         await pool.release(tunnels);
     }
     async function create(credentials) {
-        var _a, _b;
         if (!(settings === null || settings === void 0 ? void 0 : settings.serverUrl)) {
             settings !== null && settings !== void 0 ? settings : (settings = {});
             settings.serverUrl = await getTunnelServiceUrl();
         }
-        const response = await req('/tunnels', {
-            method: 'POST',
-            baseUrl: settings.serverUrl,
-            headers: {
-                'x-eyes-api-key': credentials.apiKey,
-                'x-eyes-server-url': credentials.eyesServerUrl,
-            },
-        });
-        const body = await response.json().catch(() => null);
-        if (response.status === 201)
-            return { tunnelId: body, credentials };
-        logger.error(`Failed to create tunnel with status ${response.status} and code ${(_a = body === null || body === void 0 ? void 0 : body.message) !== null && _a !== void 0 ? _a : 'UNKNOWN_ERROR'}`);
-        throw new Error(`Failed to create tunnel with code ${(_b = body === null || body === void 0 ? void 0 : body.message) !== null && _b !== void 0 ? _b : 'UNKNOWN_ERROR'}`);
+        const queueKey = JSON.stringify(credentials);
+        let queue = queues.get(queueKey);
+        if (!queue) {
+            queue = (0, queue_1.makeQueue)({ logger: logger.extend({ tags: [`queue-${queueKey}`] }) });
+            queues.set(queueKey, queue);
+        }
+        return queue.run(task);
+        async function task(signal, attempt = 1) {
+            var _a, _b;
+            if (signal.aborted)
+                return queue.pause;
+            const response = await req('/tunnels', {
+                method: 'POST',
+                baseUrl: settings.serverUrl,
+                headers: {
+                    'x-eyes-api-key': credentials.apiKey,
+                    'x-eyes-server-url': credentials.eyesServerUrl,
+                },
+                // TODO uncomment when we can throw different abort reasons for task cancelation and timeout abortion
+                // signal,
+            });
+            const body = await response.json().catch(() => null);
+            if (['CONCURRENCY_LIMIT_REACHED', 'NO_AVAILABLE_TUNNEL_PROXY'].includes(body === null || body === void 0 ? void 0 : body.message)) {
+                queue.cork();
+                // after query is corked the task might be aborted
+                if (signal.aborted)
+                    return queue.pause;
+                await utils.general.sleep(RETRY_BACKOFF[Math.min(attempt, RETRY_BACKOFF.length - 1)]);
+                return task(signal, attempt + 1);
+            }
+            else {
+                queue.uncork();
+                if (response.status === 201)
+                    return { tunnelId: body, credentials };
+                logger.error(`Failed to create tunnel with status ${response.status} and code ${(_a = body === null || body === void 0 ? void 0 : body.message) !== null && _a !== void 0 ? _a : 'UNKNOWN_ERROR'}`);
+                throw new Error(`Failed to create tunnel with code ${(_b = body === null || body === void 0 ? void 0 : body.message) !== null && _b !== void 0 ? _b : 'UNKNOWN_ERROR'}`);
+            }
+        }
     }
     async function destroy(tunnel) {
         var _a, _b;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@applitools/ec-client",
-  "version": "1.2.34",
+  "version": "1.3.0",
   "homepage": "https://applitools.com",
   "bugs": {
     "url": "https://github.com/applitools/eyes.sdk.javascript1/issues"
@@ -52,13 +52,13 @@
     "test:it": "MOCHA_GROUP=it run --top-level mocha './test/it/*.spec.ts'"
   },
   "dependencies": {
-    "@applitools/core-base": "1.1.58",
-    "@applitools/driver": "1.12.2",
+    "@applitools/core-base": "1.2.0",
+    "@applitools/driver": "1.12.3",
     "@applitools/execution-grid-tunnel": "2.1.0",
-    "@applitools/logger": "2.0.3",
+    "@applitools/logger": "2.0.4",
     "@applitools/req": "1.3.2",
-    "@applitools/socket": "1.1.3",
-    "@applitools/spec-driver-webdriver": "1.0.34",
+    "@applitools/socket": "1.1.4",
+    "@applitools/spec-driver-webdriver": "1.0.35",
     "@applitools/utils": "1.4.0",
     "abort-controller": "3.0.0",
     "webdriver": "7",