#189: configure session with operational configuration just the same as when sending an email, thereby adding the missing time-out properties when testing a connection

Author: Benny Bottema

src/main/java/org/simplejavamail/mailer/internal/mailsender/MailSender.java

@@ -243,7 +243,7 @@ private void sendMailClosure(@Nonnull final Session session, @Nonnull final Emai
 
 			configureBounceToAddress(session, email);
 
-			logSession(session);
+			logSession(session, "mail");
 			message.saveChanges(); // some headers and id's will be set for this specific message
 			//noinspection deprecation
 			email.internalSetId(message.getMessageID());
@@ -324,11 +324,11 @@ private synchronized void checkShutDownRunningProcesses() {
 	/**
 	 * Simply logs host details, credentials used and whether authentication will take place and finally the transport protocol used.
 	 */
-	private static void logSession(final Session session) {
+	private static void logSession(final Session session, final String activity) {
 		final TransportStrategy transportStrategy = TransportStrategy.findStrategyForSession(session);
 		final Properties properties = session.getProperties();
 		final String sessionDetails = (transportStrategy != null) ? transportStrategy.toString(properties) : properties.toString();
-		LOGGER.debug("starting mail with " + sessionDetails);
+		LOGGER.debug("starting {} with {}", activity, sessionDetails);
 	}
 
 	/**
@@ -369,7 +369,11 @@ private void trustHosts(@Nonnull final List<String>  hosts) {
 	 */
 	public synchronized void testConnection() {
 		boolean proxyBridgeStartedForTestingConnection = false;
-		
+
+		configureSessionWithTimeout(session, operationalConfig.getSessionTimeout());
+
+		logSession(session, "connection test");
+
 		try (Transport transport = session.getTransport()) {
 			//noinspection ConstantConditions
 			if (needsAuthenticatedProxy() && !proxyServer.isRunning()) {