[bind10-dev] libdns and problem record and loadzone loading it

[Evan Hunt]

> BTW, I personally think we're giving too much responsibility to the
> data source module by having it construct response messages.  IMO
> the data source modules should focus on the lookup service and should
> be free from other DNS related details.  (Likewise, IMO we should pass
> the specific lookup key (that would probably be a tuple of
> qname/qclass/qtype) to the data source module rather than passing the
> entire query message and having the module to extract the lookup key
> from the Question section).
> 
> Whether or not it's a compelling argument, however, I understand it's
> certainly a post-year1 fodder.

I have been thinking along similar lines.  I considered moving doQuery()
and most of the rest of data_source.cc into b10-auth rather than libauth,
as I can't think of a reason why any application other than b10-auth would
ever be calling it.

I hesitate, however, because the data source is soon going to undergo some
changes; we'll be making it writable to support UPDATE, we'll add caching,
we might even add some kind of awareness of recursion.

I can imagine that multiple applications might want to update the data
sources via a simple high-level API comparable to DataSrc::doQuery().  If
so, then that belongs in a library.  And if so, then maybe doQuery() ought
to live in the same library, for the sake of simplicity.

So rather than move doQuery() now, I'm waiting to see what the data source
looks like in Y2.  But I do think you have a good point.

                                        eh