mklaca commented on a change in pull request #76:
URL: https://github.com/apache/qpid-broker-j/pull/76#discussion_r595077327
##########
File path:
broker-plugins/amqp-0-10-protocol/src/main/java/org/apache/qpid/server/protocol/v0_10/ServerConnectionDelegate.java
##########
@@ -269,8 +270,22 @@ public void connectionOpen(ServerConnection sconn,
ConnectionOpen open)
try
{
+ final AMQPConnection_0_10 amqpConnection =
sconn.getAmqpConnection();
+ final Principal principal = sconn.getAuthorizedPrincipal();
+ if (principal != null)
Review comment:
That would be a change of the existing logic, it is not part of this
feature. If it was required then another bug task fix should be raised.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
