@boxyhq/saml-jackson 1.0.3 → 1.0.4

package/dist/controller/oauth.js

@@ -61,9 +61,9 @@ const validateResponse = (rawResponse, validateOpts) => __awaiter(void 0, void 0
     }
     return profile;
 });
-function getEncodedTenantProduct(client_id) {
+function getEncodedTenantProduct(param) {
     try {
-        const sp = new URLSearchParams(client_id);
+        const sp = new URLSearchParams(param);
         const tenant = sp.get('tenant');
         const product = sp.get('product');
         if (tenant && product) {
@@ -128,7 +128,7 @@ class OAuthController {
     }
     authorize(body) {
         return __awaiter(this, void 0, void 0, function* () {
-            const { response_type = 'code', client_id, redirect_uri, state, tenant, product, access_type, code_challenge, code_challenge_method = '', 
+            const { response_type = 'code', client_id, redirect_uri, state, tenant, product, access_type, scope, code_challenge, code_challenge_method = '', 
             // eslint-disable-next-line @typescript-eslint/no-unused-vars
             provider = 'saml', idp_hint, } = body;
             let requestedTenant = tenant;
@@ -137,9 +137,6 @@ class OAuthController {
             if (!redirect_uri) {
                 throw new error_1.JacksonError('Please specify a redirect URL.', 400);
             }
-            if (!state) {
-                throw new error_1.JacksonError('Please specify a state to safeguard against XSRF attacks.', 400);
-            }
             let samlConfig;
             if (tenant && product) {
                 const samlConfigs = yield this.configStore.getByIndex({
@@ -169,13 +166,15 @@ class OAuthController {
                     samlConfig = resolvedSamlConfig;
                 }
             }
-            else if ((client_id && client_id !== '' && client_id !== 'undefined' && client_id !== 'null') ||
-                (access_type && access_type !== '' && access_type !== 'undefined' && access_type !== 'null')) {
+            else if (client_id && client_id !== '' && client_id !== 'undefined' && client_id !== 'null') {
                 // if tenant and product are encoded in the client_id then we parse it and check for the relevant config(s)
                 let sp = getEncodedTenantProduct(client_id);
                 if (!sp && access_type) {
                     sp = getEncodedTenantProduct(access_type);
                 }
+                if (!sp && scope) {
+                    sp = getEncodedTenantProduct(scope);
+                }
                 if (sp && sp.tenant && sp.product) {
                     requestedTenant = sp.tenant;
                     requestedProduct = sp.product;
@@ -223,6 +222,24 @@ class OAuthController {
             if (!allowed.redirect(redirect_uri, samlConfig.redirectUrl)) {
                 throw new error_1.JacksonError('Redirect URL is not allowed.', 403);
             }
+            if (!state) {
+                return {
+                    redirect_url: (0, utils_1.OAuthErrorResponse)({
+                        error: 'invalid_request',
+                        error_description: 'Please specify a state to safeguard against XSRF attacks',
+                        redirect_uri,
+                    }),
+                };
+            }
+            if (response_type !== 'code') {
+                return {
+                    redirect_url: (0, utils_1.OAuthErrorResponse)({
+                        error: 'unsupported_response_type',
+                        error_description: 'Only Authorization Code grant is supported',
+                        redirect_uri,
+                    }),
+                };
+            }
             let ssoUrl;
             let post = false;
             const { sso } = samlConfig.idpMetadata;
@@ -235,60 +252,80 @@ class OAuthController {
                 ssoUrl = sso.postUrl;
                 post = true;
             }
-            const samlReq = saml20_1.default.request({
-                ssoUrl,
-                entityID: this.opts.samlAudience,
-                callbackUrl: this.opts.externalUrl + this.opts.samlPath,
-                signingKey: samlConfig.certs.privateKey,
-                publicKey: samlConfig.certs.publicKey,
-            });
-            const sessionId = crypto_1.default.randomBytes(16).toString('hex');
-            const requested = { client_id, state };
-            if (requestedTenant) {
-                requested.tenant = requestedTenant;
-            }
-            if (requestedProduct) {
-                requested.product = requestedProduct;
+            else {
+                return {
+                    redirect_url: (0, utils_1.OAuthErrorResponse)({
+                        error: 'invalid_request',
+                        error_description: 'SAML binding could not be retrieved',
+                        redirect_uri,
+                    }),
+                };
             }
-            if (idp_hint) {
-                requested.idp_hint = idp_hint;
-            }
-            yield this.sessionStore.put(sessionId, {
-                id: samlReq.id,
-                redirect_uri,
-                response_type,
-                state,
-                code_challenge,
-                code_challenge_method,
-                requested,
-            });
-            const relayState = utils_1.relayStatePrefix + sessionId;
-            let redirectUrl;
-            let authorizeForm;
-            if (!post) {
-                // HTTP Redirect binding
-                redirectUrl = redirect.success(ssoUrl, {
-                    RelayState: relayState,
-                    SAMLRequest: Buffer.from(yield deflateRawAsync(samlReq.request)).toString('base64'),
+            try {
+                const samlReq = saml20_1.default.request({
+                    ssoUrl,
+                    entityID: this.opts.samlAudience,
+                    callbackUrl: this.opts.externalUrl + this.opts.samlPath,
+                    signingKey: samlConfig.certs.privateKey,
+                    publicKey: samlConfig.certs.publicKey,
                 });
+                const sessionId = crypto_1.default.randomBytes(16).toString('hex');
+                const requested = { client_id, state };
+                if (requestedTenant) {
+                    requested.tenant = requestedTenant;
+                }
+                if (requestedProduct) {
+                    requested.product = requestedProduct;
+                }
+                if (idp_hint) {
+                    requested.idp_hint = idp_hint;
+                }
+                yield this.sessionStore.put(sessionId, {
+                    id: samlReq.id,
+                    redirect_uri,
+                    response_type,
+                    state,
+                    code_challenge,
+                    code_challenge_method,
+                    requested,
+                });
+                const relayState = utils_1.relayStatePrefix + sessionId;
+                let redirectUrl;
+                let authorizeForm;
+                if (!post) {
+                    // HTTP Redirect binding
+                    redirectUrl = redirect.success(ssoUrl, {
+                        RelayState: relayState,
+                        SAMLRequest: Buffer.from(yield deflateRawAsync(samlReq.request)).toString('base64'),
+                    });
+                }
+                else {
+                    // HTTP POST binding
+                    authorizeForm = saml20_1.default.createPostForm(ssoUrl, [
+                        {
+                            name: 'RelayState',
+                            value: relayState,
+                        },
+                        {
+                            name: 'SAMLRequest',
+                            value: Buffer.from(samlReq.request).toString('base64'),
+                        },
+                    ]);
+                }
+                return {
+                    redirect_url: redirectUrl,
+                    authorize_form: authorizeForm,
+                };
+            }
+            catch (err) {
+                return {
+                    redirect_url: (0, utils_1.OAuthErrorResponse)({
+                        error: 'server_error',
+                        error_description: (0, utils_1.getErrorMessage)(err),
+                        redirect_uri,
+                    }),
+                };
             }
-            else {
-                // HTTP POST binding
-                authorizeForm = saml20_1.default.createPostForm(ssoUrl, [
-                    {
-                        name: 'RelayState',
-                        value: relayState,
-                    },
-                    {
-                        name: 'SAMLRequest',
-                        value: Buffer.from(samlReq.request).toString('base64'),
-                    },
-                ]);
-            }
-            return {
-                redirect_url: redirectUrl,
-                authorize_form: authorizeForm,
-            };
         });
     }
     samlResponse(body) {
@@ -346,10 +383,27 @@ class OAuthController {
                 thumbprint: samlConfig.idpMetadata.thumbprint,
                 audience: this.opts.samlAudience,
             };
+            if (session && session.redirect_uri && !allowed.redirect(session.redirect_uri, samlConfig.redirectUrl)) {
+                throw new error_1.JacksonError('Redirect URL is not allowed.', 403);
+            }
             if (session && session.id) {
                 validateOpts.inResponseTo = session.id;
             }
-            const profile = yield validateResponse(rawResponse, validateOpts);
+            let profile;
+            const redirect_uri = (session && session.redirect_uri) || samlConfig.defaultRedirectUrl;
+            try {
+                profile = yield validateResponse(rawResponse, validateOpts);
+            }
+            catch (err) {
+                // return error to redirect_uri
+                return {
+                    redirect_url: (0, utils_1.OAuthErrorResponse)({
+                        error: 'access_denied',
+                        error_description: (0, utils_1.getErrorMessage)(err),
+                        redirect_uri,
+                    }),
+                };
+            }
             // store details against a code
             const code = crypto_1.default.randomBytes(20).toString('hex');
             const codeVal = {
@@ -361,9 +415,18 @@ class OAuthController {
             if (session) {
                 codeVal.session = session;
             }
-            yield this.codeStore.put(code, codeVal);
-            if (session && session.redirect_uri && !allowed.redirect(session.redirect_uri, samlConfig.redirectUrl)) {
-                throw new error_1.JacksonError('Redirect URL is not allowed.', 403);
+            try {
+                yield this.codeStore.put(code, codeVal);
+            }
+            catch (err) {
+                // return error to redirect_uri
+                return {
+                    redirect_url: (0, utils_1.OAuthErrorResponse)({
+                        error: 'server_error',
+                        error_description: (0, utils_1.getErrorMessage)(err),
+                        redirect_uri,
+                    }),
+                };
             }
             const params = {
                 code,
@@ -371,7 +434,7 @@ class OAuthController {
             if (session && session.state) {
                 params.state = session.state;
             }
-            const redirectUrl = redirect.success((session && session.redirect_uri) || samlConfig.defaultRedirectUrl, params);
+            const redirectUrl = redirect.success(redirect_uri, params);
             // delete the session
             try {
                 yield this.sessionStore.delete(RelayState);

package/dist/controller/utils.d.ts

@@ -1,6 +1,9 @@
+import type { OAuthErrorHandlerParams } from '../typings';
 export declare enum IndexNames {
     EntityID = "entityID",
     TenantProduct = "tenantProduct"
 }
 export declare const relayStatePrefix = "boxyhq_jackson_";
 export declare const validateAbsoluteUrl: (url: any, message: any) => void;
+export declare const OAuthErrorResponse: ({ error, error_description, redirect_uri }: OAuthErrorHandlerParams) => string;
+export declare function getErrorMessage(error: unknown): string;

package/dist/controller/utils.js

@@ -1,7 +1,31 @@
 "use strict";
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || function (mod) {
+    if (mod && mod.__esModule) return mod;
+    var result = {};
+    if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
+    __setModuleDefault(result, mod);
+    return result;
+};
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.validateAbsoluteUrl = exports.relayStatePrefix = exports.IndexNames = void 0;
+exports.getErrorMessage = exports.OAuthErrorResponse = exports.validateAbsoluteUrl = exports.relayStatePrefix = exports.IndexNames = void 0;
 const error_1 = require("./error");
+const redirect = __importStar(require("./oauth/redirect"));
 var IndexNames;
 (function (IndexNames) {
     IndexNames["EntityID"] = "entityID";
@@ -17,3 +41,14 @@ const validateAbsoluteUrl = (url, message) => {
     }
 };
 exports.validateAbsoluteUrl = validateAbsoluteUrl;
+const OAuthErrorResponse = ({ error, error_description, redirect_uri }) => {
+    return redirect.success(redirect_uri, { error, error_description });
+};
+exports.OAuthErrorResponse = OAuthErrorResponse;
+// https://kentcdodds.com/blog/get-a-catch-block-error-message-with-typescript
+function getErrorMessage(error) {
+    if (error instanceof Error)
+        return error.message;
+    return String(error);
+}
+exports.getErrorMessage = getErrorMessage;

package/dist/typings.d.ts

@@ -52,6 +52,7 @@ export interface OAuthReqBody {
     tenant?: string;
     product?: string;
     access_type?: string;
+    scope?: string;
     code_challenge: string;
     code_challenge_method: 'plain' | 'S256' | '';
     provider: 'saml';
@@ -79,6 +80,7 @@ export interface Profile {
     email: string;
     firstName: string;
     lastName: string;
+    requested: Record<string, string>;
 }
 export interface Index {
     name: string;
@@ -160,4 +162,9 @@ export interface ILogoutController {
     }>;
     handleResponse(body: SAMLResponsePayload): Promise<any>;
 }
+export interface OAuthErrorHandlerParams {
+    error: 'invalid_request' | 'access_denied' | 'unauthorized_client' | 'unsupported_response_type' | 'invalid_scope' | 'server_error' | 'temporarily_unavailable';
+    error_description: string;
+    redirect_uri: string;
+}
 export {};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@boxyhq/saml-jackson",
-  "version": "1.0.3",
+  "version": "1.0.4",
   "description": "SAML Jackson library",
   "keywords": [
     "SAML 2.0"
@@ -38,12 +38,12 @@
   "dependencies": {
     "@boxyhq/saml20": "1.0.2",
     "@opentelemetry/api-metrics": "0.27.0",
-    "@peculiar/webcrypto": "1.3.3",
-    "@peculiar/x509": "1.6.1",
-    "mongodb": "4.5.0",
+    "@peculiar/webcrypto": "1.4.0",
+    "@peculiar/x509": "1.6.3",
+    "mongodb": "4.6.0",
     "mysql2": "2.3.3",
     "pg": "8.7.3",
-    "redis": "4.1.0",
+    "redis": "4.0.6",
     "reflect-metadata": "0.1.13",
     "ripemd160": "2.0.2",
     "typeorm": "0.3.6",
@@ -51,11 +51,11 @@
     "xmlbuilder": "15.1.1"
   },
   "devDependencies": {
-    "@types/node": "17.0.31",
+    "@types/node": "17.0.34",
     "@types/sinon": "10.0.11",
     "@types/tap": "15.0.7",
-    "@typescript-eslint/eslint-plugin": "5.23.0",
-    "@typescript-eslint/parser": "5.23.0",
+    "@typescript-eslint/eslint-plugin": "5.25.0",
+    "@typescript-eslint/parser": "5.25.0",
     "cross-env": "7.0.3",
     "eslint": "8.15.0",
     "eslint-config-prettier": "8.5.0",