- Issue created by @pianomansam
- πΊπΈUnited States pianomansam
pianomansam β changed the visibility of the branch 3534024-remove-custom-eucookiecompliancedata to hidden.
- Merge request !167remove custom eu_cookie_compliance_data cache entry β (Open) created by pianomansam
- πΊπΈUnited States pianomansam
Also adding that β¨ GPC Implementation Active requires the changes in this task.
- π³πΏNew Zealand atowl
Hi @pianomansam
So i've spent some time going over this, and i think i've tested enough to make sure this works, just making sure since this will affect performance if we do get it wrong :)
In my ddev env turned on enough caching to get the X-Drupal-Cache to have a 'HIT' on a 2nd page refresh.
(For those playing along at home i had to use$settings['cache']['bins']['render'] = 'cache.backend.database'; $settings['cache']['bins']['page'] = 'cache.backend.database'; $settings['cache']['bins']['dynamic_page_cache'] = 'cache.backend.database';
)
I then took your patch, and placed a log message after it, cleared the caches (drush cr), and refreshed the page.
First page load after cache clearing, i saw X-Drupal-Cache: MISS, and the log message.
Every then every subsequent page load was a X-Drupal-Cache: HIT, and no log message, as it's obviously coming from the cache.Is that what you were expecting/tested?