search

kaazing / http2-cache.js

2 stars 11 forks source link

Add require for collections/map in cache.js #77

Closed tejaede closed 6 years ago

tejaede commented 6 years ago

This should be pretty safe so the PR is only to ensure that you are aware of the issue.

Cache uses 'instanceof Map' while caching request headers, but did not require collections/map in the module. This led to instanceof Map being a false negative in IE11 because it was checking the native Map against the collections Map.

tejaede commented 6 years ago

@dpwspoon Added you to make you aware of the issue. I will go ahead and merge.