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

adding status_id IDs #1076

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

adding status_id IDs #1076

wants to merge 4 commits into from

Conversation

SashaSelin
Copy link

Related Issue:

Missing enable/disable status Ids

Description of changes:

added status id's to status_id in dictionary.

Signed-off-by: Sasha Selin (Cyrebro) (sasha.selin@cyrebro.io)

Signed-off-by: SashaSelin <145011693+SashaSelin@users.noreply.github.com>
Signed-off-by: SashaSelin <145011693+SashaSelin@users.noreply.github.com>
Hello Team, 
I would like to add more status_id captions, 

Thanks! 


Signed-off-by: Sasha Selin  sasha.selin@cyrebro.io

Signed-off-by: SashaSelin <145011693+SashaSelin@users.noreply.github.com>
Improved Base Event [0] Class  (status_id)

Signed-off-by: SashaSelin <145011693+SashaSelin@users.noreply.github.com>
@floydtree
Copy link
Contributor

Can you provide some context on how these new enum items will be used? Throughout ocsf status attributes have been used to represent result of an activity, whereas enable/disable sound like activities themselves. Some context around the use-case will help move this along.

@SashaSelin
Copy link
Author

SashaSelin commented May 8, 2024 via email

@mikeradka
Copy link
Contributor

I think Enable and Disable are not quite representative of statuses, but rather actions. My understanding of status is that it represents state, rather than a verb. @floydtree @pagbabian-splunk @zschmerber what are your thoughts? Perhaps the definitions for status and status_id could be refined?

@pagbabian-splunk
Copy link
Contributor

pagbabian-splunk commented May 17, 2024

This came up recently elsewhere - @SashaSelin note that there is also a state_id and state sibling for the state of something independent of action, activity or lifecycle. Note there are also specialized *_state attributes too in certain cases.

Although Fortigate logs might be using the name status for this, in OCSF, for better or worse, we have tried to keep status the result of an activity that was performed. I do think we are missing a more general Configuration Change class.

Enable/Disable would be verbs which as mentioned would normally be actions/activities, the result of which would usually be Success/Failure. status_id can be overridden as it is in the Finding classes, but there status_id follows the lifecycle of the Finding.

(Configuration value changed) status

In this case, as one option, the activity might be 'configuration changed'; the status would be Success, and the state would be Enabled.

I noticed that we don't have any Enabled/Disabled states that I could see, and therefore @SashaSelin if you know what class you would be using, the state or status can be overridden to be Enabled/Disabled, or if you need new activity_ids for an existing class (e.g. Account Change) we can add Enable/Disable (note the verb form).

@SashaSelin
Copy link
Author

SashaSelin commented May 19, 2024 via email

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