search

evermeer / EVURLCache

a NSURLCache subclass for handling all web requests that use NSURLRequest
Other
297 stars 49 forks source link

Query params not taken into account when caching. #23

Closed gregkerzhner closed 8 years ago

gregkerzhner commented 8 years ago

The current caching strategy does not take into account query params. Therefore, a request like

https://snacks.com?search=stuff

and 

https://snacks.com?search=otherstuff

would be cached under the same file. To fix this

request.URL?.query

should be included in the path of the cached file if it exists.

PR: https://github.com/evermeer/EVURLCache/pull/22

evermeer commented 8 years ago

This is now pushed as version 2.7.0