Opened 9 years ago

Closed 9 years ago

#704 closed defect (fixed)

[b10-resolver] Server failed: The RTT is 0

Reported by: jreed Owned by: stephen
Priority: high Milestone: Sprint-20110419
Component: resolver Version:
Keywords: Cc:
CVSS Scoring: Parent Tickets:
Sensitive: no Defect Severity:
Sub-Project: Feature Depending on Ticket:
Estimated Difficulty: 2.0 Add Hours to Ticket: 0
Total Hours: 0 Internal?: no

Description

[b10-resolver] Timeout for c3.nstld.com. IN AAAA
 to 192.52.178.30, resending query
[b10-resolver] Look up nameserver for com. in NSAS
[b10-resolver] Found a nameserver, sending query to 2001:503:231d::2:30
[b10-resolver] Timeout for NS6.VERIZON.NET. IN AAAA
 to 199.7.83.42, resending query
[b10-resolver] Look up nameserver for . in NSAS
[b10-resolver] Found a nameserver, sending query to 2001:500:3::42
[b10-resolver] RTT: 11
[b10-resolver] Handle response
[b10-resolver] Response is referral
[b10-resolver] Referred to zone net.
[b10-resolver] Found a nameserver, sending query to 192.31.80.30
[b10-resolver] RTT: 77
[b10-resolver] Handle response
[b10-resolver] Response is referral
[b10-resolver] Referred to zone nstld.com.
[b10-resolver] Server failed: The RTT is 0
[b10-resolver] Query shutdown
[bind10] Process b10-resolver (PID 15497) terminated, exit status = 256

Subtickets

Change History (7)

comment:1 Changed 9 years ago by jreed

This was fixed in commit 2a4f9f2f6d0831520ddba84030873b70b9577c0c

Keeping this open for now since needs unit test(s).

comment:2 Changed 9 years ago by shane

  • Milestone set to Sprint-20110405

comment:3 Changed 9 years ago by stephen

  • Owner set to stephen
  • Status changed from new to assigned

comment:4 Changed 9 years ago by stephen

  • Owner changed from stephen to jreed

Test added in commit ea064b169fe62c3f71aaec4b5c6d677255f3621a. Also added is a test to ensure that setting the RTT to an absurdly high value causes no problems.

comment:5 Changed 9 years ago by shane

  • Owner changed from jreed to shane
  • Status changed from assigned to reviewing

comment:6 Changed 9 years ago by shane

  • Owner changed from shane to stephen

Looks good. I feel there must be a more clever way to insure that RTT doesn't hit 0 than the loop with 2000 iterations, but it runs quickly and does the job. :)

Please merge!

comment:7 Changed 9 years ago by stephen

  • Estimated Difficulty changed from 0.0 to 2
  • Resolution set to fixed
  • Status changed from reviewing to closed

Merged in commit 50eff6d6d5788193d055c038a5772bce2534628f.

Note: See TracTickets for help on using tickets.