Project

General

Profile

Actions

Bug #64272

closed

extbase query cache does not factor in start/stop times for pages

Added by Nils Blattner over 9 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Extbase
Target version:
-
Start date:
2015-01-14
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
6.2
PHP Version:
5.4
Tags:
Complexity:
hard
Is Regression:
No
Sprint Focus:

Description

Hi there

When extbase caches queries, it does not factor in start/stop times of pages that should be searched.

=> tx_myext_domain_model_something.pid IN (1285,1284,1283,1282,1281,1280)

If now page 1285 has a stop time in the future, it will not be removed and if there is a page 1289 with a start time in the future, it will not be added.
This can of course be mitigated with a lower time to live, however a different solution is desiarable. (Btw. the default lifetime is way too high)

=> How about the query cache collects start/stop times of the underlying pages (and possible pages) and factors them into the lifetime of the query cache entry
=> Additionally tags, that get deleted upon changes in the backend for these pages, is desirable

What do you guys think?

Can be reproduced:
=> Extbase Plugin with a page in the plugin options as data storage + recursive set
=> Subpage of the data storage which has a "start time" in the future and some entries inside
=> Call to the frontend => entries from said page are not shown (query cache entry is created now)
=> Wait until start time elapses
=> Clear frontend cache
=> Call to the frontend => entries are still not shown, even tough they should be

Cheers
Nils

Actions #1

Updated by Mathias Schreiber over 9 years ago

  • Complexity set to hard
Actions #2

Updated by Georg Ringer over 9 years ago

This is not a bug, as this has never been checked, so nothing to do with the extbase query cache.

furthermore, there is no reason to check the start/stop of sysfolders (which is the place to store records). and there is the start/stop of records if you need that. it would be easy to have a scheduler task which copies the start/stop of pages into its record.

I am for closing this one.

Actions #3

Updated by Nils Blattner over 9 years ago

Hi Georg

Just because it has never been checked does not mean it is not a bug. And it has everything to do with the extbase query cache, because the process of getting items works fine without the query cache, but breaks if it is active. Also fe_groups access on a page level is most probably not checked properly because of it.

Imagine a sys folder with a certain fe_group access. This creates a race condition: clear cache > user with this specific access loads page > query cache now holds all the pages for everybody + the pages for his specific fe_groups combination which unpriviledged user can see afterwards.

It should be possible to activate/deactivate pages based on time/access and expect them work just like they advertise to an unwary editor.

Also the default query cache lifetime should be something realistic and not several years.

Best regards
Nils

Actions #4

Updated by Benni Mack almost 7 years ago

  • Status changed from New to Needs Feedback

The query cache was removed with v8 because of conceptual issues (and its migration to Doctrine), and it's simply not necessary anymore. Can you check if this helps?

Actions #5

Updated by Riccardo De Contardi over 6 years ago

  • Status changed from Needs Feedback to Closed

90 days without feedback -> closing it.

If you think that this is the wrong decision or experience the issue again or have more information about how to reproduce it, please reopen it or open a new issue with a reference to this one. Thank you.

Actions

Also available in: Atom PDF