Hide empty Redirect URL and Comment properties.

Author: tobli

Diff for: src/ts/helpers/parse.ts

@@ -26,6 +26,10 @@ function identity<T>(source: T): T {
   return source;
 }
 
+export function parseNonEmpty(input: string): string {
+  return input.trim().length > 0 ? input : undefined;
+}
+
 export function parseDate(input: string): Date {
   const date = new Date(input);
   if (isNaN(date.getTime())) {

Diff for: src/ts/waterfall/details-overlay/extract-details-keys.ts

@@ -1,6 +1,6 @@
 import {getHeader} from "../../helpers/har";
 import {
-  formatBytes, formatDateLocalized, formatMilliseconds, parseAndFormat, parseDate,
+  formatBytes, formatDateLocalized, formatMilliseconds, parseAndFormat, parseDate, parseNonEmpty,
   parseNonNegative, parsePositive,
 } from "../../helpers/parse";
 import {Entry, Header} from "../../typing/har";
@@ -57,7 +57,7 @@ function parseRequestDetails(harEntry: Entry): KvTuple[] {
     byteSizeProperty("Bytes Out (uploaded)", harEntry._bytesOut),
     byteSizeProperty("Headers Size", request.headersSize),
     byteSizeProperty("Body Size", request.bodySize),
-    ["Comment", request.comment],
+    ["Comment", parseAndFormat(request.comment, parseNonEmpty)],
     stringHeader("User-Agent"),
     stringHeader("Host"),
     stringHeader("Connection"),
@@ -119,8 +119,8 @@ function parseResponseDetails(harEntry: Entry): KvTuple[] {
     stringHeader("Upgrade"),
     stringHeader("Vary"),
     stringHeader("Timing-Allow-Origin"),
-    ["Redirect URL", response.redirectURL],
-    ["Comment", response.comment],
+    ["Redirect URL", parseAndFormat(response.redirectURL, parseNonEmpty)],
+    ["Comment", parseAndFormat(response.comment, parseNonEmpty)],
   ];
 }