Fix OIDC looping issue - too many redriects (opensearch-project#1014)

Signed-off-by: Aozixuan Priscilla Guan <[email protected]> Signed-off-by: Vasile Negru <[email protected]>
spartan2015 · Aug 8, 2022 · 08092a4 · 08092a4
1 parent 40478f5
commit 08092a4
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/server/auth/types/openid/routes.ts b/server/auth/types/openid/routes.ts
@@ -178,8 +178,11 @@ export class OpenIdAuthRoutes {
  });
  } catch (error) {
  context.security_plugin.logger.error(`OpenId authentication failed: ${error}`);
- // redirect to login
- return this.redirectToLogin(request, response);
+ if (error.toString().toLowerCase().includes('authentication exception')) {
+ return response.unauthorized();
+ } else {
+ return this.redirectToLogin(request, response);
+ }
  }
  }
  );