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

Added support for specifying the local address or port to SshConnection. #131

Conversation

bastiaanb
Copy link
Contributor

Adds ability to bind to a specific local host address and local port to SshConnection via options 'localAddress' and 'localPort', like OpenSSHs' '-b' parameter.
Binding to a specific local address or port may be necessary for compliance with network policies (e.g. routing or firewall settings).

replaces PR bastiaanb:feature-SshConnection-bindAddress-option

@buildhive
Copy link

XebiaLabs » overthere #217 SUCCESS
This pull request looks good
(what's this?)

@hierynomus
Copy link
Contributor

Hi Bastiaan,

Just 2 more comments left ;) Can you integrate them in this PR, then I'll merge it.

@bastiaanb
Copy link
Contributor Author

@hierynomus : these properties are not inherently SSH specific, but could be used for e.g TelnetConnections as. well. That's why they are under Common Connection Options.
This PR just implements for SSH though.

@bastiaanb
Copy link
Contributor Author

Good enough explanation to merge the PR as is ? ;-)

@hierynomus
Copy link
Contributor

AS they're not currently being used for anything else, and we might not do this for a long time. Just also discussed with @vpartington to move them to SSH.

@buildhive
Copy link

XebiaLabs » overthere #219 SUCCESS
This pull request looks good
(what's this?)

hierynomus added a commit that referenced this pull request Oct 13, 2014
…ress-option

Added support for specifying the local address or port to SshConnection.
@hierynomus hierynomus merged commit 05ba9ce into xebialabs:master Oct 13, 2014
@hierynomus
Copy link
Contributor

Done! Thx!

@bastiaanb
Copy link
Contributor Author

@hierynomus Thanks!

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