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

Support for port=0 which means automatically allocated port. #40

Merged
merged 2 commits into from
Mar 17, 2020

Conversation

Elisedlund-ericsson
Copy link
Contributor

typically allocated from an ephemeral port range

java.net.ServerSocket has support for setting port to 0 and will then
automatically allocate a port.
trilead.LocalPortForwarder handles port 0 fine already however as a end
user its currently not possible to figure out which port was actually
allocated in the end which this fix changes.

Elisedlund-ericsson and others added 2 commits March 16, 2020 14:48
…ed, typically from an ephemeral port range

java.net.ServerSocket has support for setting port==0 and will then automatically allocate a port.
trilead.LocalPortForwarder handles this fine however as a user its currently not possible to figure out which port was actually allocated in the end which this fix changes.
@kuisathaverat kuisathaverat merged commit b9742d8 into jenkinsci:master Mar 17, 2020
@Elisedlund-ericsson
Copy link
Contributor Author

Nice, thanks for the quick merge.
When can I expect an official release? I'm using both the latest unreleased commits and it would be nice to use a official release.

@kuisathaverat
Copy link
Contributor

well, I have to release this module and the trilead-api-plugin, I guess I would find some time before the end of the week

@kuisathaverat
Copy link
Contributor

kuisathaverat commented Mar 18, 2020

as usual, I've made a mess with the version of this module (it does not have a regular version name) so I've burned 2 versions 🤦‍♂ so finally, your changes are in the trilead-ssh2:build-217-jenkins-19 and trilead-api-plugin:1.0.6

@Elisedlund-ericsson
Copy link
Contributor Author

big 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.

2 participants