Skip to content

Commit

Permalink
fixing this - not an issue, rolling back
Browse files Browse the repository at this point in the history
  • Loading branch information
testaccount90009 committed Aug 21, 2024
1 parent 2ca7ba9 commit f58846a
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion dojo/tools/asff/parser.py
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ def get_findings(self, file, test):
control_description = item.get("Description")

if resource_arns:
resource_arn_strings = " + ".join(resource_arns)
resource_arn_strings = ", ".join(resource_arns)
full_description = f"**AWS resource ARN:** {resource_arn_strings}\n\n{control_description}"
else:
full_description = control_description
Expand Down
2 changes: 1 addition & 1 deletion unittests/tools/test_asff_parser.py
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ def load_sample_json(self, file_name):
def common_check_finding(self, finding, data, index, guarddutydate=False):
parser = AsffParser()
resource_arns = parser.get_item_resource_arns(data[index])
resource_arn_strings = " + ".join(resource_arns)
resource_arn_strings = ", ".join(resource_arns)
control_description = data[index].get("Description", "")
full_description = f"**AWS resource ARN:** {resource_arn_strings}\n\n{control_description}"
self.assertEqual(finding.title, data[index]["Title"])
Expand Down

0 comments on commit f58846a

Please sign in to comment.