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

control fixes for existing controls #5

Merged
merged 69 commits into from
Feb 10, 2020
Merged

control fixes for existing controls #5

merged 69 commits into from
Feb 10, 2020

Conversation

kkola
Copy link
Collaborator

@kkola kkola commented Oct 10, 2019

I have reviewed about 249 controls and these include the 22 files that have changed in this PR.

The list is attached here.

Fixes #6
Fixes #7
Fixes #8
Fixes #9
Fixes #11
Fixes #12
Fixes #13
Fixes #14
Fixes #15
Fixes #16
Fixes #17
Fixes #18
Fixes #19
Fixes #20
Fixes #21
Fixes #22

Aaron Lippold lippold@gmail.com

@rx294
Copy link
Collaborator

rx294 commented Oct 15, 2019

Please address the issue addressed in #6

@rx294
Copy link
Collaborator

rx294 commented Oct 15, 2019

Please address the issue addressed in #7

@kkola
Copy link
Collaborator Author

kkola commented Nov 6, 2019

@rx294 As of 11/6/2019, 266 controls have been addressed and 9 controls are remaining since I'm unable to move forward with resolving the issues (#14 and #15). Here's the updated spreadsheet.
Windows2016Controls-11-6-2019.xlsx

@kkola
Copy link
Collaborator Author

kkola commented Nov 19, 2019

@rx294 the latest commits since my last comment resolve all remaining issues and should have addressed the entire control set in the profile. Please review.

@rx294 As of 11/6/2019, 266 controls have been addressed and 9 controls are remaining since I'm unable to move forward with resolving the issues (#14 and #15). Here's the updated spreadsheet.
Windows2016Controls-11-6-2019.xlsx

@@ -63,7 +63,7 @@
end
end

if !domain_role == '4' && !domain_role == '5'
if !(domain_role == '4') && !(domain_role == '5')
Copy link
Member

Choose a reason for hiding this comment

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

unless (domain_role == 4 || domain_role==5)

Copy link
Member

@aaronlippold aaronlippold left a comment

Choose a reason for hiding this comment

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

todo: send the team a quick primer on logical algebra....

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment