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

Feat: [#1109] - extended compliance finding - compliance by adding co… #1110

Merged
merged 2 commits into from
Jul 2, 2024

Conversation

PavelJurka
Copy link
Contributor

@PavelJurka PavelJurka commented Jun 11, 2024

#1109

The extension is:
compliance finding . compliance with:

compliance_references: Array of KB Article
compliance_standards: Array of KB Article

…g compliance_references and _standards based on kb_article to provide possibility to map KB articles with information relevant to finding.
@PavelJurka PavelJurka force-pushed the 1109-compliance-extension branch from 50477c9 to 4660399 Compare June 11, 2024 09:37
irakledibm
irakledibm previously approved these changes Jun 11, 2024
Copy link
Contributor

@irakledibm irakledibm left a comment

Choose a reason for hiding this comment

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

lgtm

jasonbreimer
jasonbreimer previously approved these changes Jun 18, 2024
Copy link
Contributor

@jasonbreimer jasonbreimer left a comment

Choose a reason for hiding this comment

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

Looks good.

@PavelJurka PavelJurka dismissed stale reviews from jasonbreimer and irakledibm via efe7ac1 June 24, 2024 09:26
objects/compliance.json Show resolved Hide resolved
objects/compliance.json Show resolved Hide resolved
@floydtree floydtree merged commit 271f94b into ocsf:main Jul 2, 2024
2 checks passed
@@ -58,6 +58,8 @@ Thankyou! -->
7. Added `whois` object. #992
8. Added `domain_contact` and array-typed `domain_contacts` object for use with `whois` object. #992
9. Added `Windows Service` object to the Windows extension. #1103
10. Added array-typed `compliacne_references` and array-typed `compliance_standards` objects as array of `kb_article` and used in `compliance` object. #1110
Copy link
Contributor

Choose a reason for hiding this comment

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

This is misspelled and should be compliance_references.

floydtree added a commit that referenced this pull request Oct 29, 2024
#### Related Issue: n/a

#### Description of changes:
1. Creating a generic key:value object. Thank you @Aniak5 for the
suggestion.
2. Creating instances of that generic object - `control_parameters` &
`tags`. Removing the specific tag object definition.
3. Adding `control_parameters` to `compliance` object.
4. Deprecating `status_detail` in the compliance object, in favor of
`status_details`, a new string array attribute
5. Adding new item in `status_id` enum for findings classes, to allow
representation of finding archival.
6. Updating descriptions & captions. 
* Attempting to distinguish between existing and extra attributes that
were added to compliance in #1110

---------

Signed-off-by: Rajas Panat <[email protected]>
Signed-off-by: Rajas <[email protected]>
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.

7 participants