-
Notifications
You must be signed in to change notification settings - Fork 136
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Expand OSINT
, add threat_intelligence
#1255
Conversation
Signed-off-by: Jonathan Rau <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you fix the whitespace issues, and restore pre-existing attributes to their original locations in the file? This would make it much easier to review. Thanks.
@@ -3232,6 +3232,11 @@ | |||
"description": "The number of the entity. See specific usage.", | |||
"type": "integer_t" | |||
}, | |||
"objective": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Term very general. Suggest: threat_objective
"type_id":{ | ||
"caption":"Indicator Type ID", | ||
"description":"The OSINT indicator type ID.", | ||
"requirement":"required", | ||
"requirement":"recommended", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a breaking change. A consumer could previously assume this attribute to be present but, if made optional, it can no longer make that assumption.
"value":{ | ||
"caption":"Indicator", | ||
"description":"The actual indicator value in scope, e.g., a SHA-256 hash hexdigest or a domain name.", | ||
"requirement":"required" | ||
}, | ||
"tlp":{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any reason why these attrbutes have been moved in the file? It makes it difficult for reviewers to see what has been deleted, which would be a breaking change. Would you be able to re-submit this without the movement of attributes?
"description": "The latest time of a given subject of the threat intelligence analysis was encountered, if known.", | ||
"requirement": "optional" | ||
}, | ||
"name": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You editor is inserting tabs. You need to change to spaces.
Description of changes:
Since there was not any appetite for STIX in the past, but there have been asks in the slack channel around more "traditional" elements of cyber threat intelligence such as reporting on the threat actor, campaign, behaviors, or otherwise I took another stab at it.
The Threat Intelligence object is purposely light meant to capture a lot of strings that can be adapted from several platforms. For instance, Crowdstrike Detects from the API provide an array of objects that describe
behaviors
and within each of these a behavior, MITRE tactics and techniques (or internal CRWD ones), scenarios, objective, and description is included (https://assets.falcon.crowdstrike.com/support/api/swagger.html#/detects/GetDetectSummaries)Additionally, other CTI and OSINT tools such as MISP, Shodan Monitor, AlienVault, and Recorded Future provide some other related strings that describe an indicator and its relationships.
This new object is within the overall
osint
object which is intended to be related to the indicator itself, and not the other way around. I can switch this around instead, but then we would have issues with the profile being swapped out.type_id
enums toosint
: Reg Key, Reg Value, Command Line (these all come from Crowdstrikeioc_type
, specifically) but also are present in MDE.related_analytics
toosint
threat_intelligence
object with a newobjective
entry in the dictionary