* rewrite system account detection and hardening * resolve failures created when resolving merge conflicts Signed-off-by: Sebastian Gumprich <[email protected]> * add tests for shell removal tasks Signed-off-by: Sebastian Gumprich <[email protected]> * Update molecule/os_hardening/prepare.yml Signed-off-by: Martin Schurz <[email protected]> * split tasks for locking and setting shell Signed-off-by: Martin Schurz <[email protected]> * fix some more linting Signed-off-by: Sebastian Gumprich <[email protected]> Signed-off-by: Sebastian Gumprich <[email protected]> Signed-off-by: Martin Schurz <[email protected]> Co-authored-by: donestefan <[email protected]> Co-authored-by: schurzi <[email protected]>
138 lines
4.4 KiB
YAML
138 lines
4.4 KiB
YAML
---
|
|
- name: Fetch OS dependent variables
|
|
ansible.builtin.include_vars:
|
|
file: "{{ item }}"
|
|
name: os_vars
|
|
with_first_found:
|
|
- files:
|
|
- "{{ ansible_facts.distribution }}_{{ ansible_facts.distribution_major_version | default() }}.yml"
|
|
- "{{ ansible_facts.distribution }}.yml"
|
|
- "{{ ansible_facts.os_family }}_{{ ansible_facts.distribution_major_version | default() }}.yml"
|
|
- "{{ ansible_facts.os_family }}.yml"
|
|
skip: true
|
|
tags: always
|
|
|
|
# we only override variables with our default, if they have not been specified already
|
|
# by default the lookup functions finds all varnames containing the string, therefore
|
|
# we add ^ and $ to denote start and end of string, so this returns only exact matches
|
|
- name: Set OS dependent variables, if not already defined by user # noqa var-naming
|
|
ansible.builtin.set_fact:
|
|
"{{ item.key }}": "{{ item.value }}"
|
|
when: not lookup('varnames', '^' + item.key + '$')
|
|
with_dict: "{{ os_vars }}"
|
|
tags: always
|
|
|
|
- name: Get openssh-version
|
|
ansible.builtin.command: ssh -V
|
|
register: sshd_version_raw
|
|
changed_when: false
|
|
check_mode: false
|
|
|
|
- name: Parse openssh-version
|
|
ansible.builtin.set_fact:
|
|
sshd_version: "{{ sshd_version_raw.stderr | regex_replace('.*_([0-9]*.[0-9]).*', '\\1') }}"
|
|
|
|
- name: Set default for ssh_host_key_files if not supplied
|
|
ansible.builtin.include_tasks: crypto_hostkeys.yml
|
|
when:
|
|
- ssh_server_hardening | bool
|
|
- not ssh_host_key_files
|
|
|
|
- name: Set default for ssh_macs if not supplied
|
|
ansible.builtin.include_tasks: crypto_macs.yml
|
|
when: not ssh_macs
|
|
|
|
- name: Set default for ssh_ciphers if not supplied
|
|
ansible.builtin.include_tasks: crypto_ciphers.yml
|
|
when: not ssh_ciphers
|
|
|
|
- name: Set default for ssh_kex if not supplied
|
|
ansible.builtin.include_tasks: crypto_kex.yml
|
|
when: not ssh_kex
|
|
|
|
- name: Create revoked_keys and set permissions to root/600
|
|
ansible.builtin.template:
|
|
src: revoked_keys.j2
|
|
dest: /etc/ssh/revoked_keys
|
|
mode: "0600"
|
|
owner: "{{ ssh_owner }}"
|
|
group: "{{ ssh_group }}"
|
|
notify: Restart sshd
|
|
when: ssh_server_hardening | bool
|
|
|
|
- name: Create sshd_config and set permissions to root/600
|
|
ansible.builtin.template:
|
|
src: opensshd.conf.j2
|
|
dest: "{{ ssh_server_config_file }}"
|
|
mode: "0600"
|
|
owner: "{{ ssh_owner }}"
|
|
group: "{{ ssh_group }}"
|
|
validate: "{{ sshd_path }} -T -C user=root -C host=localhost -C addr=localhost -C lport=22 -f %s"
|
|
notify: Restart sshd
|
|
when: ssh_server_hardening | bool
|
|
|
|
- name: Disable dynamic MOTD
|
|
community.general.pamd:
|
|
name: sshd
|
|
type: session
|
|
control: optional
|
|
module_path: pam_motd.so
|
|
state: absent
|
|
backup: true
|
|
when:
|
|
- ssh_server_hardening | bool
|
|
- ssh_pam_support | bool
|
|
- not (ssh_print_pam_motd | bool)
|
|
|
|
- name: Create ssh_config and set permissions to root/644
|
|
ansible.builtin.template:
|
|
src: openssh.conf.j2
|
|
dest: "{{ ssh_client_config_file }}"
|
|
mode: "0644"
|
|
owner: "{{ ssh_owner }}"
|
|
group: "{{ ssh_group }}"
|
|
when: ssh_client_hardening | bool
|
|
|
|
- name: Check if for weak DH parameters in {{ sshd_moduli_file }}
|
|
ansible.builtin.shell: awk '$5 < {{ sshd_moduli_minimum }}' {{ sshd_moduli_file }}
|
|
register: sshd_register_moduli
|
|
changed_when: false
|
|
check_mode: false
|
|
when: ssh_server_hardening | bool
|
|
|
|
- name: Remove all small primes
|
|
ansible.builtin.shell: >
|
|
awk '$5 >= {{ sshd_moduli_minimum }}' {{ sshd_moduli_file }} > {{ sshd_moduli_file }}.new ; [ -r {{ sshd_moduli_file }}.new
|
|
-a -s {{ sshd_moduli_file }}.new ] && mv {{ sshd_moduli_file }}.new {{ sshd_moduli_file }} || true
|
|
notify: Restart sshd
|
|
when:
|
|
- ssh_server_hardening | bool
|
|
- sshd_register_moduli.stdout
|
|
|
|
- name: Include tasks to setup ca keys and principals
|
|
ansible.builtin.include_tasks: ca_keys_and_principals.yml
|
|
when:
|
|
- ssh_trusted_user_ca_keys_file | length > 0
|
|
- ssh_trusted_user_ca_keys | length > 0
|
|
|
|
- name: Include selinux specific tasks
|
|
ansible.builtin.include_tasks: selinux.yml
|
|
when: ansible_facts.selinux and ansible_facts.selinux.status == "enabled"
|
|
|
|
- name: Gather package facts
|
|
ansible.builtin.package_facts:
|
|
check_mode: false
|
|
when:
|
|
- sshd_disable_crypto_policy | bool
|
|
|
|
- name: Disable SSH server CRYPTO_POLICY
|
|
ansible.builtin.copy:
|
|
src: sshd
|
|
dest: /etc/sysconfig/sshd
|
|
owner: root
|
|
group: root
|
|
mode: "0640"
|
|
when:
|
|
- sshd_disable_crypto_policy | bool
|
|
- ('crypto-policies' in ansible_facts.packages)
|