add threat_intel category to integrations - #3689
Conversation
| format_version: 1.0.0 | ||
| license: basic | ||
| categories: ["security", "productivity"] | ||
| categories: ["security", "threat_intel"] |
There was a problem hiding this comment.
I'm not sure why this specific package was listed with productivity even though the other ti-* packages were not. For consistency, I removed it.
| type: integration | ||
| categories: | ||
| - security | ||
| categories: ["security", "threat_intel"] |
There was a problem hiding this comment.
I marked this package with the threat_intel category because it supports pulling data from the Mimecast Threat Intel Feed.
|
@elastic/security-external-integrations did I miss any steps for updating the categories for 8.4? Happy to adjust the PR if needed. |
🌐 Coverage report
|
| @@ -1,3 +1,8 @@ | |||
| - version: "1.1.1" | |||
There was a problem hiding this comment.
Since these are marked as enhancements, they would be minor version bumps rather than patch bumps.
|
Thanks @efd6! I modified the version numbers. @elastic/protections-experience would you mind giving this a once over before I merge? |
PhilippeOberti
left a comment
There was a problem hiding this comment.
LGTM (although not exactly sure how to test this :) )
|
@stevewritescode you're missing ThreatQuotient in your list of TI feeds. Also just as an FYI, Mimecast isn't a full TI integration but they have some data streams that are Threat Intelligence related so good to keep on the list. |
|
Thanks @dhru42! I accidentally left ThreatQuotient out of the list in the PR description, though it is represented in the code changed. I updated the description to match. I will also rebase on the recent mimecast updates before merging. |
5ac6a26 to
6eecaac
Compare
What does this PR do?
As a follow-on to elastic/package-spec#366, this PR adds new
threat_intelcategory to the relevant integrations:Checklist
changelog.ymlfile.Related issues
Relates to elastic/kibana#135758, elastic/kibana#136208