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

Add user_inventory event class and associated person profile #667

Merged
merged 6 commits into from
Jun 20, 2023

Conversation

mtharp0
Copy link
Contributor

@mtharp0 mtharp0 commented Jun 9, 2023

In an effort to get collected AD type data into OCSF, we are adding a new event class called user_inventory. It would collect a lot more attributes associated with a user (person). So, in addition to the new class there is a new person profile that allows us to dynamically add the extended attributes to the user for the user_inventory class (or anywhere else that someone may want to add these attributes to the user object. In order to allow the user location to be specified without the coordinates in a location, we make the coordinates recommended and add constraints to the location object. Finally, the new attributes are added to the dictionary.

Signed-off-by: Tharp, Matthew <Matthew_Tharp2@comcast.com>
Signed-off-by: Tharp, Matthew <Matthew_Tharp2@comcast.com>
…ocumenting user office location

Signed-off-by: Tharp, Matthew <Matthew_Tharp2@comcast.com>
Signed-off-by: Tharp, Matthew <Matthew_Tharp2@comcast.com>
Signed-off-by: Tharp, Matthew <Matthew_Tharp2@comcast.com>
dictionary.json Show resolved Hide resolved
dictionary.json Outdated Show resolved Hide resolved
dictionary.json Show resolved Hide resolved
events/discovery/user_inventory.json Show resolved Hide resolved
Signed-off-by: Tharp, Matthew <Matthew_Tharp2@comcast.com>
@rroupski rroupski self-requested a review June 14, 2023 21:13
Copy link
Contributor

@pagbabian-splunk pagbabian-splunk left a comment

Choose a reason for hiding this comment

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

Separate from this PR, we need to have better guidelines on how and when to use the various IDs for users and their accounts.

@mtharp0 mtharp0 merged commit 2d60145 into ocsf:main Jun 20, 2023
1 check passed
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.

None yet

4 participants