gemmellr commented on PR #5359: URL: https://github.com/apache/activemq-artemis/pull/5359#issuecomment-2490991954
Thinking that @clebertsuconic meant to post these comments here instead of #5363: https://github.com/apache/activemq-artemis/pull/5363#issuecomment-2489998018 > 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. https://github.com/apache/activemq-artemis/pull/5363#issuecomment-2489998328 > It'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
