Merge pull request #12 from mcanoy/exception-handling-on-start

Fix url to engagement svc. Catch api exception and log out but don't crash
rht-labs · Dec 17, 2021 · 46999a1 · 46999a1
2 parents a0e91cf + da6d7ae
commit 46999a1
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/main/java/com/redhat/labs/lodestar/service/GitlabHookService.java b/src/main/java/com/redhat/labs/lodestar/service/GitlabHookService.java
@@ -14,6 +14,7 @@
 import javax.enterprise.event.Observes;
 import javax.inject.Inject;
 import javax.ws.rs.ProcessingException;
+import javax.ws.rs.WebApplicationException;
 import java.util.List;
 
 /**
@@ -56,8 +57,8 @@ void loadModifiedWebhooks() {
             try {
                 engagementApiRestClient.updateWebhooks(gitlabHookConfiguration.getHooks());
                 LOGGER.debug("gitlab webhooks updated.");
-            } catch (ProcessingException ex) { //Only needed until engagements comes online
-                LOGGER.error(ex.getMessage());
+            } catch (ProcessingException | WebApplicationException ex) { //Only needed until engagements comes online
+                LOGGER.error("This should occur before v2", ex.getMessage());
             }
         }
     }