modify regexp in bash remediation of chronyd_specify_remote_server

[vojtapolasek]

Description:

• change the regex so that is correctly matches the line, I think matching the space characters was the problem

Rationale:

• previously, new lines were inserted into the file no matter if there were some correct lines before

• I believe it was original intend of this PR: Corrections in rule chronyd_specify_remote_server #10278

Review Hints:

./build_product rhel8
cd tests
python automatus.py rule --libvirt qemu:///system rhel8 chronyd_specify_remote_server

[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

bash remediation for rule 'xccdf_org.ssgproject.content_rule_chronyd_specify_remote_server' differs.
--- xccdf_org.ssgproject.content_rule_chronyd_specify_remote_server
+++ xccdf_org.ssgproject.content_rule_chronyd_specify_remote_server
@@ -6,7 +6,7 @@
 
 config_file="/etc/chrony.conf"
 
-if ! grep -q '^[\s]*(?:server|pool)[\s]+[\w]+' "$config_file" ; then
+if ! grep -q '^[[:space:]]*\(server\|pool\)[[:space:]]\+[[:graph:]]\+' "$config_file" ; then
 if ! grep -q '#[[:space:]]*server' "$config_file" ; then
 for server in $(echo "$var_multiple_time_servers" | tr ',' '\n') ; do
 printf '\nserver %s' "$server" >> "$config_file"

[codeclimate]

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

View more on Code Climate.

[marcusburghardt]

Thanks for this fix.