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

Add native winrs as an alternative to the Java WinRM implementation #12

Closed
bmoussaud opened this issue Sep 6, 2011 · 3 comments
Closed

Comments

@bmoussaud
Copy link

Overthere 1.0 support a java implementation of Winrm.
It should also the winrs options, allowing to call directly the windows binary.
An implementation has been done here for BDF:
https://github.com/xebialabs/bdf-plugin/commit/c2919011ae38f72ddd612175cfbf721ff9a54c60
But there is a lot of code duplication and it will not be easy to switch in deployit for a given host between java implentation and winrs.

@vpartington
Copy link
Contributor

What are the advantages of a native WinRS implementation over the Java implementation?

What do you mean by your last sentence? The code duplication can be solved by some good refactoring (like I've already done when integrating the CIFS/WinRM implementation into the CIFS/Telnet implementation) and switching between the WinRS implementation and the Java implementation for a given host should then be as easy as switching between CIFS/WinRM and CIFS/Telnet: by changing an enum.

@bmoussaud
Copy link
Author

What are the advantages of a native WinRS implementation over the Java implementation?

Easier to setup on complexe Windows Environment with Domain Controler and Active Directory.
Allow to check the input parameters and later find bugs or improvements in the Java implementation

To be usefull the winrs option should be available in the Windows Hosts, allowing to switch easily from Java Implementation to the WinrsImplementation... as we want to switch from SSH/SFTP to SSH/SCP and SSH/SUDO/SCP...

bmoussaud pushed a commit that referenced this issue Sep 23, 2011
bmoussaud pushed a commit to bmoussaud/overthere that referenced this issue Oct 28, 2011
@hierynomus
Copy link
Contributor

Closing this in favour of #18, where the pull request and discussion is going on.

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

No branches or pull requests

3 participants