hotfix: include IllegalArgumentException in caught exceptions for ico…

…n loading
airbytehq · May 25, 2021 · d2c90de · d2c90de
1 parent 43b13eb
commit d2c90de
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/airbyte-server/src/main/java/io/airbyte/server/handlers/DestinationDefinitionsHandler.java b/airbyte-server/src/main/java/io/airbyte/server/handlers/DestinationDefinitionsHandler.java
@@ -164,7 +164,7 @@ public DestinationDefinitionRead updateDestinationDefinition(DestinationDefiniti
   public static String loadIcon(String name) {
     try {
       return name == null ? null : MoreResources.readResource("icons/" + name);
-    } catch (IOException e) {
+    } catch (Exception e) {
       return null;
     }
   }

diff --git a/airbyte-server/src/main/java/io/airbyte/server/handlers/SourceDefinitionsHandler.java b/airbyte-server/src/main/java/io/airbyte/server/handlers/SourceDefinitionsHandler.java
@@ -161,7 +161,7 @@ public SourceDefinitionRead updateSourceDefinition(SourceDefinitionUpdate source
   public static String loadIcon(String name) {
     try {
       return name == null ? null : MoreResources.readResource("icons/" + name);
-    } catch (IOException e) {
+    } catch (Exception e) {
       return null;
     }
   }