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

Ensure PAM Enforces Password Requirements - Minimum Different Categor… #93

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

peteryang
Copy link

modification is based on behavior of Ubuntu Pro with command : sudo usg audit cis_level1_server

@Sebastian-RG
Copy link
Contributor

Dear Peter, all your commits on the toai branch of your fork update your PR! Please create a new branch with only the changes you wish to contribute and create a new PR.

@Sebastian-RG
Copy link
Contributor

I would also recommend setting your fork to private since it contains sensitive information

## 5.2.17 Ensure SSH access is limited
allowed_users: ubuntu root #Put None or list of users space between each user
allowed_users: adminuser #Put None or list of users space between each user
Copy link
Owner

Choose a reason for hiding this comment

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

Since it will work in the default version of ubuntu; the user adminusr is not existed; so keeping it as ubuntu and root and the user can change it when they apply this playbook

Copy link
Collaborator

Choose a reason for hiding this comment

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

yep, we need ubuntu and root here as defaults

@@ -148,7 +155,7 @@ pass_warn_age: 7
pass_min_days: 1
## 5.4.1.1 Ensure password expiration is 365 days or less
list_of_os_users:
- ubuntu
- adminuser
Copy link
Owner

Choose a reason for hiding this comment

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

keep it for the default user ubuntu,

## 1.7.1 Ensure permissions on /etc/motd are configured: allow for custom motd template (if the file doesn't exist, the
## default template in files/templates/motd.j2 will be used)
custom_motd_file_path: "{{ inventory_dir }}/custom_templates/motd_custom.txt"
custom_motd_file_path: "{{ inventory_dir }}/../custom_templates/motd_custom.txt"
Copy link
Collaborator

Choose a reason for hiding this comment

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

why this change?

## 1.7.2 Ensure permissions on /etc/issue are configured: allow for custom issue template (if the file doesn't exist, the
## default template in files/templates/issue.j2 will be used)
custom_issue_file_path: "{{ inventory_dir }}/custom_templates/issue_custom.txt"
custom_issue_file_path: "{{ inventory_dir }}/../custom_templates/issue_custom.txt"
Copy link
Collaborator

Choose a reason for hiding this comment

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

why this change?

## 3.5.1.6 Ensure firewall rules exist for all open ports | defined ports
firewall_list_of_ports_to_allow:
- { rule: "allow", port: "8080", proto: "tcp" }
- { rule: "allow", port: "22", proto: "tcp" }
Copy link
Collaborator

Choose a reason for hiding this comment

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

why should we add these strange ports by default?

@@ -124,14 +131,14 @@ grub_backlog_limit: 8192

# Section 5 settings
## 5.1.8 Ensure cron is restricted to authorized users
allowd_hosts: "ALL: 0.0.0.0/0.0.0.0, 192.168.2.0/255.255.255.0"
allowd_hosts: "ALL: 0.0.0.0/0.0.0.0, 10.2.0.0/255.255.0.0"
Copy link
Collaborator

Choose a reason for hiding this comment

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

reason for this change?

## 5.2.17 Ensure SSH access is limited
allowed_users: ubuntu root #Put None or list of users space between each user
allowed_users: adminuser #Put None or list of users space between each user
Copy link
Collaborator

Choose a reason for hiding this comment

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

yep, we need ubuntu and root here as defaults

@@ -96,10 +96,17 @@ remove_RPC: yes
disable_wifi: no
IPv6_is_enabled: no
enable_firewall: yes
UFWEnable: yes # Running both ufw and the services included in the iptables-persistent package may lead to conflict
UFWEnable: no # Running both ufw and the services included in the iptables-persistent package may lead to conflict
Copy link
Collaborator

Choose a reason for hiding this comment

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

why changing this?

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.

4 participants