@openstax/ts-utils 1.1.39 → 1.1.42

package/dist/cjs/config/lambdaParameterConfig.js

@@ -19,7 +19,7 @@ const lambdaParameterConfig = (parameterName) => async () => {
     if (!lambdaExtensionReadyPromise) {
         // This request will return 400 Bad Request,
         // but we only care that it'll block until the extension is ready
-        lambdaExtensionReadyPromise = (0, node_fetch_1.default)(lambdaExtensionUrl);
+        lambdaExtensionReadyPromise = (0, helpers_1.retryWithDelay)(() => (0, node_fetch_1.default)(lambdaExtensionUrl));
     }
     await lambdaExtensionReadyPromise;
     const resp = await (0, helpers_1.retryWithDelay)(() => (0, node_fetch_1.default)(

package/dist/cjs/errors.d.ts

@@ -10,3 +10,7 @@ export declare class NotFoundError extends Error {
     static readonly TYPE = "NotFoundError";
     static matches: (e: any) => e is typeof NotFoundError;
 }
+export declare class SessionExpiredError extends Error {
+    static readonly TYPE = "SessionExpiredError";
+    static matches: (e: any) => e is typeof SessionExpiredError;
+}

package/dist/cjs/errors.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.NotFoundError = exports.UnauthorizedError = exports.InvalidRequestError = void 0;
+exports.SessionExpiredError = exports.NotFoundError = exports.UnauthorizedError = exports.InvalidRequestError = void 0;
 /*
  * if code is split into multiple bundles, sometimes each bundle
  * will get its own definition of this module and then instanceof checks
@@ -30,3 +30,8 @@ class NotFoundError extends Error {
 exports.NotFoundError = NotFoundError;
 NotFoundError.TYPE = 'NotFoundError';
 NotFoundError.matches = errorIsType(NotFoundError);
+class SessionExpiredError extends Error {
+}
+exports.SessionExpiredError = SessionExpiredError;
+SessionExpiredError.TYPE = 'SessionExpiredError';
+SessionExpiredError.matches = errorIsType(SessionExpiredError);

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

@@ -31,6 +31,7 @@ const pathToRegexp = __importStar(require("path-to-regexp"));
 const query_string_1 = __importDefault(require("query-string"));
 const __1 = require("../..");
 const config_1 = require("../../config");
+const errors_1 = require("../../errors");
 const loadResponse = (response) => () => {
     const [contentType] = (response.headers.get('content-type') || '').split(';');
     switch (contentType) {
@@ -61,16 +62,24 @@ const makeRouteClient = (initializer, config, route, authProvider) => {
                 ...fetchConfig === null || fetchConfig === void 0 ? void 0 : fetchConfig.headers,
                 ...(body ? { 'content-type': 'application/json' } : {}),
             }
-        })).then(response => ({
-            status: response.status,
-            acceptStatus: (...status) => {
-                if (!status.includes(response.status)) {
-                    throw new Error('unexpected response from api');
-                }
-                return { status: response.status, load: (0, exports.loadResponse)(response) };
-            },
-            load: (0, exports.loadResponse)(response),
-        }));
+        })).then(response => {
+            if (response.status === 401) {
+                throw new errors_1.UnauthorizedError();
+            }
+            if (response.status === 440) {
+                throw new errors_1.SessionExpiredError();
+            }
+            return {
+                status: response.status,
+                acceptStatus: (...status) => {
+                    if (!status.includes(response.status)) {
+                        throw new Error('unexpected response from api');
+                    }
+                    return { status: response.status, load: (0, exports.loadResponse)(response) };
+                },
+                load: (0, exports.loadResponse)(response),
+            };
+        });
     };
     routeClient.renderUrl = renderUrl;
     return routeClient;

package/dist/cjs/services/authProvider/utils/decryptAndVerify.js

@@ -30,6 +30,7 @@ exports.decryptAndVerify = void 0;
 const crypto = __importStar(require("crypto"));
 const util_1 = require("util");
 const jsonwebtoken_1 = __importDefault(require("jsonwebtoken"));
+const errors_1 = require("../../../errors");
 const guards_1 = require("../../../guards");
 const decrypt = (input, key) => {
     const splitInput = input.split('.');
@@ -51,7 +52,7 @@ const decryptAndVerify = (token, encryptionPrivateKey, signaturePublicKey) => {
         // Decrypt SSO cookie
         const plaintext = decrypt(token, encryptionPrivateKey);
         const payload = jsonwebtoken_1.default.verify(plaintext, signaturePublicKey, {
-            clockTolerance: 300 // 5 minutes 
+            clockTolerance: 300 // 5 minutes
         });
         if (!(0, guards_1.isPlainObject)(payload) || !(0, guards_1.isPlainObject)(payload.sub) || !payload.sub.uuid) {
             return undefined;
@@ -59,7 +60,10 @@ const decryptAndVerify = (token, encryptionPrivateKey, signaturePublicKey) => {
         // TS is confused because the library types the `sub` as a string
         return payload.sub;
     }
-    catch {
+    catch (err) {
+        if (err instanceof jsonwebtoken_1.default.TokenExpiredError) {
+            throw new errors_1.SessionExpiredError();
+        }
         return undefined;
     }
 };

package/dist/cjs/services/lrsGateway/attempt-utils.js

@@ -205,8 +205,7 @@ const createAttemptStatement = (activity, parentActivity) => {
 exports.createAttemptStatement = createAttemptStatement;
 /* resolves with the statement id */
 const putAttemptStatement = async (gateway, activity, parentActivity) => {
-    return await gateway.putXapiStatements([(0, exports.createAttemptStatement)(activity, parentActivity)])
-        .then(statements => statements[0]);
+    return (await gateway.putXapiStatements([(0, exports.createAttemptStatement)(activity, parentActivity)]))[0];
 };
 exports.putAttemptStatement = putAttemptStatement;
 /*
@@ -230,8 +229,7 @@ const createAttemptActivityStatement = (attemptStatement, verb, result) => {
 };
 exports.createAttemptActivityStatement = createAttemptActivityStatement;
 const putAttemptActivityStatement = async (gateway, attemptStatement, verb, result) => {
-    return await gateway.putXapiStatements([(0, exports.createAttemptActivityStatement)(attemptStatement, verb, result)])
-        .then(statements => statements[0]);
+    return (await gateway.putXapiStatements([(0, exports.createAttemptActivityStatement)(attemptStatement, verb, result)]))[0];
 };
 exports.putAttemptActivityStatement = putAttemptActivityStatement;
 /*
@@ -269,7 +267,6 @@ const createCompletedStatement = (attemptStatement, result) => {
 };
 exports.createCompletedStatement = createCompletedStatement;
 const putCompletedStatement = async (gateway, attemptStatement, result) => {
-    return await gateway.putXapiStatements([(0, exports.createCompletedStatement)(attemptStatement, result)])
-        .then(statements => statements[0]);
+    return (await gateway.putXapiStatements([(0, exports.createCompletedStatement)(attemptStatement, result)]))[0];
 };
 exports.putCompletedStatement = putCompletedStatement;

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

@@ -40,6 +40,7 @@ const lrsGateway = (initializer) => (configProvider) => {
     const lrsHost = (0, __1.once)(() => (0, config_1.resolveConfigValue)(config.lrsHost));
     const lrsAuthorization = (0, __1.once)(() => (0, config_1.resolveConfigValue)(config.lrsAuthorization));
     return (authProvider) => {
+        // Note: This method actually uses POST
         const putXapiStatements = async (statements) => {
             const user = (0, assertions_1.assertDefined)(await authProvider.getUser(), new errors_1.UnauthorizedError);
             const statementsWithDefaults = statements.map(statement => ({
@@ -53,7 +54,7 @@ const lrsGateway = (initializer) => (configProvider) => {
                 },
                 timestamp: (0, formatISO_1.default)(new Date())
             }));
-            return initializer.fetch((await lrsHost()).replace(/\/+$/, '') + '/data/xAPI/statements', {
+            const response = await initializer.fetch((await lrsHost()).replace(/\/+$/, '') + '/data/xAPI/statements', {
                 body: JSON.stringify(statementsWithDefaults),
                 headers: {
                     Authorization: await lrsAuthorization(),
@@ -61,42 +62,50 @@ const lrsGateway = (initializer) => (configProvider) => {
                     'X-Experience-API-Version': '1.0.0',
                 },
                 method: routing_1.METHOD.POST,
-            })
-                .then(response => response.json())
-                .then(ids => ids.map((id, index) => ({ id, ...statementsWithDefaults[index] })));
+            });
+            if (![200, 201].includes(response.status)) {
+                throw new Error(`Unexpected LRS POST statements response code ${response.status} with body:
+
+${await response.text()}`);
+            }
+            const ids = await response.json();
+            return ids.map((id, index) => ({ id, ...statementsWithDefaults[index] }));
         };
-        const getMoreXapiStatements = async (more) => {
-            return initializer.fetch((await lrsHost()).replace(/\/+$/, '') + more, {
-                headers: {
-                    Authorization: await lrsAuthorization(),
-                    'X-Experience-API-Version': '1.0.0',
-                },
-            })
-                .then(response => response.json())
-                .then(json => json);
+        // Note: This code does not currently handle a single statement response,
+        // which can return 404 if the statement is not found
+        const formatGetXapiStatementsResponse = async (responsePromise) => {
+            const response = await responsePromise;
+            if (response.status !== 200) {
+                throw new Error(`Unexpected LRS GET statements response code ${response.status} with body:
+
+${await response.text()}`);
+            }
+            return response.json();
         };
-        const getXapiStatements = async ({ user, anyUser, ...options }) => {
-            return initializer.fetch((await lrsHost()).replace(/\/+$/, '') + '/data/xAPI/statements?' + queryString.stringify({
-                ...options,
-                ...(anyUser === true ? {} : {
-                    agent: JSON.stringify({
-                        account: {
-                            homePage: 'https://openstax.org',
-                            name: user || (0, assertions_1.assertDefined)(await authProvider.getUser(), new errors_1.UnauthorizedError()).uuid,
-                        },
-                        objectType: 'Agent',
-                    }),
-                })
-            }), {
-                headers: {
-                    Authorization: await lrsAuthorization(),
-                    'X-Experience-API-Version': '1.0.0',
-                },
+        const getMoreXapiStatements = async (more) => formatGetXapiStatementsResponse(initializer.fetch((await lrsHost()).replace(/\/+$/, '') + more, {
+            headers: {
+                Authorization: await lrsAuthorization(),
+                'X-Experience-API-Version': '1.0.0',
+            },
+        }));
+        const getXapiStatements = async ({ user, anyUser, ...options }) => formatGetXapiStatementsResponse(initializer.fetch((await lrsHost()).replace(/\/+$/, '') + '/data/xAPI/statements?' + queryString.stringify({
+            ...options,
+            ...(anyUser === true ? {} : {
+                agent: JSON.stringify({
+                    account: {
+                        homePage: 'https://openstax.org',
+                        name: user || (0, assertions_1.assertDefined)(await authProvider.getUser(), new errors_1.UnauthorizedError()).uuid,
+                    },
+                    objectType: 'Agent',
+                }),
             })
-                .then(response => response.json())
-                .then(json => json);
-        };
-        const getAllXapiStatements = (...args) => {
+        }), {
+            headers: {
+                Authorization: await lrsAuthorization(),
+                'X-Experience-API-Version': '1.0.0',
+            },
+        }));
+        const getAllXapiStatements = async (...args) => {
             const loadRemaining = async (result) => {
                 if (!result.more) {
                     return result.statements;
@@ -104,7 +113,7 @@ const lrsGateway = (initializer) => (configProvider) => {
                 const { more, statements } = await getMoreXapiStatements(result.more);
                 return loadRemaining({ more, statements: [...result.statements, ...statements] });
             };
-            return getXapiStatements(...args).then(loadRemaining);
+            return loadRemaining(await getXapiStatements(...args));
         };
         return {
             putXapiStatements,