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

Windows securable resources should support SIDs #4397

Closed
smurawski opened this issue Jan 13, 2016 · 4 comments · Fixed by #10423
Closed

Windows securable resources should support SIDs #4397

smurawski opened this issue Jan 13, 2016 · 4 comments · Fixed by #10423

Comments

@smurawski
Copy link
Contributor

We should support supplying the SID for users and groups for user resource, group resource, and any windows securables.

@drmmarsunited
Copy link

Hey folks!

Just wondering if this is going to get prioritized soon. I've just started a new project with Chef that deals with multiple base language versions of Windows, so the ability to target rights via SID is the way I would like to go.

Thanks for all your awesome efforts!

@eedwardsdisco
Copy link

+1

Windows DSC supports using SIDs, would be nice to have parity since due to issues with dsc_resource and dsc_script I'm forced to use the directory resource.

@thommay thommay added Priority: Critical Fix immediately and removed High Priority labels Jan 25, 2017
@jugatsu
Copy link
Contributor

jugatsu commented Apr 2, 2017

#5981

@stuartpreston stuartpreston changed the title Support SIDs Windows securable resources should support SIDs Jul 11, 2018
@stuartpreston stuartpreston removed this from the Accepted Minor milestone Jul 11, 2018
@ISHBAR
Copy link

ISHBAR commented Jun 12, 2020

Checking in on this, (I'm open to other options out there), currently trying to set permissions for some ohai plugins

e.g. rights :full_control, 'Administrators' and getting

Chef::Exceptions::Win32APIError: No mapping between account names and security IDs was done.

This is whenever 'Administrators' / Authenticated Users is in another language, i.e.
Usuários autenticados. This would be a lot more universal with SIDs instead.

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

Successfully merging a pull request may close this issue.

10 participants