Closed Bug 1460330 Opened 6 years ago Closed 6 years ago

TRR: make use of the TTL data independent of the TTL pref for TRR resolved addresses

Categories

(Core :: Networking: DNS, enhancement, P1)

Product:
Core
Component:
Networking: DNS
Type:
enhancement

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla62
Tracking Flags:
Tracking Status
firefox62 --- fixed

People

(Reporter: bagder, Assigned: bagder)

Details

(Whiteboard: [necko-triaged][trr])

Attachments

(1 file)

0001-bug-1460330-always-use-use-TTL-with-TRR-r-mcmanus.patch
1006 bytes, patch
mcmanus
: review+
Details | Diff | Splinter Review 
When storing a DNS cache entry, the use of the TTL number is conditional on a pref variable.

Since TRR resolved names always have a TTL number that involves no extra overhead and makes the cache data more reliable and accurate, the use of the TTL should be unconditional when TRR was used to resolve the host name and only do it conditional on the pref if resolved natively.
Attachment #8974693 - Flags: review?(mcmanus) → review+
Keywords: checkin-needed
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/ce7a39303eff
Always use use TTL with TRR. r=mcmanus
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/ce7a39303eff
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox62: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: