Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Introduce threat.indicator.name field #2121

Merged
merged 3 commits into from
Jan 5, 2023
Merged

Introduce threat.indicator.name field #2121

merged 3 commits into from
Jan 5, 2023

Conversation

lgestc
Copy link
Contributor

@lgestc lgestc commented Dec 13, 2022

This PR addresses issue #1998

@lgestc lgestc requested a review from a team as a code owner December 13, 2022 09:38
@kgeller
Copy link
Contributor

kgeller commented Dec 15, 2022

@lgmys this looks great, just one question. Were we still planning on adding threat.enrichments.indicator.name as well? I see it noted #1998 (comment)

@lgestc
Copy link
Contributor Author

lgestc commented Dec 16, 2022 •
edited
Loading

@lgmys this looks great, just one question. Were we still planning on adding threat.enrichments.indicator.name as well? I see it noted #1998 (comment)

Right, let me add that as well

Edit: @kgeller done!

@kgeller kgeller added the 8.7.0 label Dec 19, 2022
@lgestc lgestc merged commit 753a893 into elastic:main Jan 5, 2023
@andrewkroh andrewkroh mentioned this pull request Apr 3, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kgeller kgeller kgeller approved these changes

Assignees
No one assigned
Labels
8.7.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@lgestc @kgeller @lgmys