[
https://issues.apache.org/jira/browse/ARTEMIS-2372?focusedWorklogId=254636&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-254636
]
ASF GitHub Bot logged work on ARTEMIS-2372:
-------------------------------------------
Author: ASF GitHub Bot
Created on: 05/Jun/19 20:49
Start Date: 05/Jun/19 20:49
Worklog Time Spent: 10m
Work Description: tabish121 commented on pull request #2695: ARTEMIS-2372
Filtering on Message Annotations
URL: https://github.com/apache/activemq-artemis/pull/2695#discussion_r290927630
##########
File path:
artemis-protocols/artemis-amqp-protocol/src/main/java/org/apache/activemq/artemis/protocol/amqp/broker/AMQPMessage.java
##########
@@ -1263,7 +1263,10 @@ public Long getLongProperty(String key) throws
ActiveMQPropertyConversionExcepti
@Override
public Object getObjectProperty(String key) {
- if (key.equals(MessageUtil.TYPE_HEADER_NAME.toString())) {
+
+ if
(key.startsWith(AMQPMessageSupport.JMS_AMQP_MESSAGE_ANNOTATION_PREFIX)) {
Review comment:
This doesn't seem right at all, as the specification reserves all
Annotations Symbol and UnsignedLong keys for its own use but the broker seems
to be populating annotations that violate that. Any client or other
intermediary would be within its rights (technically required) to close the
connection upon receiving a message with these broker specific message
annotations as per the spec:
←
3.2.10 Annotations - The annotations type is a map where the keys are
restricted to be of type symbol or of type ulong. All ulong keys, and all
symbolic keys except those beginning with "x-" are reserved. Keys beginning
with "x-opt-" MUST be ignored if not understood. On receiving an annotation key
which is not understood, and which does not begin with "x-opt", the receiving
AMQP container MUST detach the link with a not-implemented error.
----------------------------------------------------------------
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:
[email protected]
Issue Time Tracking
-------------------
Worklog Id: (was: 254636)
Time Spent: 0.5h (was: 20m)
> Support Filtering on Message Annotations
> ----------------------------------------
>
> Key: ARTEMIS-2372
> URL: https://issues.apache.org/jira/browse/ARTEMIS-2372
> Project: ActiveMQ Artemis
> Issue Type: Improvement
> Affects Versions: 2.9.0
> Reporter: clebert suconic
> Assignee: clebert suconic
> Priority: Major
> Fix For: 2.10.0
>
> Time Spent: 0.5h
> Remaining Estimate: 0h
>
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
