Opened 9 years ago

Closed 9 years ago

#450 closed defect (fixed)

a few cleanup/correction to the new query logic with memory data source

Reported by: jinmei Owned by: jinmei
Priority: medium Milestone: y2 12 month milestone
Component: b10-auth Version:
Keywords: Cc:
CVSS Scoring: Parent Tickets:
Sensitive: no Defect Severity:
Sub-Project: Feature Depending on Ticket:
Estimated Difficulty: 0.0 Add Hours to Ticket: 0
Total Hours: 0 Internal?: no

Description

  • we should return REFUSED (as opposed to SERVFAIL) when we cannot find a matchign zone as we discussed at the bind10-dev list
  • we shouldn't use the meory data source unless the query class is IN (which is the only supported class at the moment)
  • the "NXRRSET" result of Zone::find() should actually result in "NOERROR" rcode in the response (NXRRSET is dynamic update specific)

I'm going to submit a proposed fix.

Subtickets

Change History (3)

comment:1 Changed 9 years ago by jinmei

  • Owner changed from jinmei to zzchen_pku
  • Status changed from new to reviewing

Committed a proposed fix to branches/trac450, changeset r4005.

I believe Jerry is the best person to review this, so I'm going to give this ticket to him. Please reassign it to someone else if you cannot handle it.

comment:2 follow-up: Changed 9 years ago by zzchen_pku

  • Owner changed from zzchen_pku to jinmei

Looks okay, please merge.

comment:3 in reply to: ↑ 2 Changed 9 years ago by jinmei

  • Resolution set to fixed
  • Status changed from reviewing to closed

Replying to zzchen_pku:

Looks okay, please merge.

Thanks, merged to trunk. Closing ticket.

Note: See TracTickets for help on using tickets.