[HttpCache][WIP] Use RFC9211 Cache-Status #60078
Conversation
| @@ -29,6 +29,8 @@ class HeaderBag implements \IteratorAggregate, \Countable, \Stringable | |||
| protected array $headers = []; | |||
| protected array $cacheControl = []; | |||
|
|
|||
There was a problem hiding this comment.
Blank line can be removed
| @@ -127,6 +127,7 @@ public function testInvalidatesOnPostPutDeleteRequests() | |||
|
|
|||
| $this->assertHttpKernelIsCalled(); | |||
| $this->assertResponseOk(); | |||
|
|
|||
|
@franckranaivo do you intend to continue this ? Btw, it would be great to update the description of the PR to include some usage examples. It makes the review much easier. |
| @@ -29,6 +29,8 @@ class HeaderBag implements \IteratorAggregate, \Countable, \Stringable | |||
| protected array $headers = []; | |||
| protected array $cacheControl = []; | |||
|
|
|||
| protected array $cacheStatus = []; | |||
There was a problem hiding this comment.
yes, I intended to use this array later in order to separate cache status headers (in the rfc) from cache control headers. It is still a wip, but yes soon it will be used.
@stof yes, sorry I was a bit busy these days. I will continue this asap :)
Sure I will, I will add some more description within the next days |
This PR is intended to answer #58985 by using Cache-Status directives as described here.