Opened 8 years ago

Closed 8 years ago

#1586 closed task (complete)

Check for queries for NSEC3 owner names in in-memory data source

Reported by: jinmei Owned by: vorner
Priority: medium Milestone: Sprint-20120221
Component: data source Version:
Keywords: Cc:
CVSS Scoring: Parent Tickets:
Sensitive: no Defect Severity: N/A
Sub-Project: DNS Feature Depending on Ticket: NSEC3
Estimated Difficulty: 4 Add Hours to Ticket: 0
Total Hours: 1.15 Internal?: no

Description

It's RFC5155 7.2.8.

In the in memory data source the desired behavior should be achieved
by design - there shouldn't be need for explicit code to do that. But
it's worth testing anyway, so this task is to test the case, both for
the NXDOMAIN case and the case where a non NSECC3 RR really exists for
an NSEC3 owner name.

Query::auth shouldn't have to be updated for this part.

Depend on #1574.

Subtickets

Change History (9)

comment:1 Changed 8 years ago by jelte

  • Estimated Difficulty changed from 0 to 4

comment:2 Changed 8 years ago by jelte

  • Milestone changed from Year 3 Task Backlog to Sprint-20120207

comment:3 Changed 8 years ago by vorner

  • Owner set to vorner
  • Status changed from new to accepted

comment:4 Changed 8 years ago by vorner

  • Owner changed from vorner to UnAssigned
  • Status changed from accepted to reviewing

Hello

It should be ready for review. I added tests for when the name does not exist in the real namespace, when it does (both NXDOMAIN and success) and when there's something below the name (eg. empty nonterminal in the same name as the NSEC3 one).

Obviously, I don't think this should have a changelog entry.

comment:5 Changed 8 years ago by jinmei

  • Milestone changed from Sprint-20120221 to Sprint-20120207
  • Owner changed from UnAssigned to jinmei

comment:6 Changed 8 years ago by jinmei

  • Owner changed from jinmei to vorner

Looks okay, please merge.

comment:7 Changed 8 years ago by jinmei

  • Milestone changed from Sprint-20120207 to Sprint-20120221

comment:8 Changed 8 years ago by jinmei

  • Total Hours changed from 0 to 0.1

comment:9 Changed 8 years ago by vorner

  • Resolution set to complete
  • Status changed from reviewing to closed
  • Total Hours changed from 0.1 to 1.15

Thank you, closing

Note: See TracTickets for help on using tickets.