Project

General

Profile

Actions

Bug #78002

closed

Require cHash for cached plugin actions in Extbase

Added by Stefan Froemken almost 8 years ago. Updated almost 6 years ago.

Status:
Closed
Priority:
Must have
Assignee:
-
Category:
Caching
Target version:
Start date:
2016-09-20
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
8
PHP Version:
7.0
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

As you have seen the discussion with Dmitry Dulepov at twitter reqCHash is only valid for cached plugins: pi_base and also extbase.

Kasper has documented it here:
https://github.com/TYPO3/TYPO3.CMS/blob/master/typo3/sysext/frontend/Classes/Plugin/AbstractPlugin.php#L250-L261

In TYPO3 4.5 there was no reqCHash in makeCacheHash. In TYPO3 4.7 it was introduced:
https://review.typo3.org/#/c/4704/

I wonder a bit, because Dmitry wrote "Looks ok to me", so it would be interessting to here why it's not OK today.

Stefan


Related issues 4 (0 open4 closed)

Related to TYPO3 Core - Bug #81297: Extbase record preview leads to 404 due to missing cHashClosed2017-05-22

Actions
Related to TYPO3 Core - Bug #81293: cacheHash fails to be generated because of missing id although id is present in request-stringClosedDavid Bascom2017-05-22

Actions
Related to TYPO3 Core - Bug #95297: cHash not enforced for Extbase actions anymoreClosedOliver Hader2021-09-21

Actions
Precedes TYPO3 Core - Task #85942: Fix a typo and add an example for requireCHashArgumentForActionArgumentsClosed2016-09-212016-09-21

Actions
Actions #1

Updated by Gerrit Code Review almost 8 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/49976

Actions #2

Updated by Dmitry Dulepov almost 8 years ago

In TYPO3 4.5 there was no reqCHash in makeCacheHash. In TYPO3 4.7 it was introduced:

It was added because of doParametersRequireCacheHash. This is a new thing, it allows to set some parameters to always require cHash. This is for special applications, not for regular plugins. Do not remove that piece of code or you will break it.

See my comments in the pull request. It needs some changes.

Actions #3

Updated by Gerrit Code Review almost 8 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/49976

Actions #4

Updated by Gerrit Code Review almost 8 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/49976

Actions #5

Updated by Gerrit Code Review almost 8 years ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/49976

Actions #6

Updated by Gerrit Code Review over 7 years ago

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/49976

Actions #7

Updated by Gerrit Code Review over 7 years ago

Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/49976

Actions #8

Updated by Gerrit Code Review over 7 years ago

Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/49976

Actions #9

Updated by Gerrit Code Review over 7 years ago

Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/49976

Actions #10

Updated by Gerrit Code Review over 7 years ago

Patch set 9 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/49976

Actions #11

Updated by Dmitry Dulepov over 7 years ago

  • Subject changed from Wrong position of reqCHash() to Require cHash for cached plugin actions in Extbase
  • Priority changed from Should have to Must have
Actions #12

Updated by Gerrit Code Review over 7 years ago

Patch set 10 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/49976

Actions #13

Updated by Gerrit Code Review over 7 years ago

Patch set 11 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/49976

Actions #14

Updated by Gerrit Code Review over 7 years ago

Patch set 12 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/49976

Actions #15

Updated by Gerrit Code Review over 7 years ago

Patch set 13 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/49976

Actions #16

Updated by Gerrit Code Review over 7 years ago

Patch set 14 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/49976

Actions #17

Updated by Gerrit Code Review over 7 years ago

Patch set 15 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/49976

Actions #18

Updated by Gerrit Code Review over 7 years ago

Patch set 16 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/49976

Actions #19

Updated by Benni Mack over 7 years ago

  • Target version changed from 8.4 to 8.5
Actions #20

Updated by Gerrit Code Review over 7 years ago

Patch set 17 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/49976

Actions #21

Updated by Gerrit Code Review over 7 years ago

Patch set 18 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/49976

Actions #22

Updated by Gerrit Code Review over 7 years ago

Patch set 19 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/49976

Actions #23

Updated by Anonymous over 7 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions #24

Updated by Dmitry Dulepov over 7 years ago

The change will add lots of identical page cache entries for every non-cached action. Very inefficient for the cache and database space in general.

Also I wonder how this works for method="get" and forms. I think it does not.

Anyway, it is already done, so nothing to do now...

Actions #25

Updated by Mathias Brodala about 7 years ago

  • Related to Bug #81297: Extbase record preview leads to 404 due to missing cHash added
Actions #26

Updated by Markus Klein about 7 years ago

  • Related to Bug #81293: cacheHash fails to be generated because of missing id although id is present in request-string added
Actions #27

Updated by Lukas Weiss over 6 years ago

I don't know where to report this, but there's an error in the documentation:
https://docs.typo3.org/typo3cms/extensions/core/8.7/Changelog/8.5/Breaking-78002-EnforceCHashArgumentForExtbaseActions.html

feature.requireCHashArgumentForActionArguments

should be

features.requireCHashArgumentForActionArguments
Actions #28

Updated by Riccardo De Contardi over 6 years ago

  • Status changed from Resolved to Closed
Actions #29

Updated by Stephan Großberndt almost 6 years ago

The documentation error was fixed in https://review.typo3.org/#/c/58002/

Actions #30

Updated by Stephan Großberndt almost 6 years ago

  • Precedes Task #85942: Fix a typo and add an example for requireCHashArgumentForActionArguments added
Actions #31

Updated by Anonymous almost 3 years ago

  • Related to Bug #95297: cHash not enforced for Extbase actions anymore added
Actions

Also available in: Atom PDF