Fix NPE's trying to cache profiles & when there are no blacklisted paths

src/main/java/com/botdarr/Config.java

@@ -116,10 +116,13 @@ public static ChatClientType getChatClientType() {
 
   public static List<String> getExistingItemBlacklistPaths() {
     String paths = getProperty(Constants.EXISTING_ITEMS_PATHS_BLACKLIST);
-    if (paths != null && paths.contains(",")) {
-      return Arrays.asList(paths.split(","));
+    if (paths != null) {
+      if (paths.contains(",")) {
+        return Arrays.asList(paths.split(","));
+      }
+      return new ArrayList<String>() {{add(paths);}};
     }
-    return new ArrayList<String>() {{add(paths);}};
+    return new ArrayList<>();
   }
 
   public static final class Constants {

src/main/java/com/botdarr/api/AddStrategy.java

@@ -44,7 +44,7 @@ public ChatClientResponse addWithSearchId(String searchText, String id) {
         }
       }
       return chatClientResponseBuilder.createErrorMessage("Could not find " + contentDisplayName + " with search text=" + searchText + " and id=" + id);
-    } catch (Exception e) {
+    } catch (Throwable e) {
       LOGGER.error("Error trying to add " + contentDisplayName, e);
       return chatClientResponseBuilder.createErrorMessage("Error adding content, e=" + e.getMessage());
     }
@@ -80,7 +80,7 @@ public List<ChatClientResponse> addWithSearchTitle(String searchText) {
         return Arrays.asList(chatClientResponseBuilder.createInfoMessage("No new " + contentDisplayName + "s found, check existing " + contentDisplayName + "s"));
       }
       return restOfItems;
-    } catch (Exception e) {
+    } catch (Throwable e) {
       LOGGER.error("Error trying to add " + contentDisplayName, e);
       return Arrays.asList(chatClientResponseBuilder.createErrorMessage("Error trying to add " + contentDisplayName + ", for search text=" + searchText + ", e=" + e.getMessage()));
     }

src/main/java/com/botdarr/api/CacheProfileStrategy.java

@@ -1,5 +1,8 @@
 package com.botdarr.api;
 
+import org.apache.logging.log4j.LogManager;
+import org.apache.logging.log4j.Logger;
+
 import java.util.ArrayList;
 import java.util.List;
 
@@ -11,10 +14,15 @@ public abstract class CacheProfileStrategy<Z extends KeyBased<X>, X> {
   public void cacheData() {
     List<X> profilesAddUpdated = new ArrayList<>();
     List<Z> profiles = getProfiles();
+    if (profiles == null) {
+      LOGGER.warn("Did not find any profiles available for caching, class=" + this.getClass().toString());
+      return;
+    }
     for (Z profile : profiles) {
       addProfile(profile);
       profilesAddUpdated.add(profile.getKey());
     }
     deleteFromCache(profilesAddUpdated);
   }
+  private static final Logger LOGGER = LogManager.getLogger();
 }

src/main/java/com/botdarr/commands/CommandProcessor.java

@@ -40,7 +40,7 @@ public <T extends ChatClientResponse, Z extends Api> CommandResponse processMess
         return new CommandResponse(chatClientResponseBuilder.createErrorMessage("Invalid command - type " + commandPrefix + "help for command usage"));
       }
 
-    } catch (Exception e) {
+    } catch (Throwable e) {
       LOGGER.error("Error trying to execute command " + strippedMessage, e);
       return new CommandResponse(chatClientResponseBuilder.createErrorMessage("Error trying to parse command " + strippedMessage  + ", error=" + e.getMessage()));
     }

src/main/java/com/botdarr/connections/ConnectionHelper.java

@@ -84,7 +84,7 @@ public List<ChatClientResponse> onFailure(int statusCode, String reason) {
 
     @Override
     public List<ChatClientResponse> onException(Exception e) {
-      return Arrays.asList(chatClientResponseBuilder.createErrorMessage("Requested failed with exception, e=" + e.getMessage()));
+      return Arrays.asList(chatClientResponseBuilder.createErrorMessage("Requested failed with exception, e=" + e.getMessage() + ",class=" + e.getClass()));
     }
     private ChatClientResponseBuilder<? extends ChatClientResponse> chatClientResponseBuilder;
   }

src/main/resources/version.txt

@@ -1 +1 @@
-5.1.4
+5.1.5