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

Fix accounts_password_pam_retry #13144

Merged
merged 1 commit into from
Mar 7, 2025
Merged

Conversation

jan-cerny
Copy link
Collaborator

The rule accounts_password_pam_retry failed after kickstart or Anaconda installation. The problem is that the rule passes in the initial scan but fails in the final scan because in the initial scan the rule enable_authselect fails and the remediation for enable_authselect made accounts_password_pam_retry fail but remediation for accounts_password_pam_retry isn't executed because it passed in the initial scan.

Fixes: #12277

Used test: /hardening/anaconda/anssi_bp28_high on RHEL 9

The rule `accounts_password_pam_retry` failed after kickstart or
Anaconda installation. The problem is that the rule passes
in the initial scan but fails in the final scan because in the
initial scan the rule `enable_authselect` fails and the remediation
for `enable_authselect` made `accounts_password_pam_retry` fail
but remediation for `accounts_password_pam_retry` isn't executed
because it passed in the initial scan.

Fixes: ComplianceAsCode#12277

Used test: `/hardening/anaconda/anssi_bp28_high` on RHEL 9
@jan-cerny jan-cerny added OVAL OVAL update. Related to the systems assessments. productization-issue Issue found in upstream stabilization process. labels Mar 5, 2025
@jan-cerny jan-cerny added this to the 0.1.77 milestone Mar 5, 2025
@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Used by openshift-ci bot. label Mar 5, 2025
Copy link

openshift-ci bot commented Mar 5, 2025

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

Copy link

github-actions bot commented Mar 5, 2025

This datastream diff is auto generated by the check Compare DS/Generate Diff

Click here to see the full diff
OVAL for rule 'xccdf_org.ssgproject.content_rule_accounts_password_pam_retry' differs.
--- oval:ssg-accounts_password_pam_retry:def:1
+++ oval:ssg-accounts_password_pam_retry:def:1
@@ -1,3 +1,5 @@
+criteria AND
+extend_definition oval:ssg-enable_authselect:def:1
 criteria OR
 criteria AND
 criterion oval:ssg-test_password_pam_pwquality_retry_password_auth:tst:1

Copy link

codeclimate bot commented Mar 5, 2025

Code Climate has analyzed commit cd0138d 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.1% (0.0% change).

View more on Code Climate.

@jan-cerny
Copy link
Collaborator Author

/packit build

@jan-cerny
Copy link
Collaborator Author

/packit delete

@jan-cerny
Copy link
Collaborator Author

/packit build

@jan-cerny jan-cerny marked this pull request as ready for review March 7, 2025 11:54
@openshift-ci openshift-ci bot removed the do-not-merge/work-in-progress Used by openshift-ci bot. label Mar 7, 2025
@Mab879 Mab879 self-assigned this Mar 7, 2025
Copy link
Member

@Mab879 Mab879 left a comment

Choose a reason for hiding this comment

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

Please propose a PR to remove the wavier once this merged.

Thanks!

@Mab879
Copy link
Member

Mab879 commented Mar 7, 2025

Waving Automatus tests as contest tests pass.

@Mab879 Mab879 merged commit aaf0a84 into ComplianceAsCode:master Mar 7, 2025
103 of 111 checks passed
jan-cerny added a commit to jan-cerny/contest that referenced this pull request Mar 10, 2025
matusmarhefka pushed a commit to RHSecurityCompliance/contest that referenced this pull request Mar 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OVAL OVAL update. Related to the systems assessments. productization-issue Issue found in upstream stabilization process.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Rule accounts_password_pam_retry fails after kickstart installation
2 participants