-
Notifications
You must be signed in to change notification settings - Fork 99
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
Move windows updates to go #719
Move windows updates to go #719
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Couple comments. overall this looks pretty good, I suspect that there is the potential for bugs in the pkg/windows
code, e.g. not matching the windows api types properly, but this seems to work well enough for our purposes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Move the windows update data from a powershell exec to go-ole.
This leverages the work in https://github.com/ceshihao/windowsupdate