cardamon commented on PR #4820: URL: https://github.com/apache/activemq-artemis/pull/4820#issuecomment-2061015235
FYI this introduces a breaking change that should probably be documented @ https://activemq.apache.org/components/artemis/documentation/latest/versions.html#upgrading-from-2-32-0 If you're embedding the broker, and using a custom `org.apache.activemq.artemis.spi.core.security.ActiveMQSecurityManager5` implementation, one must now either override `org.apache.activemq.artemis.spi.core.security.ActiveMQSecurityManager#getUserFromSubject` or ensure that `org.apache.activemq.artemis.spi.core.security.jaas.UserPrincipal` is used in the return value of the implementation of `org.apache.activemq.artemis.spi.core.security.ActiveMQSecurityManager5#authenticate`. Not doing so will cause authentication failures (`AMQ222216`) that can be quite difficult to debug ... -- 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
