You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@activemq.apache.org by ma...@apache.org on 2017/12/15 14:31:41 UTC

[2/2] activemq-artemis git commit: [ARTEMIS-1310] [ARTEMIS-1264] - remove some dead code - krb5 login module and jaas callback handler now have the smarts to query netty

[ARTEMIS-1310] [ARTEMIS-1264] - remove some dead code - krb5 login module and jaas callback handler now have the smarts to query netty


Project: http://git-wip-us.apache.org/repos/asf/activemq-artemis/repo
Commit: http://git-wip-us.apache.org/repos/asf/activemq-artemis/commit/7b3b22d1
Tree: http://git-wip-us.apache.org/repos/asf/activemq-artemis/tree/7b3b22d1
Diff: http://git-wip-us.apache.org/repos/asf/activemq-artemis/diff/7b3b22d1

Branch: refs/heads/master
Commit: 7b3b22d1840ab124e7aefcd4a98f54cb8831e62f
Parents: facb644
Author: gtully <ga...@gmail.com>
Authored: Thu Dec 14 11:07:15 2017 +0000
Committer: Martyn Taylor <mt...@redhat.com>
Committed: Fri Dec 15 14:31:16 2017 +0000

----------------------------------------------------------------------
 .../core/remoting/impl/netty/NettyConnection.java        | 11 -----------
 1 file changed, 11 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/activemq-artemis/blob/7b3b22d1/artemis-core-client/src/main/java/org/apache/activemq/artemis/core/remoting/impl/netty/NettyConnection.java
----------------------------------------------------------------------
diff --git a/artemis-core-client/src/main/java/org/apache/activemq/artemis/core/remoting/impl/netty/NettyConnection.java b/artemis-core-client/src/main/java/org/apache/activemq/artemis/core/remoting/impl/netty/NettyConnection.java
index 601bbfc..a73983d 100644
--- a/artemis-core-client/src/main/java/org/apache/activemq/artemis/core/remoting/impl/netty/NettyConnection.java
+++ b/artemis-core-client/src/main/java/org/apache/activemq/artemis/core/remoting/impl/netty/NettyConnection.java
@@ -497,17 +497,6 @@ public class NettyConnection implements Connection {
    //never allow this
    @Override
    public final ActiveMQPrincipal getDefaultActiveMQPrincipal() {
-      ChannelHandler channelHandler = channel.pipeline().get("ssl");
-      if (channelHandler != null && channelHandler instanceof SslHandler) {
-         SslHandler sslHandler = (SslHandler) channelHandler;
-         try {
-            return new ActiveMQPrincipal(sslHandler.engine().getSession().getPeerPrincipal().getName(), "");
-         } catch (SSLPeerUnverifiedException ignored) {
-            if (logger.isTraceEnabled()) {
-               logger.trace(ignored.getMessage(), ignored);
-            }
-         }
-      }
       return null;
    }