Commit 616537c5 authored by Stephen Morris's avatar Stephen Morris
Browse files

Merge branch 'master' into trac1025

parents 7c7238ca b5bbfb2f
......@@ -25,6 +25,7 @@
#include <acl/dns.h>
#include <acl/ip_check.h>
#include <acl/loader.h>
#include <acl/logic_check.h>
using namespace std;
using boost::shared_ptr;
......@@ -98,6 +99,15 @@ getRequestLoader() {
// Register default check creator(s)
loader_ptr->registerCreator(shared_ptr<internal::RequestCheckCreator>(
new internal::RequestCheckCreator()));
loader_ptr->registerCreator(
shared_ptr<NotCreator<RequestContext> >(
new NotCreator<RequestContext>("NOT")));
loader_ptr->registerCreator(
shared_ptr<LogicCreator<AnyOfSpec, RequestContext> >(
new LogicCreator<AnyOfSpec, RequestContext>("ANY")));
loader_ptr->registerCreator(
shared_ptr<LogicCreator<AllOfSpec, RequestContext> >(
new LogicCreator<AllOfSpec, RequestContext>("ALL")));
// From this point there shouldn't be any exception thrown
loader = loader_ptr.release();
......
......@@ -200,6 +200,86 @@ private:
const std::string name_;
};
/**
* \brief The NOT operator for ACLs.
*
* This simply returns the negation of whatever returns the subexpression.
*/
template<typename Context>
class NotOperator : public CompoundCheck<Context> {
public:
/**
* \brief Constructor
*
* \param expr The subexpression to be negated by this NOT.
*/
NotOperator(const boost::shared_ptr<Check<Context> >& expr) :
expr_(expr)
{ }
/**
* \brief The list of subexpressions
*
* \return The vector will contain single value and it is the expression
* passed by constructor.
*/
virtual typename CompoundCheck<Context>::Checks getSubexpressions() const {
typename CompoundCheck<Context>::Checks result;
result.push_back(expr_.get());
return (result);
}
/// \brief The matching function
virtual bool matches(const Context& context) const {
return (!expr_->matches(context));
}
private:
/// \brief The subexpression
const boost::shared_ptr<Check<Context> > expr_;
};
template<typename Context, typename Action = BasicAction>
class NotCreator : public Loader<Context, Action>::CheckCreator {
public:
/**
* \brief Constructor
*
* \param name The name of the NOT operator to be loaded as.
*/
NotCreator(const std::string& name) :
name_(name)
{ }
/**
* \brief List of the names this loads
*
* \return Single-value vector containing the name passed to the
* constructor.
*/
virtual std::vector<std::string> names() const {
std::vector<std::string> result;
result.push_back(name_);
return (result);
}
/// \brief Create the check.
virtual boost::shared_ptr<Check<Context> > create(const std::string&,
data::ConstElementPtr
definition,
const Loader<Context,
Action>& loader)
{
return (boost::shared_ptr<Check<Context> >(new NotOperator<Context>(
loader.loadCheck(definition))));
}
/**
* \brief Or-abbreviated form.
*
* This returns false. In theory, the NOT operator could be used with
* the abbreviated form, but it would be confusing. Such syntax is
* therefore explicitly forbidden.
*/
virtual bool allowListAbbreviation() const { return (false); }
public:
const std::string name_;
};
}
}
......
......@@ -184,4 +184,22 @@ TEST_F(RequestCheckTest, checkIPv6) {
EXPECT_FALSE(createIPCheck("32.1.13.184")->matches(getRequest6()));
}
// The following tests test only the creators are registered, they are tested
// elsewhere
TEST(DNSACL, notLoad) {
EXPECT_NO_THROW(getRequestLoader().loadCheck(isc::data::Element::fromJSON(
"{\"NOT\": {\"from\": \"192.0.2.1\"}}")));
}
TEST(DNSACL, allLoad) {
EXPECT_NO_THROW(getRequestLoader().loadCheck(isc::data::Element::fromJSON(
"{\"ALL\": [{\"from\": \"192.0.2.1\"}]}")));
}
TEST(DNSACL, anyLoad) {
EXPECT_NO_THROW(getRequestLoader().loadCheck(isc::data::Element::fromJSON(
"{\"ANY\": [{\"from\": \"192.0.2.1\"}]}")));
}
}
......@@ -93,6 +93,7 @@ public:
LogicCreator<AllOfSpec, Log>("ALL")));
loader_.registerCreator(CreatorPtr(new ThrowCreator));
loader_.registerCreator(CreatorPtr(new LogCreator));
loader_.registerCreator(CreatorPtr(new NotCreator<Log>("NOT")));
}
// To mark which parts of the check did run
Log log_;
......@@ -242,4 +243,49 @@ TEST_F(LogicCreatorTest, nested) {
log_.checkFirst(2);
}
void notTest(bool value) {
NotOperator<Log> notOp(shared_ptr<Check<Log> >(new ConstCheck(value, 0)));
Log log;
// It returns negated value
EXPECT_EQ(!value, notOp.matches(log));
// And runs the only one thing there
log.checkFirst(1);
// Check the getSubexpressions does sane things
ASSERT_EQ(1, notOp.getSubexpressions().size());
EXPECT_EQ(value, notOp.getSubexpressions()[0]->matches(log));
}
TEST(Not, trueValue) {
notTest(true);
}
TEST(Not, falseValue) {
notTest(false);
}
TEST_F(LogicCreatorTest, notInvalid) {
EXPECT_THROW(loader_.loadCheck(Element::fromJSON("{\"NOT\": null}")),
LoaderError);
EXPECT_THROW(loader_.loadCheck(Element::fromJSON("{\"NOT\": \"hello\"}")),
LoaderError);
EXPECT_THROW(loader_.loadCheck(Element::fromJSON("{\"NOT\": true}")),
LoaderError);
EXPECT_THROW(loader_.loadCheck(Element::fromJSON("{\"NOT\": 42}")),
LoaderError);
EXPECT_THROW(loader_.loadCheck(Element::fromJSON("{\"NOT\": []}")),
LoaderError);
EXPECT_THROW(loader_.loadCheck(Element::fromJSON("{\"NOT\": [{"
"\"logcheck\": [0, true]"
"}]}")),
LoaderError);
}
TEST_F(LogicCreatorTest, notValid) {
shared_ptr<NotOperator<Log> > notOp(load<NotOperator<Log> >("{\"NOT\":"
" {\"logcheck\":"
" [0, true]}}"));
EXPECT_FALSE(notOp->matches(log_));
log_.checkFirst(1);
}
}
......@@ -31,5 +31,14 @@ libcache_la_SOURCES += cache_entry_key.h cache_entry_key.cc
libcache_la_SOURCES += rrset_copy.h rrset_copy.cc
libcache_la_SOURCES += local_zone_data.h local_zone_data.cc
libcache_la_SOURCES += message_utility.h message_utility.cc
libcache_la_SOURCES += logger.h logger.cc
nodist_libcache_la_SOURCES = cache_messages.cc cache_messages.h
CLEANFILES = *.gcno *.gcda
BUILT_SOURCES = cache_messages.cc cache_messages.h
cache_messages.cc cache_messages.h: cache_messages.mes
$(top_builddir)/src/lib/log/compiler/message $(top_srcdir)/src/lib/cache/cache_messages.mes
CLEANFILES = *.gcno *.gcda cache_messages.cc cache_messages.h
EXTRA_DIST = cache_messages.mes
# Copyright (C) 2010 Internet Systems Consortium, Inc. ("ISC")
#
# Permission to use, copy, modify, and/or distribute this software for any
# purpose with or without fee is hereby granted, provided that the above
# copyright notice and this permission notice appear in all copies.
#
# THE SOFTWARE IS PROVIDED "AS IS" AND ISC DISCLAIMS ALL WARRANTIES WITH
# REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF MERCHANTABILITY
# AND FITNESS. IN NO EVENT SHALL ISC BE LIABLE FOR ANY SPECIAL, DIRECT,
# INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES WHATSOEVER RESULTING FROM
# LOSS OF USE, DATA OR PROFITS, WHETHER IN AN ACTION OF CONTRACT, NEGLIGENCE
# OR OTHER TORTIOUS ACTION, ARISING OUT OF OR IN CONNECTION WITH THE USE OR
# PERFORMANCE OF THIS SOFTWARE.
$NAMESPACE isc::cache
% CACHE_ENTRY_MISSING_RRSET missing RRset to generate message for %1
The cache tried to generate the complete answer message. It knows the structure
of the message, but some of the RRsets to be put there are not in cache (they
probably expired already). Therefore it pretends the message was not found.
% CACHE_LOCALZONE_FOUND found entry with key %1 in local zone data
Debug message, noting that the requested data was successfully found in the
local zone data of the cache.
% CACHE_LOCALZONE_UNKNOWN entry with key %1 not found in local zone data
Debug message. The requested data was not found in the local zone data.
% CACHE_LOCALZONE_UPDATE updating local zone element at key %1
Debug message issued when there's update to the local zone section of cache.
% CACHE_MESSAGES_DEINIT deinitialized message cache
Debug message. It is issued when the server deinitializes the message cache.
% CACHE_MESSAGES_EXPIRED found an expired message entry for %1 in the message cache
Debug message. The requested data was found in the message cache, but it
already expired. Therefore the cache removes the entry and pretends it found
nothing.
% CACHE_MESSAGES_FOUND found a message entry for %1 in the message cache
Debug message. We found the whole message in the cache, so it can be returned
to user without any other lookups.
% CACHE_MESSAGES_INIT initialized message cache for %1 messages of class %2
Debug message issued when a new message cache is issued. It lists the class
of messages it can hold and the maximum size of the cache.
% CACHE_MESSAGES_REMOVE removing old instance of %1/%2/%3 first
Debug message. This may follow CACHE_MESSAGES_UPDATE and indicates that, while
updating, the old instance is being removed prior of inserting a new one.
% CACHE_MESSAGES_UNCACHEABLE not inserting uncacheable message %1/%2/%3
Debug message, noting that the given message can not be cached. This is because
there's no SOA record in the message. See RFC 2308 section 5 for more
information.
% CACHE_MESSAGES_UNKNOWN no entry for %1 found in the message cache
Debug message. The message cache didn't find any entry for the given key.
% CACHE_MESSAGES_UPDATE updating message entry %1/%2/%3
Debug message issued when the message cache is being updated with a new
message. Either the old instance is removed or, if none is found, new one
is created.
% CACHE_RESOLVER_DEEPEST looking up deepest NS for %1/%2
Debug message. The resolver cache is looking up the deepest known nameserver,
so the resolution doesn't have to start from the root.
% CACHE_RESOLVER_INIT_INFO initializing resolver cache for class %1
Debug message, the resolver cache is being created for this given class. The
difference from CACHE_RESOLVER_INIT is only in different format of passed
information, otherwise it does the same.
% CACHE_RESOLVER_INIT initializing resolver cache for class %1
Debug message. The resolver cache is being created for this given class.
% CACHE_RESOLVER_LOCAL_MSG message for %1/%2 found in local zone data
Debug message. The resolver cache found a complete message for the user query
in the zone data.
% CACHE_RESOLVER_LOCAL_RRSET RRset for %1/%2 found in local zone data
Debug message. The resolver cache found a requested RRset in the local zone
data.
% CACHE_RESOLVER_LOOKUP_MSG looking up message in resolver cache for %1/%2
Debug message. The resolver cache is trying to find a message to answer the
user query.
% CACHE_RESOLVER_LOOKUP_RRSET looking up RRset in resolver cache for %1/%2
Debug message. The resolver cache is trying to find an RRset (which usually
originates as internally from resolver).
% CACHE_RESOLVER_NO_QUESTION answer message for %1/%2 has empty question section
The cache tried to fill in found data into the response message. But it
discovered the message contains no question section, which is invalid.
This is likely a programmer error, please submit a bug report.
% CACHE_RESOLVER_UNKNOWN_CLASS_MSG no cache for class %1
Debug message. While trying to lookup a message in the resolver cache, it was
discovered there's no cache for this class at all. Therefore no message is
found.
% CACHE_RESOLVER_UNKNOWN_CLASS_RRSET no cache for class %1
Debug message. While trying to lookup an RRset in the resolver cache, it was
discovered there's no cache for this class at all. Therefore no data is found.
% CACHE_RESOLVER_UPDATE_MSG updating message for %1/%2/%3
Debug message. The resolver is updating a message in the cache.
% CACHE_RESOLVER_UPDATE_RRSET updating RRset for %1/%2/%3
Debug message. The resolver is updating an RRset in the cache.
% CACHE_RESOLVER_UPDATE_UNKNOWN_CLASS_MSG no cache for class %1
Debug message. While trying to insert a message into the cache, it was
discovered that there's no cache for the class of message. Therefore
the message will not be cached.
% CACHE_RESOLVER_UPDATE_UNKNOWN_CLASS_RRSET no cache for class %1
Debug message. While trying to insert an RRset into the cache, it was
discovered that there's no cache for the class of the RRset. Therefore
the message will not be cached.
% CACHE_RRSET_EXPIRED found expired RRset %1/%2/%3
Debug message. The requested data was found in the RRset cache. However, it is
expired, so the cache removed it and is going to pretend nothing was found.
% CACHE_RRSET_INIT initializing RRset cache for %2 RRsets of class %1
Debug message. The RRset cache to hold at most this many RRsets for the given
class is being created.
% CACHE_RRSET_LOOKUP looking up %1/%2/%3 in RRset cache
Debug message. The resolver is trying to look up data in the RRset cache.
% CACHE_RRSET_NOT_FOUND no RRset found for %1/%2/%3
Debug message which can follow CACHE_RRSET_LOOKUP. This means the data is not
in the cache.
% CACHE_RRSET_REMOVE_OLD removing old RRset for %1/%2/%3 to make space for new one
Debug message which can follow CACHE_RRSET_UPDATE. During the update, the cache
removed an old instance of the RRset to replace it with the new one.
% CACHE_RRSET_UNTRUSTED not replacing old RRset for %1/%2/%3, it has higher trust level
Debug message which can follow CACHE_RRSET_UPDATE. The cache already holds the
same RRset, but from more trusted source, so the old one is kept and new one
ignored.
% CACHE_RRSET_UPDATE updating RRset %1/%2/%3 in the cache
Debug message. The RRset is updating its data with this given RRset.
......@@ -16,6 +16,7 @@
#include "local_zone_data.h"
#include "cache_entry_key.h"
#include "rrset_copy.h"
#include "logger.h"
using namespace std;
using namespace isc::dns;
......@@ -33,8 +34,10 @@ LocalZoneData::lookup(const isc::dns::Name& name,
string key = genCacheEntryName(name, type);
RRsetMapIterator iter = rrsets_map_.find(key);
if (iter == rrsets_map_.end()) {
LOG_DEBUG(logger, DBG_TRACE_DATA, CACHE_LOCALZONE_UNKNOWN).arg(key);
return (RRsetPtr());
} else {
LOG_DEBUG(logger, DBG_TRACE_DATA, CACHE_LOCALZONE_FOUND).arg(key);
return (iter->second);
}
}
......@@ -43,6 +46,7 @@ void
LocalZoneData::update(const isc::dns::RRset& rrset) {
//TODO Do we really need to recreate the rrset again?
string key = genCacheEntryName(rrset.getName(), rrset.getType());
LOG_DEBUG(logger, DBG_TRACE_DATA, CACHE_LOCALZONE_UPDATE).arg(key);
RRset* rrset_copy = new RRset(rrset.getName(), rrset.getClass(),
rrset.getType(), rrset.getTTL());
......
// Copyright (C) 2011 Internet Systems Consortium, Inc. ("ISC")
//
// Permission to use, copy, modify, and/or distribute this software for any
// purpose with or without fee is hereby granted, provided that the above
// copyright notice and this permission notice appear in all copies.
//
// THE SOFTWARE IS PROVIDED "AS IS" AND ISC DISCLAIMS ALL WARRANTIES WITH
// REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF MERCHANTABILITY
// AND FITNESS. IN NO EVENT SHALL ISC BE LIABLE FOR ANY SPECIAL, DIRECT,
// INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES WHATSOEVER RESULTING FROM
// LOSS OF USE, DATA OR PROFITS, WHETHER IN AN ACTION OF CONTRACT, NEGLIGENCE
// OR OTHER TORTIOUS ACTION, ARISING OUT OF OR IN CONNECTION WITH THE USE OR
// PERFORMANCE OF THIS SOFTWARE.
#include <cache/logger.h>
namespace isc {
namespace cache {
isc::log::Logger logger("cache");
}
}
// Copyright (C) 2011 Internet Systems Consortium, Inc. ("ISC")
//
// Permission to use, copy, modify, and/or distribute this software for any
// purpose with or without fee is hereby granted, provided that the above
// copyright notice and this permission notice appear in all copies.
//
// THE SOFTWARE IS PROVIDED "AS IS" AND ISC DISCLAIMS ALL WARRANTIES WITH
// REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF MERCHANTABILITY
// AND FITNESS. IN NO EVENT SHALL ISC BE LIABLE FOR ANY SPECIAL, DIRECT,
// INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES WHATSOEVER RESULTING FROM
// LOSS OF USE, DATA OR PROFITS, WHETHER IN AN ACTION OF CONTRACT, NEGLIGENCE
// OR OTHER TORTIOUS ACTION, ARISING OUT OF OR IN CONNECTION WITH THE USE OR
// PERFORMANCE OF THIS SOFTWARE.
#ifndef __DATASRC_LOGGER_H
#define __DATASRC_LOGGER_H
#include <log/macros.h>
#include <cache/cache_messages.h>
/// \file logger.h
/// \brief Cache library global logger
///
/// This holds the logger for the cache library. It is a private header
/// and should not be included in any publicly used header, only in local
/// cc files.
namespace isc {
namespace cache {
/// \brief The logger for this library
extern isc::log::Logger logger;
enum {
/// \brief Trace basic operations
DBG_TRACE_BASIC = 10,
/// \brief Trace data operations
DBG_TRACE_DATA = 40,
};
}
}
#endif
// Copyright (C) 2010 Internet Systems Consortium, Inc. ("ISC")
//
// Permission to use, copy, modify, and/or distribute this software for any
//
// purpose with or without fee is hereby granted, provided that the above
// copyright notice and this permission notice appear in all copies.
//
......@@ -20,6 +21,7 @@
#include "message_cache.h"
#include "message_utility.h"
#include "cache_entry_key.h"
#include "logger.h"
namespace isc {
namespace cache {
......@@ -39,11 +41,14 @@ MessageCache::MessageCache(const RRsetCachePtr& rrset_cache,
message_lru_((3 * cache_size),
new HashDeleter<MessageEntry>(message_table_))
{
LOG_DEBUG(logger, DBG_TRACE_BASIC, CACHE_MESSAGES_INIT).arg(cache_size).
arg(RRClass(message_class));
}
MessageCache::~MessageCache() {
// Destroy all the message entries in the cache.
message_lru_.clear();
LOG_DEBUG(logger, DBG_TRACE_BASIC, CACHE_MESSAGES_DEINIT);
}
bool
......@@ -57,26 +62,38 @@ MessageCache::lookup(const isc::dns::Name& qname,
if(msg_entry) {
// Check whether the message entry has expired.
if (msg_entry->getExpireTime() > time(NULL)) {
LOG_DEBUG(logger, DBG_TRACE_DATA, CACHE_MESSAGES_FOUND).
arg(entry_name);
message_lru_.touch(msg_entry);
return (msg_entry->genMessage(time(NULL), response));
} else {
// message entry expires, remove it from hash table and lru list.
LOG_DEBUG(logger, DBG_TRACE_DATA, CACHE_MESSAGES_EXPIRED).
arg(entry_name);
message_table_.remove(entry_key);
message_lru_.remove(msg_entry);
return (false);
}
}
LOG_DEBUG(logger, DBG_TRACE_DATA, CACHE_MESSAGES_UNKNOWN).arg(entry_name);
return (false);
}
bool
MessageCache::update(const Message& msg) {
if (!canMessageBeCached(msg)){
LOG_DEBUG(logger, DBG_TRACE_DATA, CACHE_MESSAGES_UNCACHEABLE).
arg((*msg.beginQuestion())->getName()).
arg((*msg.beginQuestion())->getType()).
arg((*msg.beginQuestion())->getClass());
return (false);
}
QuestionIterator iter = msg.beginQuestion();
LOG_DEBUG(logger, DBG_TRACE_DATA, CACHE_MESSAGES_UPDATE).
arg((*iter)->getName()).arg((*iter)->getType()).
arg((*iter)->getClass());
std::string entry_name = genCacheEntryName((*iter)->getName(),
(*iter)->getType());
HashKey entry_key = HashKey(entry_name, RRClass(message_class_));
......@@ -88,6 +105,9 @@ MessageCache::update(const Message& msg) {
// add the message entry, maybe there is one way to touch it once.
MessageEntryPtr old_msg_entry = message_table_.get(entry_key);
if (old_msg_entry) {
LOG_DEBUG(logger, DBG_TRACE_DATA, CACHE_MESSAGES_REMOVE).
arg((*iter)->getName()).arg((*iter)->getType()).
arg((*iter)->getClass());
message_lru_.remove(old_msg_entry);
}
......
......@@ -39,7 +39,7 @@ private:
MessageCache& operator=(const MessageCache& source);
public:
/// \param rrset_cache The cache that stores the RRsets that the
/// message entry will points to
/// message entry will point to
/// \param cache_size The size of message cache.
/// \param message_class The class of the message cache
/// \param negative_soa_cache The cache that stores the SOA record
......
......@@ -20,6 +20,7 @@
#include "message_entry.h"
#include "message_utility.h"
#include "rrset_cache.h"
#include "logger.h"
using namespace isc::dns;
using namespace std;
......@@ -64,7 +65,7 @@ static uint32_t MAX_UINT32 = numeric_limits<uint32_t>::max();
// tunable. Values of one to three hours have been found to work well
// and would make sensible a default. Values exceeding one day have
// been found to be problematic. (sec 5, RFC2308)
// The default value is 3 hourse (10800 seconds)
// The default value is 3 hours (10800 seconds)
// TODO:Give an option to let user configure
static uint32_t MAX_NEGATIVE_CACHE_TTL = 10800;
......@@ -142,6 +143,8 @@ MessageEntry::genMessage(const time_t& time_now,
// has expired, if it is, return false.
vector<RRsetEntryPtr> rrset_entry_vec;
if (false == getRRsetEntries(rrset_entry_vec, time_now)) {
LOG_DEBUG(logger, DBG_TRACE_DATA, CACHE_ENTRY_MISSING_RRSET).
arg(entry_name_);
return (false);
}
......
......@@ -17,6 +17,7 @@
#include "resolver_cache.h"
#include "dns/message.h"
#include "rrset_cache.h"
#include "logger.h"
#include <string>
#include <algorithm>
......@@ -29,6 +30,7 @@ namespace cache {
ResolverClassCache::ResolverClassCache(const RRClass& cache_class) :
cache_class_(cache_class)
{
LOG_DEBUG(logger, DBG_TRACE_BASIC, CACHE_RESOLVER_INIT).arg(cache_class);
local_zone_data_ = LocalZoneDataPtr(new LocalZoneData(cache_class_.getCode()));
rrsets_cache_ = RRsetCachePtr(new RRsetCache(RRSET_CACHE_DEFAULT_SIZE,
cache_class_.getCode()));
......@@ -45,6 +47,8 @@ ResolverClassCache::ResolverClassCache(const RRClass& cache_class) :
ResolverClassCache::ResolverClassCache(const CacheSizeInfo& cache_info) :
cache_class_(cache_info.cclass)
{
LOG_DEBUG(logger, DBG_TRACE_BASIC, CACHE_RESOLVER_INIT_INFO).
arg(cache_class_);
uint16_t klass = cache_class_.getCode();
// TODO We should find one way to load local zone data.
local_zone_data_ = LocalZoneDataPtr(new LocalZoneData(klass));
......@@ -69,8 +73,11 @@ ResolverClassCache::lookup(const isc::dns::Name& qname,
const isc::dns::RRType& qtype,
isc::dns::Message& response) const
{
LOG_DEBUG(logger, DBG_TRACE_DATA, CACHE_RESOLVER_LOOKUP_MSG).
arg(qname).arg(qtype);
// message response should has question section already.
if (response.beginQuestion() == response.endQuestion()) {
LOG_ERROR(logger, CACHE_RESOLVER_NO_QUESTION).arg(qname).arg(qtype);
isc_throw(MessageNoQuestionSection, "Message has no question section");
}
......@@ -79,6 +86,8 @@ ResolverClassCache::lookup(const isc::dns::Name& qname,
// answer section.
RRsetPtr rrset_ptr = local_zone_data_->lookup(qname, qtype);
if (rrset_ptr) {
LOG_DEBUG(logger, DBG_TRACE_DATA, CACHE_RESOLVER_LOCAL_MSG).
arg(qname).arg(qtype);
response.addRRset(Message::SECTION_ANSWER, rrset_ptr);
return (true);
}
......@@ -91,11 +100,15 @@ isc::dns::RRsetPtr
ResolverClassCache::lookup(const isc::dns::Name& qname,
const isc::dns::RRType& qtype) const
{
LOG_DEBUG(logger, DBG_TRACE_DATA, CACHE_RESOLVER_LOOKUP_RRSET).
arg(qname).arg(qtype);
// Algorithm:
// 1. Search in local zone data first,