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

Update winrm password key for winrm-v2 #94

Merged
merged 1 commit into from
Sep 4, 2016
Merged

Conversation

mwrock
Copy link
Contributor

@mwrock mwrock commented Jul 9, 2016

No description provided.

@@ -174,7 +174,7 @@ def runner_options_for_winrm(config_data)
'host' => URI(kitchen[:endpoint]).hostname,
'port' => URI(kitchen[:endpoint]).port,
'user' => kitchen[:user],
'password' => kitchen[:pass],
'password' => kitchen[:password],
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we make a fallback to kitchen[:pass], this would allow us to use kitchen-inspec with older and newer versions?

@chris-rock
Copy link
Collaborator

Thanks @mwrock Great work!

@mwrock
Copy link
Contributor Author

mwrock commented Aug 3, 2016

Falls back to :pass. good call @chris-rock! Still WIP pending WinRb/WinRM#191 and other upstream branches

@mwrock mwrock changed the title WIP: Update winrm password key for winrm-v2 Update winrm password key for winrm-v2 Sep 2, 2016
@mwrock
Copy link
Contributor Author

mwrock commented Sep 2, 2016

This is no longer a work in progress :)

@chris-rock
Copy link
Collaborator

Thanks @mwrock for the great work.

@chris-rock chris-rock merged commit 74eef9f into inspec:master Sep 4, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants