Sometimes timings.phases.dns is not lookup duration
Motiveko opened this issue · comments
I use got http client and I use option {dnsCache: true}
.
In http-timer packages, theres's two code to calculate dns duration.
-
on socket's
lookup event
listener
Lines 93 to 98 in 62f6b38
-
deferToConnect()
's connect callback function
Lines 102 to 110 in 62f6b38
When use got with dnsCache, once hostname resolved, then until ttl, no lookup event emitted on the socket, so lookup time should be 0.
But above the code 2, dns never be 0 but tcp(connection time) will be 0.
When Server got a lot of traffic and got's socket connection delayed, value of tcp will not increased ( this always be 0 ), but value of dns increased.
I think this is causing confusion and needs to be fixed
This is also a problem when the request is made directly to an IP. There's obviously no DNS resolution in that case, but the timings report it.