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

Chef-run does not warn if user and password incorrectly formatted and hang indefinitely on winrm connections #77

Open
floodman23 opened this issue Dec 12, 2018 · 2 comments
Labels
Aspect: Integration Works correctly with other projects or systems. Triage: Confirmed Indicates and issue has been confirmed as described. Type: Bug Does not work as expected.

Comments

@floodman23
Copy link

floodman23 commented Dec 12, 2018

Description

Incorrectly formatted --user and --password parameters cause chef-run to attempt to connect to the target but hang indefinitely. Eg:

chef-run winrm://10.202.0.175 user timmy --user=Administrator --password=Password123
 [✔] Packaging cookbook... done!
 [✔] Generating local policyfile... exporting... done!
 [\] Applying user[timmy] from resource to target.
 |__ [\] [10.202.0.175] Connecting as Administrator...

The chef-run process then just sits there.
After a lot of experimentation, removing the equals sign from the parameters allowed it to work successfully.
What threw me was that the username was picked up correctly

Chef Workstation Version

0.2.41

Platform Version

Windows 10

@jonsmorrow
Copy link
Contributor

@floodman23 Thanks for the report. Of the top of my head we probably are not parsing the equals sign correctly. Have you tried to run with it? We'll look into handling the parsing better or providing more clear documentation on the command.

Also, moving this to chef-apply.

@jonsmorrow jonsmorrow transferred this issue from chef/chef-workstation May 13, 2019
@tyler-ball tyler-ball added Aspect: Correctness Aspect: Integration Works correctly with other projects or systems. Triage: Confirmed Indicates and issue has been confirmed as described. Type: Bug Does not work as expected. labels Sep 9, 2019
@tyler-ball
Copy link
Contributor

I know we had to do some specific work to support weird password Characters for Windows connections, seems like we're probably just handing a regex incorrectly somewhere

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Aspect: Integration Works correctly with other projects or systems. Triage: Confirmed Indicates and issue has been confirmed as described. Type: Bug Does not work as expected.
Projects
None yet
Development

No branches or pull requests

3 participants