diff options
| author | Carl C. Trieloff <cctrieloff@apache.org> | 2008-07-31 20:45:44 +0000 |
|---|---|---|
| committer | Carl C. Trieloff <cctrieloff@apache.org> | 2008-07-31 20:45:44 +0000 |
| commit | 384cd0a4681418fe03fc6121ee71eebc3878b9d4 (patch) | |
| tree | ce77cb966d8a1a21cc8f021e94b076e6c3fc70da /cpp/src/qpid/broker | |
| parent | 240358bf1ded2bc076e96892592e0028f5c277c4 (diff) | |
| download | qpid-python-384cd0a4681418fe03fc6121ee71eebc3878b9d4.tar.gz | |
- Implementation of ACL plugin
- Apply ACL to Exchange, Queue, Binding, Subscribe
- Follow Java ACL types, few added
To complete the implementation of ACL the following items are remaining.
- ACL on message transfer
- ACL on MGNT commands
- Reading ACL from File, into auth map.
git-svn-id: https://svn.apache.org/repos/asf/incubator/qpid/trunk/qpid@681479 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'cpp/src/qpid/broker')
| -rw-r--r-- | cpp/src/qpid/broker/Broker.cpp | 1 | ||||
| -rw-r--r-- | cpp/src/qpid/broker/Broker.h | 51 | ||||
| -rw-r--r-- | cpp/src/qpid/broker/SessionAdapter.cpp | 103 |
3 files changed, 134 insertions, 21 deletions
diff --git a/cpp/src/qpid/broker/Broker.cpp b/cpp/src/qpid/broker/Broker.cpp index ae24283651..d876f6d253 100644 --- a/cpp/src/qpid/broker/Broker.cpp +++ b/cpp/src/qpid/broker/Broker.cpp @@ -128,6 +128,7 @@ Broker::Broker(const Broker::Options& conf) : config(conf), managementAgentSingleton(!config.enableMgmt), store(0), + acl(0), dataDir(conf.noDataDir ? std::string () : conf.dataDir), links(this), factory(*this), diff --git a/cpp/src/qpid/broker/Broker.h b/cpp/src/qpid/broker/Broker.h index 741447b6c0..369902bbea 100644 --- a/cpp/src/qpid/broker/Broker.h +++ b/cpp/src/qpid/broker/Broker.h @@ -45,6 +45,7 @@ #include "qpid/framing/ProtocolInitiation.h" #include "qpid/sys/Runnable.h" #include "qpid/RefCounted.h" +#include "AclModule.h" #include <boost/intrusive_ptr.hpp> #include <vector> @@ -88,7 +89,33 @@ class Broker : public sys::Runnable, public Plugin::Target, size_t replayHardLimit; uint queueLimit; }; - + + private: + boost::shared_ptr<sys::Poller> poller; + Options config; + management::ManagementAgent::Singleton managementAgentSingleton; + std::vector< boost::shared_ptr<sys::ProtocolFactory> > protocolFactories; + MessageStore* store; + AclModule* acl; + DataDir dataDir; + + QueueRegistry queues; + ExchangeRegistry exchanges; + LinkRegistry links; + ConnectionFactory factory; + DtxManager dtxManager; + SessionManager sessionManager; + management::ManagementAgent* managementAgent; + management::Broker* mgmtObject; + Vhost::shared_ptr vhostObject; + System::shared_ptr systemObject; + + void declareStandardExchange(const std::string& name, const std::string& type); + + + public: + + virtual ~Broker(); Broker(const Options& configuration); @@ -114,6 +141,8 @@ class Broker : public sys::Runnable, public Plugin::Target, void setStore (MessageStore*); MessageStore& getStore() { return *store; } + void setAcl (AclModule* _acl) {acl = _acl;} + AclModule* getAcl() { return acl; } QueueRegistry& getQueues() { return queues; } ExchangeRegistry& getExchanges() { return exchanges; } LinkRegistry& getLinks() { return links; } @@ -147,26 +176,6 @@ class Broker : public sys::Runnable, public Plugin::Target, // For the present just return the first ProtocolFactory registered. boost::shared_ptr<sys::ProtocolFactory> getProtocolFactory() const; - private: - boost::shared_ptr<sys::Poller> poller; - Options config; - management::ManagementAgent::Singleton managementAgentSingleton; - std::vector< boost::shared_ptr<sys::ProtocolFactory> > protocolFactories; - MessageStore* store; - DataDir dataDir; - - QueueRegistry queues; - ExchangeRegistry exchanges; - LinkRegistry links; - ConnectionFactory factory; - DtxManager dtxManager; - SessionManager sessionManager; - management::ManagementAgent* managementAgent; - management::Broker* mgmtObject; - Vhost::shared_ptr vhostObject; - System::shared_ptr systemObject; - - void declareStandardExchange(const std::string& name, const std::string& type); }; }} diff --git a/cpp/src/qpid/broker/SessionAdapter.cpp b/cpp/src/qpid/broker/SessionAdapter.cpp index a978762674..0c0d7ff70a 100644 --- a/cpp/src/qpid/broker/SessionAdapter.cpp +++ b/cpp/src/qpid/broker/SessionAdapter.cpp @@ -53,6 +53,18 @@ void SessionAdapter::ExchangeHandlerImpl::declare(const string& exchange, const const string& alternateExchange, bool passive, bool durable, bool /*autoDelete*/, const FieldTable& args){ + AclModule* acl = getBroker().getAcl(); + if (acl) + { + std::map<std::string, std::string> params; + params.insert(make_pair("TYPE", type)); + params.insert(make_pair("ALT", alternateExchange)); + params.insert(make_pair("PAS", std::string(passive ? "Y" : "N") )); + params.insert(make_pair("DURA", std::string(durable ? "Y" : "N"))); + if (!acl->authorise(getConnection().getUserId(),acl::CREATE,acl::EXCHANGE,exchange,¶ms) ) + throw NotAllowedException("ACL denied exhange declare request"); + } + //TODO: implement autoDelete Exchange::shared_ptr alternate; if (!alternateExchange.empty()) { @@ -100,6 +112,15 @@ void SessionAdapter::ExchangeHandlerImpl::checkAlternate(Exchange::shared_ptr ex } void SessionAdapter::ExchangeHandlerImpl::delete_(const string& name, bool /*ifUnused*/){ + + AclModule* acl = getBroker().getAcl(); + if (acl) + { + if (!acl->authorise(getConnection().getUserId(),acl::DELETE,acl::EXCHANGE,name,NULL) ) + throw NotAllowedException("ACL denied exhange delete request"); + } + + //TODO: implement unused Exchange::shared_ptr exchange(getBroker().getExchanges().get(name)); if (exchange->inUseAsAlternate()) throw NotAllowedException(QPID_MSG("Exchange in use as alternate-exchange.")); @@ -110,6 +131,14 @@ void SessionAdapter::ExchangeHandlerImpl::delete_(const string& name, bool /*ifU ExchangeQueryResult SessionAdapter::ExchangeHandlerImpl::query(const string& name) { + + AclModule* acl = getBroker().getAcl(); + if (acl) + { + if (!acl->authorise(getConnection().getUserId(),acl::ACCESS,acl::EXCHANGE,name,NULL) ) + throw NotAllowedException("ACL denied exhange query request"); + } + try { Exchange::shared_ptr exchange(getBroker().getExchanges().get(name)); return ExchangeQueryResult(exchange->getType(), exchange->isDurable(), false, exchange->getArgs()); @@ -121,6 +150,15 @@ void SessionAdapter::ExchangeHandlerImpl::bind(const string& queueName, const string& exchangeName, const string& routingKey, const FieldTable& arguments){ + AclModule* acl = getBroker().getAcl(); + if (acl) + { + std::map<std::string, std::string> params; + params.insert(make_pair("RKEY", routingKey)); + if (!acl->authorise(getConnection().getUserId(),acl::BIND,acl::EXCHANGE,exchangeName,¶ms) ) + throw NotAllowedException("ACL denied exhange bind request"); + } + Queue::shared_ptr queue = getQueue(queueName); Exchange::shared_ptr exchange = getBroker().getExchanges().get(exchangeName); if(exchange){ @@ -142,6 +180,18 @@ SessionAdapter::ExchangeHandlerImpl::unbind(const string& queueName, const string& exchangeName, const string& routingKey) { + + AclModule* acl = getBroker().getAcl(); + if (acl) + { + std::map<std::string, std::string> params; + params.insert(make_pair("QN", queueName)); + params.insert(make_pair("RKEY", routingKey)); + if (!acl->authorise(getConnection().getUserId(),acl::UNBIND,acl::EXCHANGE,exchangeName,¶ms) ) + throw NotAllowedException("ACL denied exchange unbind request"); + } + + Queue::shared_ptr queue = getQueue(queueName); if (!queue.get()) throw NotFoundException("Unbind failed. No such exchange: " + exchangeName); @@ -160,6 +210,16 @@ ExchangeBoundResult SessionAdapter::ExchangeHandlerImpl::bound(const std::string const std::string& key, const framing::FieldTable& args) { + AclModule* acl = getBroker().getAcl(); + if (acl) + { + std::map<std::string, std::string> params; + params.insert(make_pair("QUEUE", queueName)); + params.insert(make_pair("RKEY", queueName)); + if (!acl->authorise(getConnection().getUserId(),acl::CREATE,acl::EXCHANGE,exchangeName,¶ms) ) + throw NotAllowedException("ACL denied exhange bound request"); + } + Exchange::shared_ptr exchange; try { exchange = getBroker().getExchanges().get(exchangeName); @@ -220,6 +280,13 @@ bool SessionAdapter::QueueHandlerImpl::isLocal(const ConnectionToken* t) const QueueQueryResult SessionAdapter::QueueHandlerImpl::query(const string& name) { + AclModule* acl = getBroker().getAcl(); + if (acl) + { + if (!acl->authorise(getConnection().getUserId(),acl::ACCESS,acl::QUEUE,name,NULL) ) + throw NotAllowedException("ACL denied queue query request"); + } + Queue::shared_ptr queue = session.getBroker().getQueues().find(name); if (queue) { @@ -242,6 +309,19 @@ void SessionAdapter::QueueHandlerImpl::declare(const string& name, const string& bool passive, bool durable, bool exclusive, bool autoDelete, const qpid::framing::FieldTable& arguments){ + AclModule* acl = getBroker().getAcl(); + if (acl) + { + std::map<std::string, std::string> params; + params.insert(make_pair("ALT", alternateExchange)); + params.insert(make_pair("PAS", std::string(passive ? "Y" : "N") )); + params.insert(make_pair("DURA", std::string(durable ? "Y" : "N"))); + params.insert(make_pair("EXCLUS", std::string(exclusive ? "Y" : "N"))); + params.insert(make_pair("AUTOD", std::string(autoDelete ? "Y" : "N"))); + if (!acl->authorise(getConnection().getUserId(),acl::CREATE,acl::QUEUE,name,¶ms) ) + throw NotAllowedException("ACL denied queue create request"); + } + Exchange::shared_ptr alternate; if (!alternateExchange.empty()) { alternate = getBroker().getExchanges().get(alternateExchange); @@ -289,10 +369,24 @@ void SessionAdapter::QueueHandlerImpl::declare(const string& name, const string& void SessionAdapter::QueueHandlerImpl::purge(const string& queue){ + AclModule* acl = getBroker().getAcl(); + if (acl) + { + if (!acl->authorise(getConnection().getUserId(),acl::DELETE,acl::QUEUE,queue,NULL) ) + throw NotAllowedException("ACL denied queue purge request"); + } getQueue(queue)->purge(); } void SessionAdapter::QueueHandlerImpl::delete_(const string& queue, bool ifUnused, bool ifEmpty){ + + AclModule* acl = getBroker().getAcl(); + if (acl) + { + if (!acl->authorise(getConnection().getUserId(),acl::DELETE,acl::QUEUE,queue,NULL) ) + throw NotAllowedException("ACL denied queue delete request"); + } + ChannelException error(0, ""); Queue::shared_ptr q = getQueue(queue); if(ifEmpty && q->getMessageCount() > 0){ @@ -347,6 +441,15 @@ SessionAdapter::MessageHandlerImpl::subscribe(const string& queueName, uint64_t /*resumeTtl*/, const FieldTable& arguments) { + + AclModule* acl = getBroker().getAcl(); + if (acl) + { + // add flags as needed + if (!acl->authorise(getConnection().getUserId(),acl::CONSUME,acl::QUEUE,queueName,NULL) ) + throw NotAllowedException("ACL denied Session subscribe request"); + } + Queue::shared_ptr queue = getQueue(queueName); if(!destination.empty() && state.exists(destination)) throw NotAllowedException(QPID_MSG("Consumer tags must be unique")); |
