@workos-inc/node 7.72.0 → 7.72.1

package/lib/common/net/fetch-client.js

@@ -34,7 +34,7 @@ class FetchHttpClient extends http_client_1.HttpClient {
     get(path, options) {
         return __awaiter(this, void 0, void 0, function* () {
             const resourceURL = http_client_1.HttpClient.getResourceURL(this.baseURL, path, options.params);
-            if (path.startsWith('/fga/')) {
+            if (http_client_1.HttpClient.isPathRetryable(path)) {
                 return yield this.fetchRequestWithRetry(resourceURL, 'GET', null, options.headers);
             }
             else {
@@ -45,7 +45,7 @@ class FetchHttpClient extends http_client_1.HttpClient {
     post(path, entity, options) {
         return __awaiter(this, void 0, void 0, function* () {
             const resourceURL = http_client_1.HttpClient.getResourceURL(this.baseURL, path, options.params);
-            if (path.startsWith('/fga/')) {
+            if (http_client_1.HttpClient.isPathRetryable(path)) {
                 return yield this.fetchRequestWithRetry(resourceURL, 'POST', http_client_1.HttpClient.getBody(entity), Object.assign(Object.assign({}, http_client_1.HttpClient.getContentTypeHeader(entity)), options.headers));
             }
             else {
@@ -56,7 +56,7 @@ class FetchHttpClient extends http_client_1.HttpClient {
     put(path, entity, options) {
         return __awaiter(this, void 0, void 0, function* () {
             const resourceURL = http_client_1.HttpClient.getResourceURL(this.baseURL, path, options.params);
-            if (path.startsWith('/fga/')) {
+            if (http_client_1.HttpClient.isPathRetryable(path)) {
                 return yield this.fetchRequestWithRetry(resourceURL, 'PUT', http_client_1.HttpClient.getBody(entity), Object.assign(Object.assign({}, http_client_1.HttpClient.getContentTypeHeader(entity)), options.headers));
             }
             else {
@@ -67,7 +67,7 @@ class FetchHttpClient extends http_client_1.HttpClient {
     delete(path, options) {
         return __awaiter(this, void 0, void 0, function* () {
             const resourceURL = http_client_1.HttpClient.getResourceURL(this.baseURL, path, options.params);
-            if (path.startsWith('/fga/')) {
+            if (http_client_1.HttpClient.isPathRetryable(path)) {
                 return yield this.fetchRequestWithRetry(resourceURL, 'DELETE', null, options.headers);
             }
             else {

package/lib/common/net/fetch-client.spec.js

@@ -58,6 +58,38 @@ describe('Fetch client', () => {
             expect((0, test_utils_1.fetchURL)()).toBe('https://test.workos.com/fga/v1/resources/user/user-1');
             expect(yield response.toJSON()).toEqual({ data: 'response' });
         }));
+        it('get for Vault path should call fetchRequestWithRetry and return response', () => __awaiter(void 0, void 0, void 0, function* () {
+            (0, test_utils_1.fetchOnce)({ data: 'response' });
+            const mockFetchRequestWithRetry = jest.spyOn(fetch_client_1.FetchHttpClient.prototype, 'fetchRequestWithRetry');
+            const response = yield fetchClient.get('/vault/v1/kv', {});
+            expect(mockFetchRequestWithRetry).toHaveBeenCalledTimes(1);
+            expect((0, test_utils_1.fetchURL)()).toBe('https://test.workos.com/vault/v1/kv');
+            expect(yield response.toJSON()).toEqual({ data: 'response' });
+        }));
+        it('post for Vault path should call fetchRequestWithRetry and return response', () => __awaiter(void 0, void 0, void 0, function* () {
+            (0, test_utils_1.fetchOnce)({ data: 'response' });
+            const mockFetchRequestWithRetry = jest.spyOn(fetch_client_1.FetchHttpClient.prototype, 'fetchRequestWithRetry');
+            const response = yield fetchClient.post('/vault/v1/kv', {}, {});
+            expect(mockFetchRequestWithRetry).toHaveBeenCalledTimes(1);
+            expect((0, test_utils_1.fetchURL)()).toBe('https://test.workos.com/vault/v1/kv');
+            expect(yield response.toJSON()).toEqual({ data: 'response' });
+        }));
+        it('put for Vault path should call fetchRequestWithRetry and return response', () => __awaiter(void 0, void 0, void 0, function* () {
+            (0, test_utils_1.fetchOnce)({ data: 'response' });
+            const mockFetchRequestWithRetry = jest.spyOn(fetch_client_1.FetchHttpClient.prototype, 'fetchRequestWithRetry');
+            const response = yield fetchClient.put('/vault/v1/kv/secret', {}, {});
+            expect(mockFetchRequestWithRetry).toHaveBeenCalledTimes(1);
+            expect((0, test_utils_1.fetchURL)()).toBe('https://test.workos.com/vault/v1/kv/secret');
+            expect(yield response.toJSON()).toEqual({ data: 'response' });
+        }));
+        it('delete for Vault path should call fetchRequestWithRetry and return response', () => __awaiter(void 0, void 0, void 0, function* () {
+            (0, test_utils_1.fetchOnce)({ data: 'response' });
+            const mockFetchRequestWithRetry = jest.spyOn(fetch_client_1.FetchHttpClient.prototype, 'fetchRequestWithRetry');
+            const response = yield fetchClient.delete('/vault/v1/kv/secret', {});
+            expect(mockFetchRequestWithRetry).toHaveBeenCalledTimes(1);
+            expect((0, test_utils_1.fetchURL)()).toBe('https://test.workos.com/vault/v1/kv/secret');
+            expect(yield response.toJSON()).toEqual({ data: 'response' });
+        }));
         it('should retry request on 500 status code', () => __awaiter(void 0, void 0, void 0, function* () {
             (0, test_utils_1.fetchOnce)({}, {
                 status: 500,

package/lib/common/net/http-client.d.ts

@@ -18,6 +18,7 @@ export declare abstract class HttpClient implements HttpClientInterface {
     static getQueryString(queryObj?: Record<string, any>): string | undefined;
     static getContentTypeHeader(entity: any): RequestHeaders | undefined;
     static getBody(entity: any): BodyInit | null | undefined;
+    static isPathRetryable(path: string): boolean;
     private getSleepTimeInMilliseconds;
     sleep: (retryAttempt: number) => Promise<unknown>;
 }

package/lib/common/net/http-client.js

@@ -8,7 +8,7 @@ class HttpClient {
         this.MAX_RETRY_ATTEMPTS = 3;
         this.BACKOFF_MULTIPLIER = 1.5;
         this.MINIMUM_SLEEP_TIME_IN_MILLISECONDS = 500;
-        this.RETRY_STATUS_CODES = [500, 502, 504];
+        this.RETRY_STATUS_CODES = [408, 500, 502, 504];
         this.sleep = (retryAttempt) => new Promise((resolve) => setTimeout(resolve, this.getSleepTimeInMilliseconds(retryAttempt)));
     }
     /** The HTTP client name used for diagnostics */
@@ -52,6 +52,9 @@ class HttpClient {
         }
         return JSON.stringify(entity);
     }
+    static isPathRetryable(path) {
+        return path.startsWith('/fga/') || path.startsWith('/vault/');
+    }
     getSleepTimeInMilliseconds(retryAttempt) {
         const sleepTime = this.MINIMUM_SLEEP_TIME_IN_MILLISECONDS *
             Math.pow(this.BACKOFF_MULTIPLIER, retryAttempt);

package/lib/common/net/node-client.js

@@ -67,7 +67,7 @@ class NodeHttpClient extends http_client_1.HttpClient {
     get(path, options) {
         return __awaiter(this, void 0, void 0, function* () {
             const resourceURL = http_client_1.HttpClient.getResourceURL(this.baseURL, path, options.params);
-            if (path.startsWith('/fga/')) {
+            if (http_client_1.HttpClient.isPathRetryable(path)) {
                 return yield this.nodeRequestWithRetry(resourceURL, 'GET', null, options.headers);
             }
             else {
@@ -78,7 +78,7 @@ class NodeHttpClient extends http_client_1.HttpClient {
     post(path, entity, options) {
         return __awaiter(this, void 0, void 0, function* () {
             const resourceURL = http_client_1.HttpClient.getResourceURL(this.baseURL, path, options.params);
-            if (path.startsWith('/fga/')) {
+            if (http_client_1.HttpClient.isPathRetryable(path)) {
                 return yield this.nodeRequestWithRetry(resourceURL, 'POST', NodeHttpClient.getBody(entity), Object.assign(Object.assign({}, http_client_1.HttpClient.getContentTypeHeader(entity)), options.headers));
             }
             else {
@@ -89,7 +89,7 @@ class NodeHttpClient extends http_client_1.HttpClient {
     put(path, entity, options) {
         return __awaiter(this, void 0, void 0, function* () {
             const resourceURL = http_client_1.HttpClient.getResourceURL(this.baseURL, path, options.params);
-            if (path.startsWith('/fga/')) {
+            if (http_client_1.HttpClient.isPathRetryable(path)) {
                 return yield this.nodeRequestWithRetry(resourceURL, 'PUT', NodeHttpClient.getBody(entity), Object.assign(Object.assign({}, http_client_1.HttpClient.getContentTypeHeader(entity)), options.headers));
             }
             else {
@@ -100,7 +100,7 @@ class NodeHttpClient extends http_client_1.HttpClient {
     delete(path, options) {
         return __awaiter(this, void 0, void 0, function* () {
             const resourceURL = http_client_1.HttpClient.getResourceURL(this.baseURL, path, options.params);
-            if (path.startsWith('/fga/')) {
+            if (http_client_1.HttpClient.isPathRetryable(path)) {
                 return yield this.nodeRequestWithRetry(resourceURL, 'DELETE', null, options.headers);
             }
             else {

package/lib/common/net/node-client.spec.js

@@ -58,6 +58,42 @@ describe('Node client', () => {
         expect(mockNodeRequestWithRetry).toHaveBeenCalledTimes(1);
         expect(yield response.toJSON()).toEqual({ data: 'response' });
     }));
+    it('get for Vault path should call nodeRequestWithRetry and return response', () => __awaiter(void 0, void 0, void 0, function* () {
+        (0, nock_1.default)('https://test.workos.com')
+            .get('/vault/v1/kv')
+            .reply(200, { data: 'response' });
+        const mockNodeRequestWithRetry = jest.spyOn(node_client_1.NodeHttpClient.prototype, 'nodeRequestWithRetry');
+        const response = yield nodeClient.get('/vault/v1/kv', {});
+        expect(mockNodeRequestWithRetry).toHaveBeenCalledTimes(1);
+        expect(yield response.toJSON()).toEqual({ data: 'response' });
+    }));
+    it('post for Vault path should call nodeRequestWithRetry and return response', () => __awaiter(void 0, void 0, void 0, function* () {
+        (0, nock_1.default)('https://test.workos.com')
+            .post('/vault/v1/kv')
+            .reply(200, { data: 'response' });
+        const mockNodeRequestWithRetry = jest.spyOn(node_client_1.NodeHttpClient.prototype, 'nodeRequestWithRetry');
+        const response = yield nodeClient.post('/vault/v1/kv', {}, {});
+        expect(mockNodeRequestWithRetry).toHaveBeenCalledTimes(1);
+        expect(yield response.toJSON()).toEqual({ data: 'response' });
+    }));
+    it('put for Vault path should call nodeRequestWithRetry and return response', () => __awaiter(void 0, void 0, void 0, function* () {
+        (0, nock_1.default)('https://test.workos.com')
+            .put('/vault/v1/kv/secret')
+            .reply(200, { data: 'response' });
+        const mockNodeRequestWithRetry = jest.spyOn(node_client_1.NodeHttpClient.prototype, 'nodeRequestWithRetry');
+        const response = yield nodeClient.put('/vault/v1/kv/secret', {}, {});
+        expect(mockNodeRequestWithRetry).toHaveBeenCalledTimes(1);
+        expect(yield response.toJSON()).toEqual({ data: 'response' });
+    }));
+    it('delete for Vault path should call nodeRequestWithRetry and return response', () => __awaiter(void 0, void 0, void 0, function* () {
+        (0, nock_1.default)('https://test.workos.com')
+            .delete('/vault/v1/kv/secret')
+            .reply(200, { data: 'response' });
+        const mockNodeRequestWithRetry = jest.spyOn(node_client_1.NodeHttpClient.prototype, 'nodeRequestWithRetry');
+        const response = yield nodeClient.delete('/vault/v1/kv/secret', {});
+        expect(mockNodeRequestWithRetry).toHaveBeenCalledTimes(1);
+        expect(yield response.toJSON()).toEqual({ data: 'response' });
+    }));
     it('should retry request on 500 status code', () => __awaiter(void 0, void 0, void 0, function* () {
         (0, nock_1.default)('https://test.workos.com')
             .get('/fga/v1/resources')

package/lib/workos.js

@@ -33,7 +33,7 @@ const actions_1 = require("./actions/actions");
 const vault_1 = require("./vault/vault");
 const conflict_exception_1 = require("./common/exceptions/conflict.exception");
 const parse_error_1 = require("./common/exceptions/parse-error");
-const VERSION = '7.72.0';
+const VERSION = '7.72.1';
 const DEFAULT_HOSTNAME = 'api.workos.com';
 const HEADER_AUTHORIZATION = 'Authorization';
 const HEADER_IDEMPOTENCY_KEY = 'Idempotency-Key';

package/package.json

@@ -1,5 +1,5 @@
 {
-  "version": "7.72.0",
+  "version": "7.72.1",
   "name": "@workos-inc/node",
   "author": "WorkOS",
   "description": "A Node wrapper for the WorkOS API",