Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
[FIXED JENKINS-28169] retry with legacy SSL if Jabber server doesn'
response at all to plain connect attempts
  • Loading branch information
kutzi committed May 2, 2015
1 parent 4b3a746 commit adfd604
Showing 1 changed file with 5 additions and 2 deletions.
Expand Up @@ -403,7 +403,7 @@ public boolean verify(String hostname, SSLSession session) {
);

boolean retryWithLegacySSL = false;
XMPPException originalException = null;
Exception originalException = null;
try {
this.connection = new XMPPTCPConnection(cfg);
this.connection.connect();
Expand All @@ -413,6 +413,9 @@ public boolean verify(String hostname, SSLSession session) {
} catch (XMPPException e) {
retryWithLegacySSL = true;
originalException = e;
} catch (SmackException.NoResponseException e) {
retryWithLegacySSL = true;
originalException = e;
} catch (SmackException e) {
LOGGER.warning(ExceptionHelper.dump(e));
} catch (IOException e) {
Expand Down Expand Up @@ -491,7 +494,7 @@ public void run() {
* See JENKINS-6863
*/
private void retryConnectionWithLegacySSL(
final ConnectionConfiguration cfg, @Nullable XMPPException originalException)
final ConnectionConfiguration cfg, @Nullable Exception originalException)
throws XMPPException, SmackException {
try {
LOGGER.info("Retrying connection with legacy SSL");
Expand Down

0 comments on commit adfd604

Please sign in to comment.