#204 add note to Connect time with TLS

micmro · May 18, 2017 · b1301c1 · b1301c1
1 parent f917048
commit b1301c1
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/src/ts/transformers/extract-details-keys.ts b/src/ts/transformers/extract-details-keys.ts
@@ -146,11 +146,16 @@ function parseTimings(entry: Entry, start: number, end: number): KvTuple[] {
   const optionalTiming = (timing?: number) => parseAndFormat(timing, parseNonNegative, formatMilliseconds);
   const total = (typeof start !== "number" || typeof end !== "number") ? undefined : (end - start);
 
+  let connectVal = optionalTiming(timings.connect);
+  if (timings.ssl > 0) {
+    // SSL time is also included in the connect field (to ensure backward compatibility with HAR 1.1).
+    connectVal = `${connectVal} (without TLS: ${optionalTiming(timings.connect - timings.ssl)})`;
+  }
   return [
     ["Total", formatMilliseconds(total)],
     ["Blocked", optionalTiming(timings.blocked)],
     ["DNS", optionalTiming(timings.dns)],
-    ["Connect", optionalTiming(timings.connect)],
+    ["Connect", connectVal],
     ["SSL (TLS)", optionalTiming(timings.ssl)],
     ["Send", formatMilliseconds(timings.send)],
     ["Wait", formatMilliseconds(timings.wait)],