fix: Use only the "data" event to catch the firstByte timing

It is triggered before the "readable" event. Less code and more correct measurement.
prantlf · Nov 4, 2017 · f5bb930 · f5bb930
1 parent 13ab821
commit f5bb930
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/lib/nettime.js b/lib/nettime.js
@@ -12,11 +12,14 @@ function nettime (options) {
   return new Promise((resolve, reject) => {
     start = getTime(process.hrtime())
     protocol.get(parameters, localResponse => {
+      let firstByte
+
       response = localResponse
-      response.once('readable', () => {
-        timings.firstByte = getDuration()
-      })
-      .on('data', () => {
+      response.on('data', () => {
+        if (!firstByte) {
+          timings.firstByte = getDuration()
+          firstByte = true
+        }
       })
       .on('end', () => {
         timings.contentTransfer = getDuration()