Include query string in cache key by jpb · Pull Request #38 · npm/make-fetch-happen (original) (raw)

Include the URL's query string in the cache key. Not including the query component in the cache key can result in an ineffective cache or subtle bugs. The cache key for URLs that do not have a query component have not changed.

RFC 7234 states that "the primary cache key consists of the request method and target URI", the query component being part of this.

Fixes #7