[jira] Commented: (QPID-992) SimpleACLTest fails in trunk

["Martin Ritchie (JIRA)" <qp...@incubator.apache.org>]

    [ 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.