From a113681f1f05fd145aa5fe9e43a6f15d859a9310 Mon Sep 17 00:00:00 2001 From: uowis Date: Thu, 9 May 2024 18:04:34 +0200 Subject: [PATCH] fix linting warning --- app/server/lib/OIDCConfig.ts | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/app/server/lib/OIDCConfig.ts b/app/server/lib/OIDCConfig.ts index d4414c72..05c92fac 100644 --- a/app/server/lib/OIDCConfig.ts +++ b/app/server/lib/OIDCConfig.ts @@ -176,7 +176,7 @@ export class OIDCConfig { const amr = tokenSet.claims().amr; if (this._forceMfa && (!amr || !amr.includes("mfa"))) { if (!amr) { - throw new Error('OIDCConfig: could not verify mfa status due to missing amr claim. Make sure your IDP returns it.'); + throw new Error('OIDCConfig: could not verify mfa status due to missing amr claim.'); } else if (!amr.includes("mfa")) { log.error(`OIDCConfig: multi-factor-authentication is not enabled for ${userInfo.email}.`); delete mreq.session.oidc; @@ -184,7 +184,9 @@ export class OIDCConfig { // Convert absolute URL into relative, since it will be prefixed further down the line const targetURL = new URL(targetUrl as string); let targetUrlRelative = targetURL.pathname; - if (targetURL.searchParams.toString()) targetUrlRelative += "?" + targetURL.searchParams.toString(); + if (targetURL.searchParams.toString()) { + targetUrlRelative += "?" + targetURL.searchParams.toString(); + } res.redirect(`/login/error/mfa-not-enabled?next=${targetUrlRelative}`); return;