From 967c236fa5fb1e87e1b5ee788fe77d3a07361da1 Mon Sep 17 00:00:00 2001 From: Rory Malcolm Date: Mon, 28 Oct 2019 16:23:39 +0000 Subject: Add CustomInspect for Headers (#3130) Worth noting due to implementation of the Headers class the contents of headersMap have lowercase keys, although this matches the specification as header keys are case agnostic it does seem to not match behaviour of other implementations in other languages I have seen, would require some rewriting of Headers.ts --- cli/js/headers.ts | 13 +++++++++++++ 1 file changed, 13 insertions(+) (limited to 'cli/js/headers.ts') diff --git a/cli/js/headers.ts b/cli/js/headers.ts index dc0de54dd..cc22077ad 100644 --- a/cli/js/headers.ts +++ b/cli/js/headers.ts @@ -2,6 +2,7 @@ import * as domTypes from "./dom_types.ts"; import { DomIterableMixin } from "./mixins/dom_iterable.ts"; import { requiredArguments } from "./util.ts"; +import { customInspect } from "./console.ts"; // From node-fetch // Copyright (c) 2016 David Frank. MIT License. @@ -85,6 +86,18 @@ class HeadersBase { } } + [customInspect](): string { + let headerSize = this[headerMap].size; + let output = ""; + this[headerMap].forEach((value, key) => { + const prefix = headerSize === this[headerMap].size ? " " : ""; + const postfix = headerSize === 1 ? " " : ", "; + output = output + `${prefix}${key}: ${value}${postfix}`; + headerSize--; + }); + return `Headers {${output}}`; + } + // ref: https://fetch.spec.whatwg.org/#concept-headers-append append(name: string, value: string): void { requiredArguments("Headers.append", arguments.length, 2); -- cgit v1.2.3