Skip to content

Commit

Permalink
Removed constraint from group_management (#1193)
Browse files Browse the repository at this point in the history
#### Related Issue:  N/A (Slack discussion)

#### Description of changes:
The `Group Management` class had an unnecessary restriction to populate
either `user` or `privileges`. This is too stringent for the case where
the group is created or deleted where no users or privileges are known
or relevant. Therefore the PR removes the constraint.

---------

Signed-off-by: Paul Agbabian <[email protected]>
  • Loading branch information
pagbabian-splunk authored Oct 8, 2024
1 parent 227cfdf commit c483f1f
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 7 deletions.
3 changes: 2 additions & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,8 @@ Thankyou! -->
* #### Event Classes
1. Added `evidences` to `compliance_finding` class. #1157
2. Added `is_alert` to `detection_finding` and `data_security_finding` classes. #1178
3. Added `risk_details` to `data_security_finding` class
3. Added `risk_details` to `data_security_finding` class. #1178
4. Removed constraint from `group_management` class. #1193
* #### Objects
1. Added `phone_number` to `user` and `ldap_person` objects. #1155
2. Added `has_mfa` to `user` object. #1155
Expand Down
6 changes: 0 additions & 6 deletions events/iam/group_management.json
Original file line number Diff line number Diff line change
Expand Up @@ -53,11 +53,5 @@
"group": "primary",
"requirement": "recommended"
}
},
"constraints": {
"at_least_one": [
"privileges",
"user"
]
}
}

0 comments on commit c483f1f

Please sign in to comment.