Smack 4.2.2

-----BEGIN PGP SIGNATURE-----
 
 iQGTBAABCgB9FiEEl3UFnzoh3OFr5PuuIjmn6PWFIFIFAloZnf5fFIAAAAAALgAo
 aXNzdWVyLWZwckBub3RhdGlvbnMub3BlbnBncC5maWZ0aGhvcnNlbWFuLm5ldDk3
 NzUwNTlGM0EyMURDRTE2QkU0RkJBRTIyMzlBN0U4RjU4NTIwNTIACgkQIjmn6PWF
 IFLcIQgAsTLRI3sWIZDPRuI5YaU/y6beqqzMp5cQgk/9+9DHGBoziLEU1spkvBs2
 Yvlwu3NtHSJsI+ibzYLEFRnzEwuW07vXw7R5J+kRSZrsE40z1HCQRwUIXDkPgghe
 MuL6vT6OV+kLqKb33YLSBuT2fj2pzVE6mRmMsnFekay/weEqOGUZaY9Hd5lJ31/9
 33fQC0FQfKULQ5t7PbdfX4dDDNWn0n6v+KjvjaskHI0oA+vqPWxPkj8gNXWW8b72
 tVV4h5uXQ0ziK3oED79+GH+DSiET9N2PmsZ7woXiFy5H8KbcQDVe5Pazf+Iq2VQ+
 Vi6Vxikr6Ak+v2Xkt+e3x9E6mZDI1w==
 =CVqq
 -----END PGP SIGNATURE-----

Merge tag '4.2.2'

Smack 4.2.2

resources/releasedocs/changelog.html

@@ -141,6 +141,37 @@ hr {
 
 	<div id="pageBody">
 
+
+
+<h2>4.2.2 -- <span style="font-weight: normal;">2017-11-25</span></h2>
+                    
+<h2>        Bug
+</h2>
+<ul>
+<li>[<a href='https://issues.igniterealtime.org/browse/SMACK-775'>SMACK-775</a>] -         Create callback interface for ReconnectionManager
+</li>
+<li>[<a href='https://issues.igniterealtime.org/browse/SMACK-778'>SMACK-778</a>] -         ReconnectionManager.reconnect() can throw NotConnectedException
+</li>
+<li>[<a href='https://issues.igniterealtime.org/browse/SMACK-779'>SMACK-779</a>] -         smack-android erroneously depends on smack-omemo and smack-omemo-signal
+</li>
+<li>[<a href='https://issues.igniterealtime.org/browse/SMACK-780'>SMACK-780</a>] -         PushNotificationManager&#39;s isSupported logic does query the server, whereas it should query the bare JID
+</li>
+<li>[<a href='https://issues.igniterealtime.org/browse/SMACK-781'>SMACK-781</a>] -          MiniDnsResolver does not correctly handle the case when NOERROR is returned  together with an empty answer section.
+</li>
+<li>[<a href='https://issues.igniterealtime.org/browse/SMACK-782'>SMACK-782</a>] -         MultiUserChat does not remove the subject listener causing a memory leak
+</li>
+<li>[<a href='https://issues.igniterealtime.org/browse/SMACK-783'>SMACK-783</a>] -         InvitationRejectionListener fires multiple times
+</li>
+<li>[<a href='https://issues.igniterealtime.org/browse/SMACK-784'>SMACK-784</a>] -         StringUtils.numbersAndLetters has the numbers twice, resulting in a lower entropy
+</li>
+<li>[<a href='https://issues.igniterealtime.org/browse/SMACK-785'>SMACK-785</a>] -         OfflineMessageManager.getMessages() does count the pending messages incorrectly, causing an unnecessary delay
+</li>
+<li>[<a href='https://issues.igniterealtime.org/browse/SMACK-786'>SMACK-786</a>] -         Race condition when resuming a stream
+</li>
+<li>[<a href='https://issues.igniterealtime.org/browse/SMACK-787'>SMACK-787</a>] -         Presence.getPriority() may return Integer.MIN_VALUE.
+</li>
+</ul>
+
 <h2>4.2.1 -- <span style="font-weight: normal;">2017-08-14</span></h2>
 
 <h2>        Bug

smack-bosh/src/main/java/org/jivesoftware/smack/bosh/XMPPBOSHConnection.java

@@ -432,7 +432,6 @@ public class XMPPBOSHConnection extends AbstractXMPPConnection {
                                     throw new RuntimeException(e);
                                 }
                             }
