-
Notifications
You must be signed in to change notification settings - Fork 724
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
OPENSCAP-5464: Use more fine granular audit rules for E8 and ISM profiles in RHEL #13149
Conversation
Instead of audit_rules_usergroup_modification use audit_rules_usergroup_modification_group audit_rules_usergroup_modification_gshadow audit_rules_usergroup_modification_opasswd audit_rules_usergroup_modification_passwd audit_rules_usergroup_modification_shadow
In RHEL10 ISM_O control file since the original rule doesn't have ansible remediation.
/packit retest-failed |
1 similar comment
/packit retest-failed |
/packit retest-failed |
@@ -107,7 +107,6 @@ selections: | |||
- audit_rules_login_events_tallylog | |||
- audit_rules_login_events_faillock | |||
- audit_rules_login_events_lastlog | |||
- audit_rules_login_events |
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.
Please add the removed rules to the RHEL 9 default profile so that they will be kept in the built RHEL 9 data stream.
@@ -574,3 +574,4 @@ selections: | |||
- file_permissions_etc_audit_rulesd | |||
- configure_openssl_tls_crypto_policy | |||
- configure_openssl_crypto_policy | |||
- audit_rules_login_events |
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.
Please add also the other removed rules (audit_rules_usergroup_modification and audit_rules_unsuccessful_file_modification) to prevent their removal from RHEL 9 data stream.
Code Climate has analyzed commit e7cb339 and detected 0 issues on this pull request. The test coverage on the diff in this pull request is 100.0% (50% is the threshold). This pull request will bring the total coverage in the repository to 62.0% (0.0% change). View more on Code Climate. |
Description:
Rationale:
Fixes #13125