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

Allow anonymous SID/Name translation is now in SecurityOptions #500

Conversation

dan-hughes
Copy link

The checks for Allow anonymous SID/Name translation in the CIS was looking at a registry location which was correct in the CIS documentation but not on a system.

@TuemmlerKelch TuemmlerKelch added this to the 5.8.0 milestone Jan 10, 2024
@TuemmlerKelch TuemmlerKelch added the enhancement New feature or request label Jan 10, 2024
@TuemmlerKelch
Copy link
Collaborator

@dan-hughes Thank you for this clever method to check this setting. Registry definitely was not the correct approach, as Microsoft documentation does state, that there is no corresponding registry key.
@SteffenWinternheimer I am going to test this in my testing environment as soon as I can spare some time and get back to you. Plan is to merge this into our approve branch, so we can include this in the upcoming release 5.8.0 (presumably for mid/end of January).

@TuemmlerKelch
Copy link
Collaborator

@SteffenWinternheimer Successfully tested:
Enabled via GPO
image

Disabled via GPO
image

Copy link
Collaborator

@SteffenWinternheimer SteffenWinternheimer left a comment

Choose a reason for hiding this comment

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

Code looks good, is working properly, will accept pull request

@SteffenWinternheimer SteffenWinternheimer changed the base branch from master to approve January 15, 2024 09:57
@SteffenWinternheimer SteffenWinternheimer merged commit 7d428eb into fbprogmbh:approve Jan 15, 2024
@dan-hughes dan-hughes deleted the fix-allow-anonymous-sid-name-translation branch January 15, 2024 09:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants