-
Notifications
You must be signed in to change notification settings - Fork 112
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
Support for connecting to winrm over ssl #54
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.
This LGTM. We may also want to support the self_signed option but we could add a new PR for that when we need it in the future
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
8f83c8a
to
ba0cf06
Compare
This adds the options --ssl-verify and --[no-]ssl-verify to permit WinRM connections over https. This also moves managing translation of connection parameters to RemoteConnection instead of having some pieces in Target and some in RemoteConnection. Signed-off-by: Marc A. Paradise <marc.paradise@gmail.com>
ba0cf06
to
e1fdc97
Compare
This is now updated with a '--[no]-ssl-verify' option and handling/instructions for ssl host verification failed. |
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
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.
Small PR to enable ssl connections for WinRM.
This still needs validation against a running WinRM instance with SSL enabled, and a minor update to
Target::Converge#perform_command
in order to make connection parameters testsable.Signed-off-by: Marc A. Paradise marc.paradise@gmail.com