Simplify use of ansible_ensure_pam_module_option macro

[marcusburghardt]

Description:

No need to escape special characters anymore for the control parameter in ansible_ensure_pam_module_option macro.
It is treated within the macro.

Rationale:

Simplification.

Review Hints:

./build_product rhel9
./tests/automatus.py rule --libvirt qemu:///session rhel9 --datastream build/ssg-rhel9-ds.xml --dontclean --remediate-using ansible sssd_enable_smartcards

[github-actions]

Start a new ephemeral environment with changes proposed in this pull request:

rhel8 (from CTF) Environment (using Fedora as testing environment)

Fedora Testing Environment

Oracle Linux 8 Environment

[github-actions]

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

Click here to see the full diff

ansible remediation for rule 'xccdf_org.ssgproject.content_rule_sssd_enable_smartcards' differs.
--- xccdf_org.ssgproject.content_rule_sssd_enable_smartcards
+++ xccdf_org.ssgproject.content_rule_sssd_enable_smartcards
@@ -300,7 +300,7 @@
   - name: Enable Smartcards in SSSD - Define a fact for control already filtered in
       case filters are used
     ansible.builtin.set_fact:
-      pam_module_control: \[success=done authinfo_unavail=ignore ignore=ignore default=die\]
+      pam_module_control: '[success=done authinfo_unavail=ignore ignore=ignore default=die]'
 
   - name: Enable Smartcards in SSSD - Check if expected PAM module line is present
       in /etc/pam.d/system-auth
@@ -361,7 +361,7 @@
   - name: Enable Smartcards in SSSD - Define a fact for control already filtered in
       case filters are used
     ansible.builtin.set_fact:
-      pam_module_control: \[success=done authinfo_unavail=ignore ignore=ignore default=die\]
+      pam_module_control: '[success=done authinfo_unavail=ignore ignore=ignore default=die]'
 
   - name: Enable Smartcards in SSSD - Check if the required PAM module option is present
       in /etc/pam.d/system-auth

[github-actions]

🤖 A k8s content image for this PR is available at:
ghcr.io/complianceascode/k8scontent:12159
This image was built from commit: 068e6e7

Click here to see how to deploy it

If you alread have Compliance Operator deployed:
utils/build_ds_container.py -i ghcr.io/complianceascode/k8scontent:12159

Otherwise deploy the content and operator together by checking out ComplianceAsCode/compliance-operator and:
CONTENT_IMAGE=ghcr.io/complianceascode/k8scontent:12159 make deploy-local

[codeclimate]

Code Climate has analyzed commit 068e6e7 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 59.4% (0.0% change).

View more on Code Climate.

[jan-cerny]

I have checked build/rhel9/fixes/ansible/sssd_enable_smartcards.yml. Also, I have run the Automatus test scenarios for this rule.