Conversation
|
Update: @bdwyertech unfortunately the tests fail, I suspect the changes to Feel free to let me know what I can do to help, in the meantime I will close this PR, and let you reroll yours, we can continue our developments on this one. |
I have a feeling it is the mocks within winrmtest, however it could also be something in https://github.com/packer-community/winrmcp. It appears much of the code has not been touched in a very long time. https://github.com/dylanmei/winrmtest/blob/fbc9ae56efb6053a528b6002497191a22cbe0269/wsman.go#L55 Can test this theory with my updated fork.
I'm no WinRM expert, but the responses did not seem to align with Microsoft |
Supersedes #270, essentially re-applies the changes on top of the latest main, as the PR had conflicts.
Since we're releasing a new version of the SDK this week, I opted to open this one as to expedite the process, hope this is OK @bdwyertech!