Slightly more changes to support small ta object

Author: pyth0n1c

contentctl/actions/validate.py

@@ -38,10 +38,8 @@ def execute(self, input_dto: validate) -> DirectorOutputDto:
         director.execute(input_dto)
         self.ensure_no_orphaned_files_in_lookups(input_dto.path, director_output_dto)
         if input_dto.data_source_TA_validation:
-            if self.validate_latest_TA_information(director_output_dto.data_sources) != 1:
-                print("All TA versions are up to date.")
-            else:
-                raise Exception("One or more TA versions are out of date. Please update the data source with the latest version.")
+            self.validate_latest_TA_information(director_output_dto.data_sources)
+            
         return director_output_dto
 
 
@@ -83,9 +81,9 @@ def ensure_no_orphaned_files_in_lookups(self, repo_path:pathlib.Path, director_o
         return
 
 
-    def validate_latest_TA_information(self, data_sources: list[DataSource]) -> int:
+    def validate_latest_TA_information(self, data_sources: list[DataSource]) -> None:
         validated_TAs: list[tuple[str, str]] = []
-        error_occurred = False
+        errors:list[str] = []
         print("----------------------")
         print("Validating latest TA:")
         print("----------------------")
@@ -100,10 +98,18 @@ def validate_latest_TA_information(self, data_sources: list[DataSource]) -> int:
                     try:
                         splunk_app = SplunkApp(app_uid=uid)
                         if splunk_app.latest_version != supported_TA.version:
-                            raise Exception(f"Version mismatch for TA {supported_TA.name}: "
-                                            f"Latest version on Splunkbase is {splunk_app.latest_version}, "
-                                            f"but version {supported_TA.version} is specified in the data source {data_source.name}.")
+                            errors.append(f"Version mismatch in '{data_source.file_path}' supported TA '{supported_TA.name}'"
+                                          f"\n  Latest version on Splunkbase    : {splunk_app.latest_version}"
+                                          f"\n  Version specified in data source: {supported_TA.version}")
                     except Exception as e:
-                        print(f"Error processing TA {supported_TA.version}: {str(e)}")
-                        error_occurred = True
-        return 1 if error_occurred else 0
+                        errors.append(f"Error processing checking version of TA {supported_TA.name}: {str(e)}")
+                            
+        if len(errors) > 0:
+            errorString = '\n\n'.join(errors)
+            raise Exception(f"[{len(errors)}] or more TA versions are out of date or have other errors."
+                            f"Please update the following data sources with the latest versions of "
+                            f"their supported tas:\n\n{errorString}")
+        print("All TA versions are up to date.")
+        
+
+

contentctl/output/data_source_writer.py

@@ -18,10 +18,10 @@ def writeDataSourceCsv(data_source_objects: List[DataSource], file_path: pathlib
             ])
             # Write the data
             for data_source in data_source_objects:
-                if data_source.supported_TA and isinstance(data_source.supported_TA, list) and len(data_source.supported_TA) > 0:
-                    supported_TA_name = data_source.supported_TA[0].get('name', '')
-                    supported_TA_version = data_source.supported_TA[0].get('version', '')
-                    supported_TA_url = data_source.supported_TA[0].get('url', '')
+                if  len(data_source.supported_TA) > 0:
+                    supported_TA_name = data_source.supported_TA[0].name
+                    supported_TA_version = data_source.supported_TA[0].version
+                    supported_TA_url = data_source.supported_TA[0].url or ''
                 else:
                     supported_TA_name = ''
                     supported_TA_version = ''