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

Account for Multiple Users in an Event #234

Closed
eternalyperplxed opened this issue Dec 5, 2018 · 9 comments
Closed

Account for Multiple Users in an Event #234

eternalyperplxed opened this issue Dec 5, 2018 · 9 comments
Labels
discuss enhancement New feature or request

Comments

@eternalyperplxed
Copy link

Some events, specifically Active Directory change events (event.id 5136) may have multiple user names in the event. The user making the change, and the user being changed. Perhaps user.source and user.target could be used to differentiate?

@MikePaquette
Copy link
Contributor

@eternalyperplxed ECS allows for information from multiple users to be included the same event. The user.* fields are "re-usable" in the sense that they can be used "under" other objects, such as source.user.* , destination.user.* , host.user.*. So the mechanics of how to include multiple users in the same event are well defined.

See the user section of the readme file here.

However, your use case raises an interesting follow-on question - in which user fields should we populate which user's details?

Here's one way to approach it. Perhaps others will have better ideas.

  • The host.* on which the event happened is the AD server.
  • The host.user.* is the user that made the event happen (e.g. "administrator")
  • The username of the account that was changed could be added under related.user.* (Note this is not yet explicitly defined in ECS)

Logically, it could look like this:
image

@eternalyperplxed
Copy link
Author

Thanks @MikePaquette this makes sense to me and I agree with the follow on question. I believe there are some instances where an Active Directory event could have something like 6 different users contained within it, so trying to find a common ground to account for them is an interesting challenge.

@MikePaquette
Copy link
Contributor

@eternalyperplxed yes, indeed, but note that the related.* fields are expected to be arrays, so adding multiple values to each related.user field might help address this problem.

@webmat
Copy link
Contributor

webmat commented Dec 10, 2018

Thanks for submitting this question, @eternalyperplxed.

I also agree that user management use cases will need a clear way to define which user is doing the change and which user(s) are affected.

Right now I would say that related.* is not semantically the right place to officially list affected users. This is a place where we expect all values of one kind to be added to the array, to simplify pivoting (in the case of usernames, we should add the administrator name as well as the affected user names).

So I do think we'll need to work on an official place to list the affected users.

@webmat webmat added enhancement New feature or request discuss labels Dec 10, 2018
@eternalyperplxed
Copy link
Author

Understood @webmat Thanks for looking into this!

@MikePaquette
Copy link
Contributor

@webmat I agree we need to find an official place for the affected user.* info, but once we do that, I think that the related.user.* will be a very good fit for a copy of it. Of course, the host.user.* fields would be copied there as well, consistent with the way we've envisioned and specified the use of other related.* fields.

@vbohata
Copy link

vbohata commented Apr 26, 2019

I think new 2 top level fields - old/new should help in situations like changing user name and more (so new.user.name for new username, for file path new.file.path, ...).

@neu5ron
Copy link

neu5ron commented Apr 29, 2019

I would hope we could use source.user and destination AND or create a target field thus using target.user

Multiple usernames in a single field I would imagine is ok, because elasticsearch handles arrays well. Now if a category has multiple distinct fields like an SMTP log that has from,to,cc, etc.

@webmat
Copy link
Contributor

webmat commented Nov 3, 2020

Closing in favor of #1066 (see also RFC 0007 stage 3 PR #1017)

@webmat webmat closed this as completed Nov 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discuss enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

5 participants