[jboss-cvs] JBossRemoting/src/main/org/jboss/remoting ...

[Tom Elrod]

  User: telrod  
  Date: 06/08/15 12:33:23

  Modified:    src/main/org/jboss/remoting  Client.java
  Log:
  JBREM-580 - fixed connection validator so that will not do ping if is ssl transport.
  
  Revision  Changes    Path
  1.47      +17 -1     JBossRemoting/src/main/org/jboss/remoting/Client.java
  
  (In the diff below, changes in quantity of whitespace are not shown.)
  
  Index: Client.java
  ===================================================================
  RCS file: /cvsroot/jboss/JBossRemoting/src/main/org/jboss/remoting/Client.java,v
  retrieving revision 1.46
  retrieving revision 1.47
  diff -u -b -r1.46 -r1.47
  --- Client.java	27 Jul 2006 19:54:58 -0000	1.46
  +++ Client.java	15 Aug 2006 16:33:23 -0000	1.47
  @@ -59,7 +59,7 @@
    *
    * @author <a href="mailto:jhaynie at vocalocity.net">Jeff Haynie</a>
    * @author <a href="mailto:telrod at e2technologies.net">Tom Elrod</a>
  - * @version $Revision: 1.46 $
  + * @version $Revision: 1.47 $
    */
   public class Client implements Externalizable
   {
  @@ -288,6 +288,22 @@
            {
               return;
            }
  +
  +         // added for JBREM-580 (don't use connection listener if is ssl)
  +         String transport = invoker.getLocator().getProtocol();
  +         try
  +         {
  +            boolean isSSL = InvokerRegistry.isSSLSupported(transport);
  +            if(isSSL)
  +            {
  +               return;
  +            }
  +         }
  +         catch(Exception e)
  +         {
  +            log.warn("Error when trying to determine if transport (" + transport + ") supports ssl when creating connection listener.");
  +            log.debug(e.getMessage(), e);
  +         }
         }
         if (connectionValidator == null)
         {