add support for prefix invalidation#600
add support for prefix invalidation#600deniak wants to merge 10 commits intoFriendsOfSymfony:3.xfrom
Conversation
dbu
left a comment
There was a problem hiding this comment.
thanks for this pull request. prefix is basicaly a special case of ban that does not allow wildcards anywhere else than at the end, right?
Multiplexer should multiplex prefix and noop implement prefix without doing anything.
we could make all clients that support ban support prefix, by converting the prefix into a banPath statement. do you think you can add that to the varnish client?
|
please rebase on the 3.x branch to get the cleanup of phpstan that i did. after that, if phpstan is not green it would be because of changes in this branch. |
Correct!
I believe my last commits should address your suggestions. Thanks for the review. |
dbu
left a comment
There was a problem hiding this comment.
thanks a lot! one last thing: can you please add tests for the invalidatePrefix that you added to more places, to keep the test coverage high?
Sorry I missed that. I added a few more tests. |
|
thanks a lot! i added a changelog entry and merged on the cli. i released this as 3.2.0 now. |
|
Thanks @dbu |
Cloudflare supports the invalidation by prefix.
That PR adds support for that method.