Skip to content

Add/Modify LTSC Checks #21

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

Open
njohnson11 opened this issue Apr 5, 2021 · 3 comments
Open

Add/Modify LTSC Checks #21

njohnson11 opened this issue Apr 5, 2021 · 3 comments
Labels
bug Something isn't working

Comments

@njohnson11
Copy link
Contributor

njohnson11 commented Apr 5, 2021

A number of controls whose descriptions mention being NA for LTSC/B systems, are either not checking this or are checking in a way that misses new LTSC versions. The following controls need to be addressed.

  • 63349 - V3R2 Control SV-220706
  • 63659 - V3R2 Control SV-220825
  • 63699 - V3R2 Control SV-220840
  • 63701 - V3R2 Control SV-220841
  • 63709 - V3R2 Control SV-220843
  • 63713 - V3R2 Control SV-220844
  • 82139 - V3R2 Control SV-220842
@njohnson11 njohnson11 added bug Something isn't working enhancement New feature or request labels Apr 5, 2021
@njohnson11 njohnson11 self-assigned this Apr 5, 2021
@njohnson11
Copy link
Contributor Author

njohnson11 commented Apr 5, 2021

Note: I developed the following inspec commands to check if a system is LTSC or not, however there is likely a cleaner way. Thoughts?

is_ltsc = command('gwmi win32_operatingsystem | select OperatingSystemSKU').stdout.split[-1].strip.to_i >= 123

is_ltsc = command('systeminfo | findstr /B /C:"OS Name"').stdout.include?("LTS")

This seems like the fastest to check
is_ltsc = os.name.downcase.include?("lts")

is_ltsc = registry_key('HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Windows NT\CurrentVersion')["ProductName"].downcase.include?("lts")

@aaronlippold
Copy link
Member

aaronlippold commented Apr 6, 2021 via email

@njohnson11
Copy link
Contributor Author

That one seemed the cleanest. The first one is a better bet for working against IoT versions but the os name one would likely work as well.

@Amndeep7 Amndeep7 removed the enhancement New feature or request label Feb 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants