"X-Kevinrob-Cache: MISS" although response is from cache

[rincler]

Hi!

I have two layers of cache: memory and redis:

...
$stack->push(
  new CacheMiddleware(new GreedyCacheStrategy(new VolatileRuntimeStorage(), 0)),
  'identity-map'
);

$stack->push(
  new CacheMiddleware(new GreedyCacheStrategy(new Psr16CacheStorage($redisCache), 60)),
  'cache'
);
...

Suggest we have cached response in redis. We make $client->request().
I expect: X-Kevinrob-Cache: HIT
I see: X-Kevinrob-Cache: MISS

redis middleware set X-Kevinrob-Cache to HIT in

guzzle-cache-middleware/src/CacheMiddleware.php

Line 154 in 57c5793

$cacheEntry->getResponse()->withHeader(self::HEADER_CACHE_INFO, self::HEADER_CACHE_HIT)

, but then memory middleware set X-Kevinrob-Cache to MISS in

guzzle-cache-middleware/src/CacheMiddleware.php

Line 223 in 57c5793

$response = $response->withHeader(self::HEADER_CACHE_INFO, self::HEADER_CACHE_MISS);

Maybe need to check X-Kevinrob-Cache header before set it to MISS ?

[Kevinrob]

You can use ChainCache for using multiple cache storage.
But we can change for checking the presence of the header before setting it! Would you like to make a PR about this?