Skip to content
This repository has been archived by the owner on Mar 8, 2024. It is now read-only.

Cache Key should contain more than just the HTTP Method #28

Open
martinei opened this issue Nov 24, 2021 · 1 comment
Open

Cache Key should contain more than just the HTTP Method #28

martinei opened this issue Nov 24, 2021 · 1 comment

Comments

@martinei
Copy link

The library uses a Cache Key composed out of the Request Method and http.Request.RequestURI.
However "RequestURI" is a server only field and is empty on client side.
Thus all GET Request share the same cache key. ("GET "). As a result this library is basically is a single page cache.

See

key = fmt.Sprintf("%s %s", req.Method, req.RequestURI)

@pablodz
Copy link

pablodz commented Feb 1, 2022

Getting the same error when fetching GET also from different URLs

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants