Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Do not check for administrative access by locking for group name. #7

Closed
kmpm opened this Issue Oct 9, 2012 · 1 comment

Comments

Projects
None yet
2 participants

kmpm commented Oct 9, 2012

If Windows run in another language then English the group will be called differently.
Please check for SID instead of group name. S-1-5-32-544 is the local Administrators group.

Well-know security identifiers
http://support.microsoft.com/kb/243330

Owner

jfromaniello commented Oct 9, 2012

Thanks @kmpm , fixed and deployed to npm as versión 0.0.12.

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