address review comments

Signed-off-by: Surya Sashank Nistala <snistala@amazon.com>
eirsep · Jul 9, 2024 · bfe330d · bfe330d
1 parent bbe005d
commit bfe330d
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 5 deletions.
diff --git a/src/main/java/org/opensearch/securityanalytics/services/STIX2IOCFetchService.java b/src/main/java/org/opensearch/securityanalytics/services/STIX2IOCFetchService.java
@@ -270,7 +270,8 @@ public void downloadFromUrlAndIndexIOCs(SATIFSourceConfig saTifSourceConfig, Act
                 }
                 break;
             default:
-                // if the feed type doesn't match any of the supporting feed types, throw an exception
+                log.error("unsupported feed format for url download:" + source.getFeedFormat());
+                listener.onFailure(new UnsupportedOperationException("unsupported feed format for url download:" + source.getFeedFormat()));
         }
     }
 

diff --git a/...in/java/org/opensearch/securityanalytics/threatIntel/common/SourceConfigDtoValidator.java b/...in/java/org/opensearch/securityanalytics/threatIntel/common/SourceConfigDtoValidator.java
@@ -45,10 +45,10 @@ public List<String> validateSourceConfigDto(SATIFSourceConfigDto sourceConfigDto
                 break;
             case URL_DOWNLOAD:
                 if (sourceConfigDto.getSchedule() == null) {
-                    errorMsgs.add("Must pass in schedule for URL_DONWLOAD source type");
+                    errorMsgs.add("Must pass in schedule for URL_DOWNLOAD source type");
                 }
                 if (sourceConfigDto.getSource() != null && sourceConfigDto.getSource() instanceof UrlDownloadSource == false) {
-                    errorMsgs.add("Source must be URL_DONWLOAD source type");
+                    errorMsgs.add("Source must be URL_DOWNLOAD source type");
                 }
                 break;
         }

diff --git a/...opensearch/securityanalytics/threatIntel/service/DefaultTifSourceConfigLoaderService.java b/...opensearch/securityanalytics/threatIntel/service/DefaultTifSourceConfigLoaderService.java
@@ -46,7 +46,7 @@ public DefaultTifSourceConfigLoaderService(BuiltInTIFMetadataLoader tifMetadataL
     }
 
     /**
-     * check if the default
+     * check if the default tif source configs are loaded. if not, try create them from the feedMetadata.json file.
      */
     public void createDefaultTifConfigsIfNotExists(ActionListener<Void> listener) {
         List<TIFMetadata> tifMetadataList = tifMetadataLoader.getTifMetadataList();

diff --git a/src/main/java/org/opensearch/securityanalytics/threatIntel/service/TIFJobUpdateService.java b/src/main/java/org/opensearch/securityanalytics/threatIntel/service/TIFJobUpdateService.java
@@ -187,7 +187,7 @@ public void onFailure(Exception e) {
                                         }
                                         break;
                                     default:
-                                        // if the feed type doesn't match any of the supporting feed types, throw an exception
+                                        onFailure(new UnsupportedOperationException("Not a supported feed format : " + tifMetadata.getFeedType()));
                                 }
                             }
                         } catch (IOException ex) {