-
Notifications
You must be signed in to change notification settings - Fork 418
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
[RFC] added new field: threat.indicator.id - resolves GH-2252 #2307
Conversation
@ebeahan This should be good to go and ready for discussion. Let me know if there are any issues with the PR! 😄 |
This PR is stale because it has been open for 60 days with no activity. |
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.
lgtm
Looks ok to me but I would like to have another approval from ecs maintainers / security folks |
@mjwolf could you check it as well? |
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.
Hi @brett-fitz, I just have a few minor suggestions, otherwise LGTM
@trisch-me @mjwolf My organization has dissolved and formed into a new entity. I'm going to resubmit this PR under a branch off my personal fork with the requested changes from above. |
Closing this PR. Now please refer to the new one: #2324 |
Added
threat.indicator.id
field. Resolves #2252.The new field
threat.indicator.id
will allow for security systems to append a threat.indicator.id. This field can have multiple values to allow for the identification of the same indicator across systems that use different ID formats.Common serialization format you may expect to see here is a STIX 2.x indicator id. Here is an example of one being produced.