cshannon commented on PR #1851: URL: https://github.com/apache/activemq/pull/1851#issuecomment-4148391962
@arnoudja - I'm glad to hear that fixed it for AMQP. So does this mean you are not running this patch https://github.com/apache/activemq/commit/57264bf8dc9970d7d808cf24216b6185ceb644e2 ? That patch is basically the same solution for Stomp. I took a look at Stomp and during send to consumers a copy is correctly done before touching the body, but it looks like the message headers are deserialized without a copy first so that might be something we need to fix (by moving the copy earlier). If you are not running https://github.com/apache/activemq/commit/57264bf8dc9970d7d808cf24216b6185ceb644e2 then that patch should fix it for Stomp as well. -- 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
