diff --git a/routes/login.js b/routes/login.js index 6bebc755..0b2953c8 100644 --- a/routes/login.js +++ b/routes/login.js @@ -4,17 +4,19 @@ import * as authenticationFunctions from "../helpers/functions.authentication.js export const router = Router(); const getSafeRedirectURL = (possibleRedirectURL = "") => { const urlPrefix = configFunctions.getProperty("reverseProxy.urlPrefix"); - const urlToCheck = (possibleRedirectURL.startsWith(urlPrefix) ? - possibleRedirectURL.slice(urlPrefix.length) : - possibleRedirectURL).toLowerCase(); - switch (urlToCheck) { - case "/admin/fees": - case "/lotOccupancies": - case "/lots": - case "/maps": - case "/workOrders": - case "/reports": - return urlPrefix + urlToCheck; + if (typeof (possibleRedirectURL) === "string") { + const urlToCheck = (possibleRedirectURL.startsWith(urlPrefix) ? + possibleRedirectURL.slice(urlPrefix.length) : + possibleRedirectURL).toLowerCase(); + switch (urlToCheck) { + case "/admin/fees": + case "/lotOccupancies": + case "/lots": + case "/maps": + case "/workOrders": + case "/reports": + return urlPrefix + urlToCheck; + } } return urlPrefix + "/dashboard"; }; diff --git a/routes/login.ts b/routes/login.ts index 6267a0d4..90252bc8 100644 --- a/routes/login.ts +++ b/routes/login.ts @@ -15,19 +15,22 @@ const getSafeRedirectURL = (possibleRedirectURL = "") => { const urlPrefix = configFunctions.getProperty("reverseProxy.urlPrefix"); - const urlToCheck = (possibleRedirectURL.startsWith(urlPrefix) ? - possibleRedirectURL.slice(urlPrefix.length) : - possibleRedirectURL).toLowerCase(); + if (typeof (possibleRedirectURL) === "string") { + + const urlToCheck = (possibleRedirectURL.startsWith(urlPrefix) ? + possibleRedirectURL.slice(urlPrefix.length) : + possibleRedirectURL).toLowerCase(); - switch (urlToCheck) { - case "/admin/fees": - case "/lotOccupancies": - case "/lots": - case "/maps": - case "/workOrders": - case "/reports": + switch (urlToCheck) { + case "/admin/fees": + case "/lotOccupancies": + case "/lots": + case "/maps": + case "/workOrders": + case "/reports": - return urlPrefix + urlToCheck; + return urlPrefix + urlToCheck; + } } return urlPrefix + "/dashboard";