[BUGFIX] Remove `fbclid` argument from chash calculation 76/58676/2
authorAndreas Fernandez <a.fernandez@scripting-base.de>
Tue, 23 Oct 2018 17:56:10 +0000 (19:56 +0200)
committerAnja Leichsenring <aleichsenring@ab-softlab.de>
Tue, 23 Oct 2018 18:07:33 +0000 (20:07 +0200)
Facebook adds the `fbclid` argument to outbound URLs which triggers a
recalculcation of the cache hash. The argument is now added to the
blacklist for chash parameters.

Resolves: #86715
Releases: master, 8.7, 7.6
Change-Id: I8cd66fdfa2c549c65750d6ef896261cccba4b54d
Reviewed-on: https://review.typo3.org/58676
Reviewed-by: Tim Schreiner <schreiner.tim@gmail.com>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
typo3/sysext/core/Configuration/DefaultConfiguration.php

index 6e5fdcb..0dc835b 100644 (file)
@@ -1137,7 +1137,7 @@ return [
         'disableNoCacheParameter' => false,        // Boolean: If set, the no_cache request parameter will become ineffective. This is currently still an experimental feature and will require a website only with plugins that don't use this parameter. However, using "&amp;no_cache=1" should be avoided anyway because there are better ways to disable caching for a certain part of the website (see COA_INT/USER_INT documentation in TSref).
         'cacheHash' => [],        // Array: Processed values of the cHash* parameters, handled by core bootstrap internally
         'cHashIncludePageId' => false,        // Boolean: If enabled the cHash calculation is bound to the current page ID. This is recommended to avoid generation of not required page cache entries. Changing this value will void all links that have a cHash argument, as the cHash will change.
-        'cHashExcludedParameters' => 'L, pk_campaign, pk_kwd, utm_source, utm_medium, utm_campaign, utm_term, utm_content',        // String: The the given parameters will be ignored in the cHash calculation. Example: L,tx_search_pi1[query]
+        'cHashExcludedParameters' => 'L, pk_campaign, pk_kwd, utm_source, utm_medium, utm_campaign, utm_term, utm_content, fbclid',        // String: The the given parameters will be ignored in the cHash calculation. Example: L,tx_search_pi1[query]
         'cHashOnlyForParameters' => '',        // String: Only the given parameters will be evaluated in the cHash calculation. Example: tx_news_pi1[uid]
         'cHashRequiredParameters' => '',        // Optional: Configure Parameters that require a cHash. If no cHash is given but one of the parameters are set, then TYPO3 triggers the configured cHash Error behaviour
         'cHashExcludedParametersIfEmpty' => '',        // Optional: Configure Parameters that are only relevant for the chash if there's an associated value available. And asterisk "*" can be used to skip all empty parameters.