Skip to content

Print error instead of bailing out on enrichment failure - #2327

Merged
josehelps merged 1 commit into
developfrom
dontfailonenrichment
Aug 16, 2022
Merged

josehelps merged 1 commit into
developfrom
dontfailonenrichment

Conversation

@ljstella

Copy link
Copy Markdown
Contributor

PR Template for new Detections

Details

Going back to printing errors instead of failing on CVE enrichment

Author Checklist

  • Validate name matches <platform>_<mitre att&ck technique>_<short description>
  • Make sure that CI/CD detection-testing and build-and-validate jobs passed ✔️
  • Is there an Atomic Test? Is GUID on the test file under array: atomic_test_guid, example?

Review Checklist

  • Validated SPL logic.
  • Validated tags, description, and how to implement.
  • Verified references match analytic.

@auto-assign
auto-assign Bot requested a review from P4T12ICK August 16, 2022 16:05
@josehelps
josehelps merged commit 3b730ff into develop Aug 16, 2022
@delete-merged-branch
delete-merged-branch Bot deleted the dontfailonenrichment branch August 16, 2022 16:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants