qpid-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Martin Ritchie (JIRA)" <qpid-...@incubator.apache.org>
Subject [jira] Commented: (QPID-992) SimpleACLTest fails in trunk
Date Thu, 01 May 2008 15:22:55 GMT

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


Mime
View raw message