[ 
https://issues.apache.org/jira/browse/QPID-992?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12593555#action_12593555
 ] 

Martin Ritchie commented on QPID-992:
-------------------------------------

The cause of this failure is the change in the getRoutingKey() value on M2.1 we 
used the queueName which was tmp_<UUID> but now on trunk the routing key is now 
the client generated routingKey hence the requirement to change tmp_* to 
TempQueue* in the acl.config. 

NOTE: this should be a _change_ not an add. 


    public AMQShortString getRoutingKey()
    {
        //return getAMQQueueName();
        if (getAMQQueueName() != null && 
getAMQQueueName().equals(super.getRoutingKey()))
        {
            return getAMQQueueName();
        }
        else
        {
            return super.getRoutingKey();
        }
    }




> SimpleACLTest fails in trunk
> ----------------------------
>
>                 Key: QPID-992
>                 URL: https://issues.apache.org/jira/browse/QPID-992
>             Project: Qpid
>          Issue Type: Bug
>          Components: Java Tests
>    Affects Versions: M3
>            Reporter: Rajith Attapattu
>            Assignee: Rajith Attapattu
>             Fix For: M3
>
>
> The org.apache.qpid.server.security.acl.SimpleACLTest in systests is failing 
> as the temporary queue created has the form TempQueue*.
> The acl.config under PUBLISH, for user "server"  has permission for routing 
> keys of the form tmp_*.
> In order to get the test passing I will add TempQueue* under routing keys 
> allowed for publish for user "server"
> However I believe we had a thread discussing about how the java client should 
> name temp queues. 
> Once we come to a consensus and it is implemented, I will update the 
> acl.config to reflect that.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to