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.

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

Added field related.user #693

Closed
wants to merge 2 commits into from
Closed

Added field related.user #693

wants to merge 2 commits into from

Conversation

janniten
Copy link
Contributor

From discussions #589 and #678 and from working with windows events elastic/beats#13530 in the winlogbeat security module many events has related users.
In spite of what naming convention/schema will be chosen to represent n-ary user relationships it would be useful to have the related.user field in order to facilitate pivoting.

level: extended
type: keyword
description: >
All of the users seen on your event.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should clarify here that it's about user names.

@webmat
Copy link
Contributor

webmat commented Dec 13, 2019

Thanks for opening this @janniten!

In making the adjustment above, please also make sure to merge the most recent master. I think you started from an old version of the master branch. For example, the schema.json file should no longer be there, it was removed a little while ago :-)

@janniten janniten closed this Dec 16, 2019
@janniten janniten deleted the related-user branch December 16, 2019 11:21
@janniten
Copy link
Contributor Author

@webmat
My fork was out of date and I wasn´t able to merge with the master. (probably due to my poor knowledge of git )
I have created a new PR, sorry :(

@webmat
Copy link
Contributor

webmat commented Dec 16, 2019

No worries, I can review #694 just as easily :-)

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