You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@qpid.apache.org by Carl Trieloff <cc...@redhat.com> on 2011/05/12 20:11:38 UTC

why do we have this code

In Exchnage.cpp, we have an ACL check for passive...

why it it there, as all exchange create calls come through declare which
also has the ACL check, any ideas before I delete this code?

Carl.

---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project:      http://qpid.apache.org
Use/Interact: mailto:dev-subscribe@qpid.apache.org


Re: why do we have this code

Posted by Gordon Sim <gs...@redhat.com>.
On 05/12/2011 07:11 PM, Carl Trieloff wrote:
>
> In Exchnage.cpp, we have an ACL check for passive...
>
> why it it there, as all exchange create calls come through declare which
> also has the ACL check, any ideas before I delete this code?

On a related point, why does a passive declare require a check for the 
'create' permission for an exchange? (Line 86 of 
qpid/broker/SessionAdapter.cpp)

The purpose of the passive flag is to state that the exchange should 
*not* created. For authorisation a passive declare is similar to 
exchange-query.

Likewise for queues (line 296 of the same file).

---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project:      http://qpid.apache.org
Use/Interact: mailto:dev-subscribe@qpid.apache.org


Re: why do we have this code

Posted by Rajith Attapattu <ra...@gmail.com>.
Sorry I couldn't fine anything in Exchange.cpp
Could you pls point me to it ?

Regards,

Rajith

On Thu, May 12, 2011 at 2:11 PM, Carl Trieloff <cc...@redhat.com> wrote:
>
> In Exchnage.cpp, we have an ACL check for passive...
>
> why it it there, as all exchange create calls come through declare which
> also has the ACL check, any ideas before I delete this code?
>
> Carl.
>
> ---------------------------------------------------------------------
> Apache Qpid - AMQP Messaging Implementation
> Project:      http://qpid.apache.org
> Use/Interact: mailto:dev-subscribe@qpid.apache.org
>
>

---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project:      http://qpid.apache.org
Use/Interact: mailto:dev-subscribe@qpid.apache.org