-
Notifications
You must be signed in to change notification settings - Fork 21
feat: add support for CIM v5.3.2 #853
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
Changes from 7 commits
d574f64
3b4618f
85d48f4
7eee342
77fecf9
b8dddfb
b9f1915
0921ae4
3370a93
de4d0cb
20651c7
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -61,13 +61,13 @@ | |
| { | ||
| "name": "object_path", | ||
| "validity": "if(like(object_path,'%\\\"%'),null(),object_path)", | ||
| "type": "required", | ||
| "type": "optional", | ||
| "comment": "The path of the modified resource object, if applicable (such as a file, directory, or volume)." | ||
| }, | ||
| { | ||
| "name": "result", | ||
| "type": "optional", | ||
| "expected_values": ["lockout"], | ||
| "condition": "status=failure", | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. this is supposed to be the literal string the vendor uses in the log so there no constraint on what may be present. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. So we mark it as optional? @justin-splunk There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The
Is this the current logic? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @alexeisuv |
||
| "type": "conditional", | ||
| "comment": "The vendor-specific result of a change, or clarification of an action status. For instance, status=failure may be accompanied by result=blocked by policy or result=disk full. result is a string. Please use a msg_severity_id field (not included in the data model) for severity ID fields that are integer data types." | ||
| }, | ||
| { | ||
|
|
@@ -77,7 +77,7 @@ | |
| }, | ||
| { | ||
| "name": "src", | ||
| "type": "optional", | ||
| "type": "required", | ||
| "comment": "The resource where the change was originated. You can alias this from more specific fields not included in the data model, such as src_host, src_ip, or src_name." | ||
| }, | ||
| { | ||
|
|
@@ -96,6 +96,31 @@ | |
| "type": "required", | ||
| "comment": "The user or entity performing the change. For account changes, this is the account that was changed (see src_user for user or entity performing the change)." | ||
| }, | ||
| { | ||
| "name": "user_agent", | ||
| "type": "optional", | ||
| "comment": "The user agent through which the request was made, such as Mozilla/5.0 (Macintosh; Intel Mac OS X 10_14_6) or aws-cli/2.0.0 Python/3.7.4 Darwin/18.7.0 botocore/2.0.0dev4." | ||
| }, | ||
| { | ||
| "name": "user_name", | ||
| "type": "optional", | ||
| "comment": "The user name of the user or entity performing the change. For account changes, this is the account that was changed this is the account that was changed (see src_user_name)." | ||
| }, | ||
| { | ||
| "name": "user_type", | ||
| "type": "optional", | ||
| "comment": "The type of the user involved in the event or who initiated the event, such as IAMUser, Admin, or System. For account management events, this should represent the type of the user changed by the request." | ||
| }, | ||
| { | ||
| "name": "vendor_account", | ||
| "type": "optional", | ||
| "comment": "The account that manages the user that initiated the request. The account represents the organization, or a Cloud customer or a Cloud account." | ||
| }, | ||
| { | ||
| "name": "vendor_region", | ||
| "type": "optional", | ||
| "comment": "The account that manages the user that initiated the request. The account represents the organization, or a Cloud customer or a Cloud account." | ||
| }, | ||
| { | ||
| "name": "vendor_product", | ||
| "type": "required", | ||
|
|
@@ -124,6 +149,16 @@ | |
| "type": "conditional", | ||
alexeisuv marked this conversation as resolved.
Show resolved
Hide resolved
|
||
| "condition": "object_category=user", | ||
| "comment": "For account changes, the user or entity performing the change." | ||
| }, | ||
| { | ||
| "name": "src_user_type", | ||
| "type": "optional", | ||
| "comment": "For account management events, this should represent the type of the user changed by the request." | ||
| }, | ||
| { | ||
| "name": "src_user_name", | ||
| "type": "optional", | ||
| "comment": "For account changes, the user name of the user or entity performing the change." | ||
| } | ||
| ], | ||
| "child_dataset": [ | ||
|
|
@@ -194,11 +229,71 @@ | |
| ], | ||
| "search_constraints": "tag=endpoint" | ||
| }, | ||
| { | ||
| "name": "Instance_Changes", | ||
| "tags": [["change", "instance"]], | ||
| "fields_cluster": [], | ||
| "search_constraints": "tag=instance", | ||
| "fields": [ | ||
| { | ||
| "name": "image_id", | ||
| "type": "required", | ||
| "comment": "For create instance events, this field represents the image ID used for creating the instance such as the OS, applications, installed libraries, and more." | ||
| }, | ||
| { | ||
| "name": "instance_type", | ||
| "type": "required", | ||
| "comment": "For create instance events, this field represents the type of instance to build such as the combination of CPU, memory, storage, and network capacity." | ||
| } | ||
| ], | ||
| "child_dataset": [] | ||
| }, | ||
| { | ||
| "name": "Network_Changes", | ||
| "tags": [["change", "network"]], | ||
| "fields_cluster": [], | ||
| "fields": [], | ||
| "fields": [ | ||
| { | ||
| "name": "dest_ip_range", | ||
| "type": "optional", | ||
| "comment": "For network events, the outgoing traffic for a specific destination IP address range. Specify a single IP address or an IP address range in CIDR notation. For example, 203.0.113.5 or 203.0.113.5/32." | ||
| }, | ||
| { | ||
| "name": "dest_port_range", | ||
| "type": "optional", | ||
| "comment": "For network events, this field represents destination port or range. For example, 80 or 8000 - 8080 or 80,443." | ||
| }, | ||
| { | ||
| "name": "direction", | ||
| "type": "optional", | ||
| "comment": "For network events, this field represents whether the traffic is inbound or outbound." | ||
| }, | ||
| { | ||
| "name": "protocol", | ||
| "type": "optional", | ||
| "comment": "This field represents the protocol for the network event rule." | ||
| }, | ||
| { | ||
| "name": "rule_action", | ||
| "type": "optional", | ||
| "comment": "For network events, this field represents whether to allow or deny traffic." | ||
| }, | ||
| { | ||
| "name": "src_ip_range", | ||
| "type": "optional", | ||
| "comment": "For network events, this field represents the incoming traffic from a specific source IP address or range. Specify a single IP address or an IP address range in CIDR notation." | ||
| }, | ||
| { | ||
| "name": "src_port_range", | ||
| "type": "optional", | ||
| "comment": "For network events, this field represents source port or range. For example, 80 or 8000 - 8080 or 80,443" | ||
| }, | ||
| { | ||
| "name": "device_restarts", | ||
| "type": "optional", | ||
| "comment": "Monitor all infrastructure device restarts." | ||
| } | ||
| ], | ||
| "child_dataset": [ | ||
| { | ||
| "name": "Device_Restarts", | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you please double check if this is provided by A&I framework (similar to fields in https://github.com/splunk/pytest-splunk-addon/blob/3ee0404aa07cd56ee67063c4c86aaeaeb3dbb7be/pytest_splunk_addon/standard_lib/cim_tests/CommonFields.json)?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
dest_type is not provided by A&I framework, so all fields in https://github.com/splunk/pytest-splunk-addon/blob/3ee0404aa07cd56ee67063c4c86aaeaeb3dbb7be/pytest_splunk_addon/standard_lib/cim_tests/CommonFields.json have the same in description if they are provided by A&I or not.