Bug 1004715 - CLI - any string is allowed to set for message selector on queue
CLI - any string is allowed to set for message selector on queue
Status: CLOSED DUPLICATE of bug 1004789
Product: JBoss Enterprise Application Platform 6
Classification: JBoss
Component: HornetQ (Show other bugs)
Unspecified Unspecified
unspecified Severity low
: ---
: ---
Assigned To: Jeff Mesnil
Miroslav Novak
Russell Dickenson
Depends On:
  Show dependency treegraph
Reported: 2013-09-05 06:23 EDT by Miroslav Novak
Modified: 2013-09-18 15:54 EDT (History)
5 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2013-09-18 02:46:35 EDT
Type: Bug
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)

  None (edit)
Description Miroslav Novak 2013-09-05 06:23:57 EDT
Description of problem:
Using CLI is possible to set message selector on queue to any string value. Such selector does not have to correspond to message selector syntax.

How reproducible:

Add queue:


Set message selector with bad value:

    "outcome" => "success",
    "response-headers" => {
        "operation-requires-reload" => true,
        "process-state" => "reload-required"

Expected results:
This should fail with description of proper grammar.
Comment 1 Jeff Mesnil 2013-09-05 06:26:35 EDT
HornetQ does not expose any functionality to determine the correctness of a message selector.

Either it needs to provide it or it should check it at runtime when the queue is created.
Comment 2 Miroslav Novak 2013-09-05 07:29:05 EDT
JMS specification 1.1 [1] describes message selector syntax  in chapter: Message Selector Syntax

[1] http://docs.oracle.com/cd/E19957-01/816-5904-10/816-5904-10.pdf
Comment 3 Clebert Suconic 2013-09-17 17:22:13 EDT
I believe had already changed this on wildfly/master.

The issue here is that a queue's filter is a read only property.

Was this changed on 6.x as well?  if so, this is just a close
Comment 4 Miroslav Novak 2013-09-18 02:46:35 EDT
Yes, message-selector attribute should be read-only. Jeff has already created PR in bz#1004789.

I'll close this bz as duplicate.

*** This bug has been marked as a duplicate of bug 1004789 ***

Note You need to log in before you can comment on or make changes to this bug.