Currently the URL queuer is queueing everything dumbly including admin paths. In general paths accessed by authenticated users bypass external cache so do not need to be queued by url queuer. The decision method in the module should check for maxage < 0 and prevent them from being added to the url registry.
This change should dramatically reduce the number of records that in the purge_queuer_url table, and therefore reduce the number of external purges!
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | purge_queuer_url-prevent-queueing_maxage_less_than_0-2847828-1.patch | 453 bytes | adam.weingarten |
Comments
Comment #2
adam.weingarten commentedComment #3
adam.weingarten commentedComment #4
pedrop commentedWorks for me, thanks!
Comment #5
vaibhavjainThis looks to be working good.
Steps to test:
# Login as a user, and visit a URL, say node/1.
# There is no entry to purge_queuer_url table.
# Visit the same page as Anonymous user, node/1
# There is an entry to purge_queuer_url table.
Comment #6
nielsvm commentedAlso did some serious testing on this one, especially in combination with the other patches that just went in (e.g. auto-cleaning of deleted URLs)... Works really well, nice!
Thanks!
Comment #9
les limThis caused a major regression, in which it's trivially easy to cause a URL to be removed from the purge_queuer_url traffic registry and prevent Drupal from ever invalidating the URL. See #2912139: Logged-in requests remove items from traffic registry #13 for an explanation and patch.