[ 
https://issues.apache.org/jira/browse/ARTEMIS-1916?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16508219#comment-16508219
 ] 

ASF GitHub Bot commented on ARTEMIS-1916:
-----------------------------------------

Github user andytaylor commented on the issue:

    https://github.com/apache/activemq-artemis/pull/2128
  
    I would create a single PR.
    
    Also Im not sure this is a complete fix. There is still code in there that 
sets up an SSL socket layer. 
    
    Also I think we need to understand why the code as is doesn't work


> Remove Jmx ArtemisRMIServerSocketFactory
> ----------------------------------------
>
>                 Key: ARTEMIS-1916
>                 URL: https://issues.apache.org/jira/browse/ARTEMIS-1916
>             Project: ActiveMQ Artemis
>          Issue Type: Improvement
>          Components: Web Console
>    Affects Versions: 2.6.1
>            Reporter: Howard Gao
>            Assignee: Howard Gao
>            Priority: Major
>             Fix For: 2.6.2
>
>
> The ArtemisRMIServerSocketFactory doesn't do anything special, instead the 
> existence of this impl class causes jmx client failed to connect (for reason 
> not known, probably not fully implemented the functionality). It turns out 
> just fine to use JDK's impl. This class is not necessary.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to