fix: support object returned by the response.getHeaders() when create Response

[dead-horse]

koa use response.getHeaders() to get response headers, and the object returned by the response.getHeaders() method does not prototypically inherit from the JavaScript Object

[maxtruxa]

Good catch. But wouldn't the canonical way to fix this be to just explicitly call hasOwnProperty?

I.e. replace

if (options.headers.hasOwnProperty(field)) {

with

if (Object.prototype.hasOwnProperty(options.headers, field)) {

You wouldn't even need the fallback if headers is not set (|| {}). Edit: NVM, passing null or undefined still fails, so that change is required and good.

[dead-horse]

@maxtruxa Object.prototype.hasOwnProperty.call(headers, field) is better to handle with Object.create(null) situation. please check this PR again :)

[mjsalinger]

LGTM

[karladler]

Why is this not already merged?

[dead-horse]

I think this issue is already fixed in dev branch by 68ef835, when will it release?