clebertsuconic commented on PR #5359:
URL: 
https://github.com/apache/activemq-artemis/pull/5359#issuecomment-2491054878

   I really liked this...
   
   can I ask a small change.. if you can't do it.. I will do it later...
   
   if the user is null.. just don't set the user
   
   If I have a broker without an user (anonymous user).. it's currently showing 
null@tcp://localhost:61616 >
   
   I think it should use just tcp://localhost:61616 >
   
   although the check that you have on the Exception would need to be 
different.. perhaps just use a boolean to connected instead.
   
   
   
   t's currently showing this, when null:
   
   null@tcp://localhost:61616 >


-- 
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.

To unsubscribe, e-mail: gitbox-unsubscr...@activemq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscr...@activemq.apache.org
For additional commands, e-mail: gitbox-h...@activemq.apache.org
For further information, visit: https://activemq.apache.org/contact


Reply via email to