GHSA-4vr3-9v7h-5f8v
- Source
- https://github.com/advisories/GHSA-4vr3-9v7h-5f8v
- Import Source
- https://github.com/github/advisory-database/blob/main/advisories/github-reviewed/2019/06/GHSA-4vr3-9v7h-5f8v/GHSA-4vr3-9v7h-5f8v.json
- Published
- 2019-06-18T15:38:41Z
- Modified
- 2023-04-11T01:49:51.858739Z
- Details
-
Leaking cached authenticated requests
Impact
If you've been using one
MemoryCacheMethodobject in multiple instances ofGw2WebApiClientand are requesting authenticated endpoints with different access tokens, then you are likely to run into this bug.When using an instance of
MemoryCacheMethodand using it with multiple instances ofGw2WebApiClient, there's a possibility that cached authenticated responses are leaking to another request to the same endpoint, but with a different Guild Wars 2 access token. The latter request wouldn't start however, and would return the first cached response immediately. This means that the second response (or later responses) may contain the same data as the first response, therefore leaking data from another authenticated endpoint.The occurence of this is limited however. The Guild Wars 2 API doesn't use the
Expiresheader on most (if not all) authenticated endpoints. This header is checked when caching responses. If this header isn't available, the response isn't cached at all. You should still update to at least version 0.3.1 in order to be certain that it won't happen.Patches
This bug has been fixed in version 0.3.1. When using an authenticated endpoint, it will prepend the SHA-1 hash of the access token to the cache id.
Workarounds
For version 0.3.0 and lower, you can use one separate instance of
MemoryCacheMethodperGw2WebApiClientif you need to use it.For more information
If you have any questions or comments about this advisory, you can open an issue in the Gw2Sharp repository or contact me on Discord.
- References
Affected packages
NuGet / Gw2Sharp
Package
- Name
- Gw2Sharp