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

Blacklist USB-Storage to pass CIS scan #50

Conversation

rspataru
Copy link

Overall Review of Changes:
Blacklisted usb-storage in /etc/modprobe.d/blacklist.conf

Issue Fixes:
Using the CIS Benchmark we can see that this rule will fail because it was not blacklisted. Now the scan will pass

Enhancements:

How has this been tested?:
Using ansible

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Congrats on opening your first pull request and thank you for taking the time to help improve Ansible-Lockdown!
Please join in the conversation happening on the Discord Server as well.

@uk-bolly
Copy link
Member

uk-bolly commented Aug 8, 2023

hi @rspataru

Thank you for taking the time to raise this PR, Great catch and happy to add unfortunately we do need to have it signed off and signed for every commit.
The requirements can be seen in CONTRIBUTING.md.

Many thanks once again

uk-bolly

@uk-bolly uk-bolly self-requested a review August 8, 2023 13:44
uk-bolly added a commit that referenced this pull request Aug 10, 2023
@uk-bolly
Copy link
Member

hi @rspataru

Thank you once again for raising this and providing the fix.
I have added this fix into the latest updates (will be in devel and main later today) and credits are given for your work.

thank you again

uk-bolly

@uk-bolly uk-bolly mentioned this pull request Aug 10, 2023
@uk-bolly uk-bolly self-assigned this Aug 10, 2023
@rspataru
Copy link
Author

rspataru commented Aug 10, 2023 via email

@uk-bolly uk-bolly mentioned this pull request Aug 10, 2023
@uk-bolly uk-bolly closed this Aug 10, 2023
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