Uploaded image for project: 'JBoss Remoting (3+)'
  1. JBoss Remoting (3+)
  2. REM3-173

Store more information about the connection as principals

XMLWordPrintable

    • Icon: Feature Request Feature Request
    • Resolution: Unresolved
    • Icon: Major Major
    • None
    • 4.0.0.Beta1
    • None

      We currently capture this data

              private Collection<Principal> createPrincipals() {
                  final Set<Principal> principals = new LinkedHashSet<Principal>();
      
                  final SslChannel sslChannel = connection.getSslChannel();
                  if (sslChannel != null) {
                      // It might be STARTTLS, in which case we can still opt out of SSL
                      final SSLSession session = sslChannel.getSslSession();
                      if (session != null) {
                          try {
                              principals.add(session.getPeerPrincipal());
                          } catch (SSLPeerUnverifiedException ignored) {
                          }
                      }
                  }
                  String authorizationId = saslServer.getAuthorizationID();
                  if (authorizationId != null) {
                      principals.add(new UserPrincipal(authorizationId));
                  }
                  final ConnectedMessageChannel channel = connection.getChannel();
                  final InetSocketAddress address = channel.getPeerAddress(InetSocketAddress.class);
                  if (address != null) {
                      principals.add(new InetAddressPrincipal(address.getAddress()));
                  }
      
                  return Collections.unmodifiableCollection(principals);
              }
      
          }
      

      We should also capture the local address and port.

              Unassigned Unassigned
              kkhan1@redhat.com Kabir Khan
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Created:
                Updated: