Catching PA token validation error (finos#760)

* Catching PA token validation error * Update PA token validation logging Co-authored-by: Kevin Knight <[email protected]> --------- Co-authored-by: Kevin Knight <[email protected]>
rafaelbey · Mar 4, 2024 · 3071184 · 3071184
1 parent 83b2df6
commit 3071184
Showing 1 changed file with 11 additions and 1 deletion.
diff --git a/legend-sdlc-server/src/main/java/org/finos/legend/sdlc/server/tools/SessionProvider.java b/legend-sdlc-server/src/main/java/org/finos/legend/sdlc/server/tools/SessionProvider.java
@@ -134,7 +134,17 @@ public static Session getSessionUsingGitlabPersonalAccessToken(HttpServletReques
         GitlabPersonalAccessTokenProfileCreator creator = new GitlabPersonalAccessTokenProfileCreator(client.host);
         GitlabPersonalAccessTokenAuthenticator authenticator = new GitlabPersonalAccessTokenAuthenticator(client.scheme, client.host, client.apiVersion);
         GitlabPersonalAccessTokenCredentials credentials = extractor.extract(context);
-        authenticator.validate(credentials, context);
+
+        try
+        {
+            authenticator.validate(credentials, context);
+        }
+        catch (Exception e)
+        {
+            LOGGER.error("Validation failed for PA token", e);
+            return null;
+        }
+
         GitlabPersonalAccessTokenProfile profile = creator.create(credentials, context);
         return GitLabSessionBuilder.newBuilder(appInfo).withProfile(profile).build();
     }