-
Notifications
You must be signed in to change notification settings - Fork 140
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
Conversation
…g compliance_references and _standards based on kb_article to provide possibility to map KB articles with information relevant to finding.
50477c9
to
4660399
Compare
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.
lgtm
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.
Looks good.
Signed-off-by: Pavel Jurka <[email protected]>
@@ -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 |
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.
This is misspelled and should be compliance_references
.
#### 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]>
#1109
The extension is:
compliance finding . compliance with:
compliance_references: Array of KB Article
compliance_standards: Array of KB Article