-                            notifyReconnection();
                     }
                 }
                 else {

smack-core/src/main/java/org/jivesoftware/smack/AbstractConnectionListener.java

@@ -45,16 +45,19 @@ public class AbstractConnectionListener implements ConnectionListener {
         // do nothing
     }
 
+    @Deprecated
     @Override
     public void reconnectingIn(int seconds) {
         // do nothing
     }
 
+    @Deprecated
     @Override
     public void reconnectionFailed(Exception e) {
         // do nothing
     }
 
+    @Deprecated
     @Override
     public void reconnectionSuccessful() {
         // do nothing

smack-core/src/main/java/org/jivesoftware/smack/AbstractXMPPConnection.java

@@ -1241,6 +1241,8 @@ public abstract class AbstractXMPPConnection implements XMPPConnection {
     /**
      * Sends a notification indicating that the connection was reconnected successfully.
      */
+    // TODO: Remove in Smack 4.3
+    @Deprecated
     protected void notifyReconnection() {
         // Notify connection listeners of the reconnection.
         for (ConnectionListener listener : connectionListeners) {

smack-core/src/main/java/org/jivesoftware/smack/ConnectionConfiguration.java

@@ -18,6 +18,7 @@
 package org.jivesoftware.smack;
 
 import java.net.InetAddress;
+import java.security.KeyStore;
 import java.util.Arrays;
 import java.util.Collection;
 import java.util.Collections;
@@ -505,7 +506,7 @@ public abstract class ConnectionConfiguration {
         private SecurityMode securityMode = SecurityMode.ifpossible;
         private DnssecMode dnssecMode = DnssecMode.disabled;
         private String keystorePath = System.getProperty("javax.net.ssl.keyStore");
-        private String keystoreType = "jks";
+        private String keystoreType = KeyStore.getDefaultType();
         private String pkcs11Library = "pkcs11.config";
         private SSLContext customSSLContext;
         private String[] enabledSSLProtocols;

smack-core/src/main/java/org/jivesoftware/smack/ConnectionListener.java

@@ -64,7 +64,10 @@ public interface ConnectionListener {
     /**
      * The connection has reconnected successfully to the server. Connections will
      * reconnect to the server when the previous socket connection was abruptly closed.
+     * @deprecated use {@link #connected(XMPPConnection)} or {@link #authenticated(XMPPConnection, boolean)} instead.
      */
+    // TODO: Remove in Smack 4.3
+    @Deprecated
     public void reconnectionSuccessful();
 
     // The next two methods *must* only be invoked by ReconnectionManager
@@ -78,6 +81,8 @@ public interface ConnectionListener {
      * 
      * @param seconds remaining seconds before attempting a reconnection.
      */
+    // TODO: Remove in Smack 4.3
+    @Deprecated
     public void reconnectingIn(int seconds);
 
     /**
@@ -90,5 +95,7 @@ public interface ConnectionListener {
      *
      * @param e the exception that caused the reconnection to fail.
      */
+    // TODO: Remove in Smack 4.3
+    @Deprecated
     public void reconnectionFailed(Exception e);
 }

smack-core/src/main/java/org/jivesoftware/smack/ReconnectionListener.java

@@ -0,0 +1,50 @@
+/**
+ *
+ * Copyright 2017 Florian Schmaus.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.jivesoftware.smack;
+
+/**
+ * A listener for the {@link ReconnectionManager}. Use
+ * {@link ReconnectionManager#addReconnectionListener(ReconnectionListener)} to add new listeners to the reconnection
+ * manager.
+ *
+ * @since 4.2.2
+ */
+public interface ReconnectionListener {
+
+    /**
+     * The connection will retry to reconnect in the specified number of seconds.
+     * <p>
+     * Note: This method is only called if {@link ReconnectionManager#isAutomaticReconnectEnabled()} returns true, i.e.
+     * only when the reconnection manager is enabled for the connection.
+     * </p>
+     *
+     * @param seconds remaining seconds before attempting a reconnection.
+     */
+    public void reconnectingIn(int seconds);
+
+    /**
+     * An attempt to connect to the server has failed. The connection will keep trying reconnecting to the server in a
+     * moment.
+     * <p>
+     * Note: This method is only called if {@link ReconnectionManager#isAutomaticReconnectEnabled()} returns true, i.e.
+     * only when the reconnection manager is enabled for the connection.
+     * </p>
+     *
+     * @param e the exception that caused the reconnection to fail.
+     */
+    public void reconnectionFailed(Exception e);
+}

smack-core/src/main/java/org/jivesoftware/smack/ReconnectionManager.java

@@ -20,7 +20,9 @@ import java.io.IOException;
 import java.lang.ref.WeakReference;
 import java.util.Map;
 import java.util.Random;
+import java.util.Set;
 import java.util.WeakHashMap;
+import java.util.concurrent.CopyOnWriteArraySet;
 import java.util.logging.Level;
 import java.util.logging.Logger;
 
@@ -103,6 +105,8 @@ public final class ReconnectionManager {
         return enabledPerDefault;
     }
 
+    private final Set<ReconnectionListener> reconnectionListeners = new CopyOnWriteArraySet<>();
+
     // Holds the connection to the server
     private final WeakReference<AbstractXMPPConnection> weakRefConnection;
     private final int randomBase = new Random().nextInt(13) + 2; // between 2 and 15 seconds
@@ -134,6 +138,27 @@ public final class ReconnectionManager {
         defaultReconnectionPolicy = reconnectionPolicy;
     }
 
+    /**
+     * Add a new reconnection listener.
+     *
+     * @param listener the listener to add
+     * @return <code>true</code> if the listener was not already added
+     * @since 4.2.2
+     */
+    public boolean addReconnectionListener(ReconnectionListener listener) {
+        return reconnectionListeners.add(listener);
+    }
+
+    /**
+     * Remove a reconnection listener.
+     * @param listener the listener to remove
+     * @return <code>true</code> if the listener was active and got removed.
+     * @since 4.2.2
+     */
+    public boolean removeReconnectionListener(ReconnectionListener listener) {
+        return reconnectionListeners.remove(listener);
+    }
+
     /**
      * Set the fixed delay in seconds between the reconnection attempts Also set the connection
      * policy to {@link ReconnectionPolicy#FIXED_DELAY}.
@@ -208,6 +233,7 @@ public final class ReconnectionManager {
             /**
              * The process will try the reconnection until the connection succeed or the user cancel it
              */
+            @SuppressWarnings("deprecation")
             @Override
             public void run() {
                 final AbstractXMPPConnection connection = weakRefConnection.get();
@@ -233,6 +259,9 @@ public final class ReconnectionManager {
                         try {
                             Thread.sleep(1000);
                             remainingSeconds--;
+                            for (ReconnectionListener listener : reconnectionListeners) {
+                                listener.reconnectingIn(remainingSeconds);
+                            }
                             for (ConnectionListener listener : connection.connectionListeners) {
                                 listener.reconnectingIn(remainingSeconds);
                             }
@@ -244,6 +273,9 @@ public final class ReconnectionManager {
                         }
                     }
 
+                    for (ReconnectionListener listener : reconnectionListeners) {
+                        listener.reconnectingIn(0);
+                    }
                     for (ConnectionListener listener : connection.connectionListeners) {
                         listener.reconnectingIn(0);
                     }
@@ -269,6 +301,9 @@ public final class ReconnectionManager {
                     }
                     catch (SmackException | IOException | XMPPException e) {
                         // Fires the failed reconnection notification
+                        for (ReconnectionListener listener : reconnectionListeners) {
+                            listener.reconnectionFailed(e);
+                        }
                         for (ConnectionListener listener : connection.connectionListeners) {
                             listener.reconnectionFailed(e);
                         }

smack-core/src/main/java/org/jivesoftware/smack/debugger/AbstractDebugger.java

@@ -18,8 +18,13 @@ package org.jivesoftware.smack.debugger;
 
 import java.io.Reader;
 import java.io.Writer;
+import java.util.logging.Logger;
 
+import org.jivesoftware.smack.AbstractConnectionListener;
+import org.jivesoftware.smack.AbstractXMPPConnection;
 import org.jivesoftware.smack.ConnectionListener;
+import org.jivesoftware.smack.ReconnectionListener;
+import org.jivesoftware.smack.ReconnectionManager;
 import org.jivesoftware.smack.XMPPConnection;
 import org.jivesoftware.smack.packet.TopLevelStreamElement;
 import org.jivesoftware.smack.util.ObservableReader;
@@ -31,9 +36,12 @@ import org.jxmpp.jid.EntityFullJid;
 
 public abstract class AbstractDebugger extends SmackDebugger {
 
+    private static final Logger LOGGER = Logger.getLogger(AbstractDebugger.class.getName());
+
     public static boolean printInterpreted = false;
 
     private final ConnectionListener connListener;
+    private final ReconnectionListener reconnectionListener;
     private final ReaderListener readerListener;
     private final WriterListener writerListener;
 
@@ -63,7 +71,7 @@ public abstract class AbstractDebugger extends SmackDebugger {
         };
         this.writer.addWriterListener(writerListener);
 
-        connListener = new ConnectionListener() {
+        connListener = new AbstractConnectionListener() {
             @Override
             public void connected(XMPPConnection connection) {
                 log("XMPPConnection connected ("
@@ -92,6 +100,9 @@ public abstract class AbstractDebugger extends SmackDebugger {
                         connection +
                         ")", e);
             }
+        };
+
+        reconnectionListener = new ReconnectionListener() {
             @Override
             public void reconnectionFailed(Exception e) {
                 log(
@@ -100,13 +111,6 @@ public abstract class AbstractDebugger extends SmackDebugger {
                         ")", e);
             }
             @Override
-            public void reconnectionSuccessful() {
-                log(
-                        "XMPPConnection reconnected (" +
-                        connection +
-                        ")");
-            }
-            @Override
             public void reconnectingIn(int seconds) {
                 log(
                         "XMPPConnection (" +
@@ -114,6 +118,14 @@ public abstract class AbstractDebugger extends SmackDebugger {
                         ") will reconnect in " + seconds);
             }
         };
+
+        if (connection instanceof AbstractXMPPConnection) {
+            AbstractXMPPConnection abstractXmppConnection = (AbstractXMPPConnection) connection;
+            ReconnectionManager.getInstanceFor(abstractXmppConnection).addReconnectionListener(reconnectionListener);
+        } else {
+            LOGGER.info("The connection instance " + connection
+                            + " is not an instance of AbstractXMPPConnection, thus we can not install the ReconnectionListener");
+        }
     }
 
     protected abstract void log(String logMessage);

smack-core/src/test/java/org/jivesoftware/smack/DummyConnection.java

@@ -88,6 +88,7 @@ public class DummyConnection extends AbstractXMPPConnection {
         user = getUserJid();
     }
 
+    @SuppressWarnings("deprecation")
     @Override
     protected void connectInternal() {
         connected = true;
@@ -95,6 +96,7 @@ public class DummyConnection extends AbstractXMPPConnection {
         tlsHandled.reportSuccess();
         streamId = "dummy-" + new Random(new Date().getTime()).nextInt();
 
+        // TODO: Remove in Smack 4.3, and likely the suppression of the deprecation warning.
         if (reconnect) {
             notifyReconnection();
         }

smack-debug-slf4j/src/main/java/org/jivesoftware/smackx/debugger/slf4j/SLF4JLoggingConnectionListener.java

@@ -17,12 +17,15 @@
 
 package org.jivesoftware.smackx.debugger.slf4j;
 
-import org.jivesoftware.smack.ConnectionListener;
+import org.jivesoftware.smack.AbstractConnectionListener;
+import org.jivesoftware.smack.ReconnectionListener;
 import org.jivesoftware.smack.XMPPConnection;
 
 import org.slf4j.Logger;
 
-class SLF4JLoggingConnectionListener implements ConnectionListener {
+// TODO: The suppression of deprecated API can be removed in Smack 4.3.
+@SuppressWarnings("deprecation")
+class SLF4JLoggingConnectionListener extends AbstractConnectionListener implements ReconnectionListener {
     private final XMPPConnection connection;
     private final Logger logger;
 
@@ -56,11 +59,6 @@ class SLF4JLoggingConnectionListener implements ConnectionListener {
         logger.debug("({}) Reconnection failed due to an exception: {}", connection.hashCode(), e);
     }
 
-    @Override
-    public void reconnectionSuccessful() {
-        logger.debug("({}) Connection reconnected", connection.hashCode());
-    }
-
     @Override
     public void reconnectingIn(int seconds) {
         logger.debug("({}) Connection will reconnect in {}", connection.hashCode(), seconds);

smack-debug-slf4j/src/main/java/org/jivesoftware/smackx/debugger/slf4j/SLF4JSmackDebugger.java

@@ -21,6 +21,8 @@ import java.io.Reader;
 import java.io.Writer;
 import java.util.concurrent.atomic.AtomicBoolean;
 
+import org.jivesoftware.smack.AbstractXMPPConnection;
+import org.jivesoftware.smack.ReconnectionManager;
 import org.jivesoftware.smack.SmackConfiguration;
 import org.jivesoftware.smack.XMPPConnection;
 import org.jivesoftware.smack.debugger.SmackDebugger;
@@ -40,6 +42,8 @@ import org.slf4j.LoggerFactory;
  */
 public class SLF4JSmackDebugger extends SmackDebugger  {
 
+    private static final java.util.logging.Logger LOGGER = java.util.logging.Logger.getLogger(SLF4JSmackDebugger.class.getName());
+
     public static final String LOGGER_NAME = "SMACK";
     private static final Logger logger = LoggerFactory.getLogger(LOGGER_NAME);
     public static final AtomicBoolean printInterpreted = new AtomicBoolean(true);
@@ -70,7 +74,17 @@ public class SLF4JSmackDebugger extends SmackDebugger  {
         this.writer.addWriterListener(slf4JRawXmlListener);
         this.reader = new ObservableReader(Validate.notNull(reader));
         this.reader.addReaderListener(slf4JRawXmlListener);
-        this.connection.addConnectionListener(new SLF4JLoggingConnectionListener(connection, logger));
+
+        final SLF4JLoggingConnectionListener loggingConnectionListener = new SLF4JLoggingConnectionListener(connection, logger);
+        this.connection.addConnectionListener(loggingConnectionListener);
+
+        if (connection instanceof AbstractXMPPConnection) {
+            AbstractXMPPConnection abstractXmppConnection = (AbstractXMPPConnection) connection;
+            ReconnectionManager.getInstanceFor(abstractXmppConnection).addReconnectionListener(loggingConnectionListener);
+        } else {
+            LOGGER.info("The connection instance " + connection
+                            + " is not an instance of AbstractXMPPConnection, thus we can not install the ReconnectionListener");
+        }
     }
 
     @Override

smack-debug/src/main/java/org/jivesoftware/smackx/debugger/EnhancedDebugger.java

@@ -61,7 +61,10 @@ import javax.swing.table.DefaultTableModel;
 import javax.swing.text.BadLocationException;
 
 import org.jivesoftware.smack.AbstractConnectionListener;
+import org.jivesoftware.smack.AbstractXMPPConnection;
 import org.jivesoftware.smack.ConnectionListener;
+import org.jivesoftware.smack.ReconnectionListener;
+import org.jivesoftware.smack.ReconnectionManager;
 import org.jivesoftware.smack.SmackException.NotConnectedException;
 import org.jivesoftware.smack.XMPPConnection;
 import org.jivesoftware.smack.debugger.SmackDebugger;
@@ -143,6 +146,7 @@ public class EnhancedDebugger extends SmackDebugger {
     private JFormattedTextField statusField = null;
 
     private ConnectionListener connListener = null;
+    private final ReconnectionListener reconnectionListener;
 
     private Writer writer;
     private Reader reader;
@@ -169,6 +173,36 @@ public class EnhancedDebugger extends SmackDebugger {
     public EnhancedDebugger(XMPPConnection connection) {
         super(connection);
 
+        reconnectionListener = new ReconnectionListener() {
+            @Override
+            public void reconnectingIn(final int seconds) {
+                SwingUtilities.invokeLater(new Runnable() {
+                    @Override
+                    public void run() {
+                        statusField.setValue("Attempt to reconnect in " + seconds + " seconds");
+                    }
+                });
+            }
+
+            @Override
+            public void reconnectionFailed(Exception e) {
+                SwingUtilities.invokeLater(new Runnable() {
+                    @Override
+                    public void run() {
+                        statusField.setValue("Reconnection failed");
+                    }
+                });
+            }
+        };
+
+        if (connection instanceof AbstractXMPPConnection) {
+            AbstractXMPPConnection abstractXmppConnection = (AbstractXMPPConnection) connection;
+            ReconnectionManager.getInstanceFor(abstractXmppConnection).addReconnectionListener(reconnectionListener);
+        } else {
+            LOGGER.info("The connection instance " + connection
+                            + " is not an instance of AbstractXMPPConnection, thus we can not install the ReconnectionListener");
+        }
+
         // We'll arrange the UI into six tabs. The first tab contains all data, the second
         // client generated XML, the third server generated XML, the fourth allows to send 
         // ad-hoc messages and the fifth contains connection information.
@@ -208,36 +242,6 @@ public class EnhancedDebugger extends SmackDebugger {
                 });
 
             }
-            @Override
-            public void reconnectingIn(final int seconds) {
-                SwingUtilities.invokeLater(new Runnable() {
-                    @Override
-                    public void run() {
-                        statusField.setValue("Attempt to reconnect in " + seconds + " seconds");
-                    }
-                });
-            }
-
-            @Override
-            public void reconnectionSuccessful() {
-                SwingUtilities.invokeLater(new Runnable() {
-                    @Override
-                    public void run() {
-                        statusField.setValue("Reconnection stablished");
-                        EnhancedDebuggerWindow.connectionEstablished(EnhancedDebugger.this);
-                    }
-                });
-            }
-
-            @Override
-            public void reconnectionFailed(Exception e) {
-                SwingUtilities.invokeLater(new Runnable() {
-                    @Override
-                    public void run() {
-                        statusField.setValue("Reconnection failed");
-                    }
-                });
-            }
         };
 
         EnhancedDebuggerWindow.addDebugger(this);

smack-extensions/src/main/java/org/jivesoftware/smackx/bytestreams/ibb/InBandBytestreamManager.java

@@ -120,8 +120,7 @@ public final class InBandBytestreamManager extends Manager implements Bytestream
                     }
 
                     @Override
-                    public void reconnectionSuccessful() {
+                    public void connected(XMPPConnection connection) {
-                        // re-create the manager for this connection
                         InBandBytestreamManager.getByteStreamManager(connection);
                     }
 

smack-extensions/src/main/java/org/jivesoftware/smackx/jingle/transports/JingleTransportManager.java

@@ -16,7 +16,7 @@
  */
 package org.jivesoftware.smackx.jingle.transports;
 
-import org.jivesoftware.smack.ConnectionListener;
+import org.jivesoftware.smack.AbstractConnectionListener;
 import org.jivesoftware.smack.XMPPConnection;
 import org.jivesoftware.smackx.jingle.JingleSession;
 import org.jivesoftware.smackx.jingle.element.JingleContentTransport;
@@ -25,7 +25,7 @@ import org.jivesoftware.smackx.jingle.element.JingleContentTransport;
  * Manager for a JingleTransport method.
  * @param <D> JingleContentTransport.
  */
-public abstract class JingleTransportManager<D extends JingleContentTransport> implements ConnectionListener {
+public abstract class JingleTransportManager<D extends JingleContentTransport> extends AbstractConnectionListener {
 
     private final XMPPConnection connection;
 
@@ -57,18 +57,4 @@ public abstract class JingleTransportManager<D extends JingleContentTransport> i
 
     }
 
-    @Override
-    public void reconnectionSuccessful() {
-
-    }
-
-    @Override
-    public void reconnectingIn(int seconds) {
-
-    }
-
-    @Override
-    public void reconnectionFailed(Exception e) {
-
-    }
 }

smack-tcp/src/main/java/org/jivesoftware/smack/tcp/XMPPTCPConnection.java

@@ -676,10 +676,6 @@ public class XMPPTCPConnection extends AbstractXMPPConnection {
      */
     @SuppressWarnings("LiteralClassName")
     private void proceedTLSReceived() throws NoSuchAlgorithmException, CertificateException, IOException, KeyStoreException, NoSuchProviderException, UnrecoverableKeyException, KeyManagementException, SmackException {
-        SSLContext context = this.config.getCustomSSLContext();
-        KeyStore ks = null;
-        KeyManager[] kms = null;
-        PasswordCallback pcb = null;
         SmackDaneVerifier daneVerifier = null;
 
         if (config.getDnssecMode() == DnssecMode.needsDnssecAndDane) {
@@ -693,6 +689,10 @@ public class XMPPTCPConnection extends AbstractXMPPConnection {
             }
         }
 
+        SSLContext context = this.config.getCustomSSLContext();
+        KeyStore ks = null;
+        PasswordCallback pcb = null;
+
         if (context == null) {
             final String keyStoreType = config.getKeystoreType();
             final CallbackHandler callbackHandler = config.getCallbackHandler();
@@ -737,21 +737,32 @@ public class XMPPTCPConnection extends AbstractXMPPConnection {
                 }
             }
 
+            KeyManager[] kms = null;
+
             if (ks != null) {
                 String keyManagerFactoryAlgorithm = KeyManagerFactory.getDefaultAlgorithm();
-                KeyManagerFactory kmf = KeyManagerFactory.getInstance(keyManagerFactoryAlgorithm);
+                KeyManagerFactory kmf = null;
                 try {
-                    if (pcb == null) {
+                    kmf = KeyManagerFactory.getInstance(keyManagerFactoryAlgorithm);
-                        kmf.init(ks, null);
-                    }
-                    else {
-                        kmf.init(ks, pcb.getPassword());
-                        pcb.clearPassword();
-                    }
-                    kms = kmf.getKeyManagers();
                 }
-                catch (NullPointerException npe) {
+                catch (NoSuchAlgorithmException e) {
-                    LOGGER.log(Level.WARNING, "NullPointerException", npe);
+                    LOGGER.log(Level.FINE, "Could get the default KeyManagerFactory for the '"
+                                    + keyManagerFactoryAlgorithm + "' algorithm", e);
+                }
+                if (kmf != null) {
+                    try {
+                        if (pcb == null) {
+                            kmf.init(ks, null);
+                        }
+                        else {
+                            kmf.init(ks, pcb.getPassword());
+                            pcb.clearPassword();
+                        }
+                        kms = kmf.getKeyManagers();
+                    }
+                    catch (NullPointerException npe) {
+                        LOGGER.log(Level.WARNING, "NullPointerException", npe);
+                    }
                 }
             }