search

ansible-lockdown / UBUNTU22-CIS

Ansible role for Ubuntu22 CIS Baseline
https://ansible-lockdown.readthedocs.io/en/latest/
MIT License
155 stars 68 forks source link

Fixing issue https://github.com/ansible-lockdown/UBUNTU22-CIS/issues/180 #181

Closed DianaMariaDDM closed 5 months ago

DianaMariaDDM commented 7 months ago

Overall Review of Changes: This is the solution provided for issue 180. It uses the same ansible module as before, only that it adds another task that edits the necessary line right above the password [success=1 default=ignore] pam_unix.so obscure yescrypt line.

Issue Fixes:

Enhancements: None

How has this been tested?: Tested on my local test environment.

uk-bolly commented 5 months ago

hi @DianaMariaDDM

Thank you for updating all of your branches. If i could request that you do it again, this should enable this older PR to complete as well.

Many thanks

uk-bolly

uk-bolly commented 5 months ago

hi @DianaMariaDDM

I have added a warning that PRs are stale if your branch is not upto date, You will see that we have seen this on a number of your PRs. Hoping this helps us to highlight issues so we can get these resolved.

Many thanks

uk-bolly

DianaMariaDDM commented 5 months ago

Thank you for letting me know!

I have re-updated my branches and I have also fixed some indentation issues on some PRs. Now all the checks have passed on all PRs, hope this helps!