[GitHub] [activemq-artemis] jbertram opened a new pull request #3001: ARTEMIS-2639 lost notif props with OpenWire+divert

[GitBox <gi...@apache.org>]

jbertram opened a new pull request #3001: ARTEMIS-2639 lost notif props with OpenWire+divert
URL: https://github.com/apache/activemq-artemis/pull/3001
 
 
   Historically speaking, all message properties starting with AMQ HDR
   would not be passed to OpenWire messages. However, that blocked the
   properties from management notifications so ARTEMIS-1209 was raised and
   the solution there was to pass properties that started with _AMQ *if*
   the consumer was connected to the management notification address.
   However, in this case messages are diverted to a different address so
   this check fails and the properties are removed. My solution will be to
   check the message itself to see if it has the _AMQ_NotifType property
   (which all notification messages do) rather than checking where the
   consumer is connected.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [activemq-artemis] asfgit closed pull request #3001: ARTEMIS-2639 lost notif props with OpenWire+divert

[GitBox <gi...@apache.org>]

asfgit closed pull request #3001: ARTEMIS-2639 lost notif props with OpenWire+divert
URL: https://github.com/apache/activemq-artemis/pull/3001
 
 
   

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services