@openstax/ts-utils 1.18.0 → 1.20.0

package/dist/cjs/fetch/fetchStatusRetry.d.ts

@@ -0,0 +1,8 @@
+import { GenericFetch } from '.';
+interface Options {
+    status?: number[];
+    onFail?: boolean;
+    retries: number;
+}
+export declare const fetchStatusRetry: (base: GenericFetch, options: Options) => GenericFetch;
+export {};

package/dist/cjs/fetch/fetchStatusRetry.js

@@ -0,0 +1,24 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.fetchStatusRetry = void 0;
+const fetchStatusRetry = (base, options) => {
+    const fetchTry = (retries, ...params) => {
+        return base(...params)
+            .catch(e => options.onFail && retries > 0
+            ? fetchTry(retries - 1, ...params)
+            : Promise.reject(e))
+            .then(async (r) => {
+            var _a;
+            const shouldHandleStatus = (_a = options.status) === null || _a === void 0 ? void 0 : _a.includes(r.status);
+            if (shouldHandleStatus && retries > 0) {
+                return fetchTry(retries - 1, ...params);
+            }
+            else if (shouldHandleStatus) {
+                throw new Error(`fetch failed after ${options.retries} retries. ${params[0]} -- ${r.status}: ${await r.text()}`);
+            }
+            return r;
+        });
+    };
+    return (...params) => fetchTry(options.retries, ...params);
+};
+exports.fetchStatusRetry = fetchStatusRetry;

package/dist/cjs/services/apiGateway/index.d.ts

@@ -21,8 +21,8 @@ declare type RouteClient<R> = {
     })) => Promise<string>;
 };
 interface AcceptStatus<Ro> {
-    <S extends TResponseStatus<Ro>[]>(...args: S): ApiClientResponse<Extract<Ro, Record<'statusCode', S[number]>>>;
-    <S extends number[]>(...args: S): ApiClientResponse<any>;
+    <S extends TResponseStatus<Ro>[]>(...args: S): Promise<ApiClientResponse<Extract<Ro, Record<'statusCode', S[number]>>>>;
+    <S extends number[]>(...args: S): Promise<ApiClientResponse<any>>;
 }
 declare type UnsafeApiClientResponse<Ro> = {
     headers: Headers;

package/dist/cjs/services/apiGateway/index.js

@@ -32,6 +32,7 @@ const query_string_1 = __importDefault(require("query-string"));
 const __1 = require("../..");
 const config_1 = require("../../config");
 const errors_1 = require("../../errors");
+const fetchStatusRetry_1 = require("../../fetch/fetchStatusRetry");
 /** Pulls the content out of a response based on the content type */
 const loadResponse = (response) => () => {
     const [contentType] = (response.headers.get('content-type') || '').split(';');
@@ -57,7 +58,8 @@ const makeRouteClient = (initializer, config, route, authProvider) => {
         const url = await renderUrl({ params, query });
         const body = payload ? JSON.stringify(payload) : undefined;
         const baseOptions = (0, __1.merge)((await (authProvider === null || authProvider === void 0 ? void 0 : authProvider.getAuthorizedFetchConfig())) || {}, fetchConfig || {});
-        return initializer.fetch(url, (0, __1.merge)(baseOptions, {
+        const fetcher = (0, fetchStatusRetry_1.fetchStatusRetry)(initializer.fetch, { retries: 1, status: [502], onFail: true });
+        return fetcher(url, (0, __1.merge)(baseOptions, {
             method: route.method,
             body,
             headers: {
@@ -73,9 +75,9 @@ const makeRouteClient = (initializer, config, route, authProvider) => {
             }
             return {
                 status: response.status,
-                acceptStatus: (...status) => {
+                acceptStatus: async (...status) => {
                     if (!status.includes(response.status)) {
-                        throw new Error(`unexpected HTTP ${response.status} response from api`);
+                        throw new Error(`unexpected HTTP ${response.status} response from api: ${await response.text()}`);
                     }
                     return { status: response.status, headers: response.headers, load: (0, exports.loadResponse)(response) };
                 },

package/dist/cjs/services/lrsGateway/index.js

@@ -33,12 +33,14 @@ const __1 = require("../..");
 const assertions_1 = require("../../assertions");
 const config_1 = require("../../config");
 const errors_1 = require("../../errors");
+const fetchStatusRetry_1 = require("../../fetch/fetchStatusRetry");
 const guards_1 = require("../../guards");
 const routing_1 = require("../../routing");
 const lrsGateway = (initializer) => (configProvider) => {
     const config = configProvider[(0, guards_1.ifDefined)(initializer.configSpace, 'lrs')];
     const lrsHost = (0, __1.once)(() => (0, config_1.resolveConfigValue)(config.lrsHost));
     const lrsAuthorization = (0, __1.once)(() => (0, config_1.resolveConfigValue)(config.lrsAuthorization));
+    const fetcher = (0, fetchStatusRetry_1.fetchStatusRetry)(initializer.fetch, { retries: 1, status: [502], onFail: true });
     return (authProvider) => {
         // Note: This method actually uses POST
         const putXapiStatements = async (statements) => {
@@ -54,7 +56,7 @@ const lrsGateway = (initializer) => (configProvider) => {
                 },
                 timestamp: (0, formatISO_1.default)(new Date())
             }));
-            const response = await initializer.fetch((await lrsHost()).replace(/\/+$/, '') + '/data/xAPI/statements', {
+            const response = await fetcher((await lrsHost()).replace(/\/+$/, '') + '/data/xAPI/statements', {
                 body: JSON.stringify(statementsWithDefaults),
                 headers: {
                     Authorization: await lrsAuthorization(),
@@ -82,13 +84,13 @@ ${await response.text()}`);
             }
             return response.json();
         };
-        const getMoreXapiStatements = async (more) => formatGetXapiStatementsResponse(initializer.fetch((await lrsHost()).replace(/\/+$/, '') + more, {
+        const getMoreXapiStatements = async (more) => formatGetXapiStatementsResponse(fetcher((await lrsHost()).replace(/\/+$/, '') + more, {
             headers: {
                 Authorization: await lrsAuthorization(),
                 'X-Experience-API-Version': '1.0.0',
             },
         }));
-        const fetchXapiStatements = async ({ user, anyUser, ...options }) => initializer.fetch((await lrsHost()).replace(/\/+$/, '') + '/data/xAPI/statements?' + queryString.stringify({
+        const fetchXapiStatements = async ({ user, anyUser, ...options }) => fetcher((await lrsHost()).replace(/\/+$/, '') + '/data/xAPI/statements?' + queryString.stringify({
             ...options,
             ...(anyUser === true ? {} : {
                 agent: JSON.stringify({