BIND 10 #2796: Add a counter for queries with RD=1

[BIND 10 Development]

#2796: Add a counter for queries with RD=1
-------------------------------------+-------------------------------------
            Reporter:  y-aharen      |                        Owner:
                Type:  enhancement   |  jinmei
            Priority:  medium        |                       Status:
           Component:  b10-auth      |  reviewing
            Keywords:                |                    Milestone:
           Sensitive:  0             |  Sprint-20130625
         Sub-Project:  DNS           |                   Resolution:
Estimated Difficulty:  4             |                 CVSS Scoring:
         Total Hours:  0             |              Defect Severity:  N/A
                                     |  Feature Depending on Ticket:
                                     |          Add Hours to Ticket:  0
                                     |                    Internal?:  0
-------------------------------------+-------------------------------------
Changes (by y-aharen):

 * owner:  y-aharen => jinmei


Comment:

 Hello,

 Replying to [comment:15 jinmei]:
 > Replying to [comment:14 y-aharen]:
 > > Hello,
 > >
 > > Replying to [comment:12 jinmei]:
 > > > What I wanted to see is some explanation (maybe in the man page)
 > > > reading like: "note that the RD bit is not specific to queries
 > > > (requests of opcode 0), but this counter is limited to queries.
 > > > This is because <the reason, which I didn't understand yet>".
 > > >
 > > > Again, I'm not objecting, just asking why you thought it should be
 > > > limited, and whatever the reason it wasn't obvious to me (and
 probably
 > > > to others too) so it's better to be documented.
 > > OK, I added a note to describe the reason (git d6c6fe4).
 >
 > Hmm, this explanation doesn't make much sense to me...
 > {{{
 >       <para>
 >         RD bit is not specific to queries (requests of opcode 0), but
 >       <quote>qryrecursion</quote> is limited to queries because RD bit
 is
 >       meaningless for the other opcodes and they are passed to another
 >       BIND 10 module such as <command>b10-ddns</command>.
 >       </para>
 > }}}
 >
 > First, what's the source (RFC, etc) of the claim "RD bit is
 > meaningless for the other opcodes"?  It's also somewhat inconsistent
 > with the first sentence "RD bit is not specific to queries".
 > Secondly, if we can say we don't count things related to non query
 > requests because they are forwarded to different processes, then why
 > do we have per opcode counters in Auth or generic requests/* counters?
 Well, there is no specification that claims it. However, RFC 1996 (Notify)
 does not mention recursing and some implementations such as NSD, PowerDNS
 and BIND 10 don't care RD bit while processing Notify requests. Moreover,
 RFC 2136 (Update) states that Update message uses only QR flag bit and the
 other flags are explicitly zero or ignored if non-zero.
 I meant "RD bit is not specific to queries" as "RD bit can be set in the
 request regardless of opcode." If it is misleading I'll revise it.

 The latter is not a good reason; I'll remove it after the former concludes
 in this ticket.

 If you don't agree with the explanation and if it should also be counted
 requests with RD=1 regardless of opcode, I'd like to add another item
 'request.rd' for requests with RD=1 and keep 'qryrecursion' as is.

 Thanks,

-- 
Ticket URL: <http://bind10.isc.org/ticket/2796#comment:17>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development