gemmellr commented on PR #6021: URL: https://github.com/apache/activemq-artemis/pull/6021#issuecomment-3492405236
Beyond those comments, I was surprised I didnt see any test changes around the change in Netty default for hostname verification in 4.2. Either the Artemis code already handles [enabling and] disabling hostname in the required manner in advance to still work properly with the updated behaviour, which I dont recall seeing it do so, or perhaps it might suggest there is a missing test around explicitly disabling it. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected] For further information, visit: https://activemq.apache.org/contact
