This module kills Dynamic Page Cache

Open on Drupal.org →

Created on 14 February 2025, 2 months ago

Problem/Motivation

In #3258714: Manage attachments cache → , the user cache context was added to markerio_page_attachments(). This means every page now varies by user.

By the time the HtmlRenderer is done calling all page_attachments hooks, we now have a HtmlResponse object that has the user cache context. This means DynamicPageCacheSubscriber will refuse to cache the page, even though the page itself might have been perfectly cacheable otherwise.

Steps to reproduce

Enable this module, see that every page reports "UNCACHEABLE (poor cacheability)" in the "X-Drupal-Dynamic-Cache" header.

Proposed resolution

It seems you're adding some drupal settings in markerio_page_attachments, but can't you use a #lazy_builder somewhere during the render process? Then it will get placeholdered and the page will become cacheable again by DynamicPageCacheSubscriber.

When it gets to HtmlResponseSubscriber, the HtmlResponseAttachmentsProcessor will process the placeholders, adding your attachments to the response. After that, your JavaScript should still get added, but only after DPC has finished doing its thing.

I'm not sure if this widget is intended to be used by anonymous users, but if not then the above should be more than enough to fix the issue.

Remaining tasks

Rethink the caching strategy

User interface changes

/

API changes

/

Data model changes

/

🐛 Bug report

Status

Active

Version

1.0

Component

Code

Created by

🇧🇪Belgium kristiaanvandeneynde Antwerp, Belgium

Live updates comments and jobs are added and updated live.

Sign in to follow issues

Comments & Activities

Issue created by @kristiaanvandeneynde
Comment 2 months ago →
🇧🇪Belgium kristiaanvandeneynde Antwerp, Belgium
P.S.: I'm willing to explain more about caching if you need guidance.
Comment 2 months ago →
🇺🇸United States moshe weitzman Boston, MA
I wrote a patch for datalayer module that might be a helpful model - https://www.drupal.org/project/datalayer/issues/3345817 🐛 Pages are uncacheable because of the cache context user Fixed
Comment 2 months ago →
🇧🇪Belgium kristiaanvandeneynde Antwerp, Belgium
That's exactly the type of fix I had in mind, thanks @moshe weitzman!
Comment about 1 month ago →
🇧🇪Belgium tijsdeboeck Antwerp 🇧🇪 🇪🇺 🌎
Thanks @kristiaanvandeneynde @moshe weitzman, we're planning to work on this module in the coming weeks, and will update the caching strategy.

Regarding anonymous users, it is perfectly plausible that you want enable this for anonymous users. I can imagine that you would want to enable this during a test phase, where not all testers would have an account. So it is something we need to take into account.
Comment about 1 month ago →
🇬🇷Greece vensires

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024