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

[JENKINS-38539, JENKINS-37539] Update remoting from 2.62 to 2.62.2 #2585

Merged
merged 1 commit into from Oct 11, 2016

Conversation

oleg-nenashev
Copy link
Member

@oleg-nenashev oleg-nenashev commented Oct 9, 2016

2.62.1 does not exist, there was an issue during the release
Changes in 2.62.2: https://github.com/jenkinsci/remoting/blob/stable-2.x/CHANGELOG.md#2622

@jenkinsci/code-reviewers

2.61 does not exist, there was an issue during the release
Changes in 2.62: https://github.com/jenkinsci/remoting/blob/stable-2.x/CHANGELOG.md#2622

* [JENKINS-38539](https://issues.jenkins-ci.org/browse/JENKINS-38539) - 
Stability: Turn on SO_KEEPALIVE and provide CLI option to turn it off again.
(jenkinsci/remoting#110)
* [JENKINS-37539](https://issues.jenkins-ci.org/browse/JENKINS-37539) - 
Prevent <code>NullPointerException</code> in <code>Engine#connect()</code> when host or port parameters are <code>null</code> or empty.
(jenkinsci/remoting#101)
* [CID-152201] - 
Fix resource leak in <code>remoting.jnlp.Main</code>.
(jenkinsci/remoting#102)
* [CID-152200,CID-152202] - 
Resource leak in Encryption Cipher I/O streams on exceptional paths.
(jenkinsci/remoting#104)
@oleg-nenashev oleg-nenashev added the needs-more-reviews Complex change, which would benefit from more eyes label Oct 9, 2016
@daniel-beck
Copy link
Member

2.61 does not exist, there was an issue during the release
Changes in 2.62

Copy & paste problem? Why would either of these matter when this goes from 2.62 to 2.62.2?

@oleg-nenashev
Copy link
Member Author

oleg-nenashev commented Oct 9, 2016

@daniel-beck
Just PR a creation problem (PEBKAC). I meant 2.62.1 and 2.62.2. Adjusted the description

@daniel-beck
Copy link
Member

Code review diff: jenkinsci/remoting@remoting-2.62...remoting-2.62.2

@oleg-nenashev
Copy link
Member Author

BTW, my TODO is to add proper @since annotations to the code on the remoting side. Just a technical debt

@oleg-nenashev oleg-nenashev added ready-for-merge The PR is ready to go, and it will be merged soon if there is no negative feedback and removed needs-more-reviews Complex change, which would benefit from more eyes labels Oct 9, 2016
@oleg-nenashev
Copy link
Member Author

Was just a timeout, previous build was fine

@oleg-nenashev oleg-nenashev merged commit 7e0db20 into master Oct 11, 2016
oleg-nenashev added a commit that referenced this pull request Oct 15, 2016
@oleg-nenashev oleg-nenashev deleted the remoting/2.62.2 branch June 9, 2018 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-for-merge The PR is ready to go, and it will be merged soon if there is no negative feedback
Projects
None yet
2 participants