Problem/Motivation

Haven't check this very closely yet, but it looks like a site with dynamic page cache always returns the same data, the site has dynamic page cache enabled.

Proposed resolution

Figure out why, add necessary cacheabilty metadata, for example with a max-age. Start by extending the tests to make a change and then check a resource again. Write a log entry, for example.

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#5 prevent_caching_of-2834724-2.patch3.43 KBdrobnjak
#5 prevent_caching_of-2834724-2-test-only.patch1.84 KBdrobnjak
#2 prevent_caching_of-2834724-2.patch3.43 KBdrobnjak

Comments

Berdir created an issue. See original summary.

drobnjak’s picture

drobnjak commented
Status: Active » Needs review
StatusFileSize
new prevent_caching_of-2834724-2.patch3.43 KB

Setting the max-age to 0 since max-age = 0 responses are ignored by Dynamic Page Cache. Providing test coverage.

berdir’s picture

berdir
Primary language German
Location Switzerland
commented

Can I haz a test-only patch?

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Needs review » Needs work
drobnjak’s picture

drobnjak commented
Status: Needs work » Needs review
StatusFileSize
new prevent_caching_of-2834724-2-test-only.patch1.84 KB
new prevent_caching_of-2834724-2.patch3.43 KB

Providing test-only patch

The last submitted patch, 5: prevent_caching_of-2834724-2-test-only.patch, failed testing.

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Needs review » Fixed

Thanks.

  • Berdir committed ba375df on 8.x-1.x authored by drobnjak 
    Issue #2834724 by drobnjak: Prevent caching of monitoring results REST...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.