Make greedy cache TTL dynamic about guzzle-cache-middleware

[Kevinrob]

Yes, this can be a good feature!

[bpolaszek]

Hey there,

I know I'm 1 year late on that topic but I did not notice it before.

I'm really against the fact that the library modifies request objects (even if they're immutable). That's what the DelegatingCacheStrategy was designed for (changing strategy based on the request). It's a little more complex to use to achieve the same goal, but I'd have worked on improving this before having no other choice to touch on request objects.

Just saying 😺

[Kevinrob]

@bpolaszek But we don't modifies request objects, do we?

[bpolaszek]

Technically you don't, because PSR-7 objects are immutable, but my point is that you delegate to the Request object the responsibility to carry an information which is out of its scope. Nothing really important here, just quibbling with OOP 😄

[bpolaszek]

Another solution would have been to fake a real Cache-Control header, which would have behaved the same.

[Kevinrob]

Okay, I understand your point! It's true.
Both methods are possible with this. 😄