Skip to content

Conversation

@Srinivasan-tm
Copy link
Contributor

Description

Found some mistakes/general improvements that can be made in the ubunu benchmark. This PR applies fixes for those

* checking if mail transfer agent is configured for local only, the
  check was negated for some reason
* checking for rsync service is uninstalled or masked, but is checking
  if rsync service is uninstalled and masked, the check should be an any
  check not an all check
* For rsyslog config, the file creation setting should either be in
  rsyslog.conf or a file in rsyslog.d/ folder. For both to be true.
  So, changing the check from an all to an any
* Before checking for the perission of sshd_config, we need to check if
  sshd is even in the system. So automatically passing the check if sshd
  is not present in the system
@Srinivasan-tm
Copy link
Contributor Author

Reasoning behind the changes:

  • checking if mail transfer agent is configured for local only, the check was negated for some reason
  • checking for rsync service is uninstalled or masked, but is checking if rsync service is uninstalled and masked, the check should be an any check not an all check
  • For rsyslog config, the file creation setting should either be in rsyslog.conf or a file in rsyslog.d/ folder. For both to be true. So, changing the check from an all to an any
  • Before checking for the perission of sshd_config, we need to check if sshd is even in the system. So automatically passing the check if sshd is not present in the system

are tabs and not spaces. So changing the check to see for tabs and not
spaces
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants