From e7b5cf888f1b28a44d696ae0ceb0a0906bc53f2f Mon Sep 17 00:00:00 2001 From: Himanshu Mishra Date: Thu, 24 Jun 2021 11:32:57 +0200 Subject: [PATCH] remove unnecessary check for missing provider factory Signed-off-by: Himanshu Mishra --- plugins/auth-backend/src/service/router.ts | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/plugins/auth-backend/src/service/router.ts b/plugins/auth-backend/src/service/router.ts index 1945e6cb2d..9841df399a 100644 --- a/plugins/auth-backend/src/service/router.ts +++ b/plugins/auth-backend/src/service/router.ts @@ -88,15 +88,12 @@ export async function createRouter({ const providersConfig = config.getConfig('auth.providers'); const configuredProviders = providersConfig.keys(); - for (const providerId of Object.keys(allProviderFactories)) { + for (const [providerId, providerFactory] of Object.entries( + allProviderFactories, + )) { if (configuredProviders.includes(providerId)) { logger.info(`Configuring provider, ${providerId}`); try { - const providerFactory = allProviderFactories[providerId]; - if (!providerFactory) { - throw Error(`No auth provider available for '${providerId}'`); - } - const provider = providerFactory({ providerId, globalConfig: { baseUrl: authUrl, appUrl },