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

Addressing issue #36 #37

Merged
merged 1 commit into from
Mar 28, 2024
Merged

Addressing issue #36 #37

merged 1 commit into from
Mar 28, 2024

Conversation

mfortin
Copy link
Contributor

@mfortin mfortin commented Mar 26, 2024

Overall Review of Changes:
Implementing control 1.1.6

Issue Fixes:
#36

Enhancements:
None

How has this been tested?:
Build a new server using this ansible role and validated against CIS Benchmark using CIS Assessor

@MrSteve81
Copy link
Contributor

Great Find Moving through the approval process now.

Copy link
Contributor

@georgenalen georgenalen left a comment

Choose a reason for hiding this comment

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

looks good!

Copy link
Contributor

@MrSteve81 MrSteve81 left a comment

Choose a reason for hiding this comment

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

Approved

@MrSteve81 MrSteve81 merged commit 8d563f7 into ansible-lockdown:devel Mar 28, 2024
3 checks passed
@mfortin mfortin deleted the issue-36 branch March 28, 2024 21:36
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.

3 participants