activemq-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Torsten Mielke (Commented) (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (AMQ-2287) ActiveMQMessage.setXXXProperty() method needs to check if property name is a valid Java identifier.
Date Tue, 06 Mar 2012 11:10:58 GMT

    [ https://issues.apache.org/jira/browse/AMQ-2287?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13223161#comment-13223161
] 

Torsten Mielke commented on AMQ-2287:
-------------------------------------

I am going to close this bug. 
If we decide to fix this, it can likely cause backward compatibility problems. 
Given that there was no demand from the community to have this bug fixed, we will not fix
this problem right now in favor of compatibility.

If there is need to have this fixed in future, this bug can be reopened again.

                
> ActiveMQMessage.setXXXProperty() method needs to check if property name is a valid Java
identifier.
> ---------------------------------------------------------------------------------------------------
>
>                 Key: AMQ-2287
>                 URL: https://issues.apache.org/jira/browse/AMQ-2287
>             Project: ActiveMQ
>          Issue Type: Bug
>          Components: Broker
>    Affects Versions: 5.2.0
>            Reporter: Torsten Mielke
>             Fix For: 5.x
>
>
> currently the AMQ implementation of javax.jms.Message.setXXXProperty() does not check
if the property name is a valid Java identifier. 
> This however is required according to the JMS spec.
> From http://docs.oracle.com/javaee/6/api/javax/jms/Message.html:
> {quote}
> Property names must obey the rules for a message selector identifier. 
> ...
> Identifiers:
>     * An identifier is an unlimited-length sequence of letters and digits, the first
of which must be a letter. A letter is any character for which the method Character.isJavaLetter
returns true. This includes '_' and '$'. A letter or digit is any character for which the
method Character.isJavaLetterOrDigit returns true.
>     * Identifiers cannot be the names NULL, TRUE, and FALSE.
>     * Identifiers cannot be NOT, AND, OR, BETWEEN, LIKE, IN, IS, or ESCAPE.
>     * Identifiers are either header field references or property references. The type
of a property value in a message selector corresponds to the type used to set the property.
If a property that does not exist in a message is referenced, its value is NULL.
>     * The conversions that apply to the get methods for properties do not apply when
a property is used in a message selector expression. For example, suppose you set a property
as a string value, as in the following:
>       myMessage.setStringProperty("NumberOfOrders", "2");
>       The following expression in a message selector would evaluate to false, because
a string cannot be used in an arithmetic expression:
>       "NumberOfOrders > 1"
>     * Identifiers are case-sensitive.
>     * Message header field references are restricted to JMSDeliveryMode, JMSPriority,
JMSMessageID, JMSTimestamp, JMSCorrelationID, and JMSType. JMSMessageID, JMSCorrelationID,
and JMSType values may be null and if so are treated as a NULL value.
>     * Any name beginning with 'JMSX' is a JMS defined property name.
>     * Any name beginning with 'JMS_' is a provider-specific property name.
>     * Any name that does not begin with 'JMS' is an application-specific property name.

> {quote}
> Checks for this need to be added to all ActiveMQMessage.setXXXProperty() methods.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Mime
View raw message