Bug #87688
closed
routeEnhancer has problem with additional parameter longer than 31 characters or number as parameter
100%
Description
There is a problem with a page using a routeEnhancer of type Plugin.
- Calling the URL with no additional parameters works fine (e.g. example.com/route/slug )
- Calling with a parameter up to 31 characters length (e.g. example.com/route/slug?abc4567890123456789012345678901 ), everything works fine
- Calling with a parameter longer than 31 characters (e.g. example.com/route/slug?abc45678901234567890123456789012 ) there is an Exeption: #1537633463 OutOfRangeException Hash not resolvable in
typo3/sysext/core/Classes/Routing/Enhancer/VariableProcessor.php:71 - Calling with a parameter only containing number (e.g. example.com/route/slug?0 ) there is an Exception: TypeError - Argument 1 passed to TYPO3\CMS\Core\Routing\Enhancer\VariableProcessor::resolveHash() must be of the type string, integer given in
typo3/sysext/core/Classes/Routing/Enhancer/VariableProcessor.php:373.
Updated by Guido Schmechel over 5 years ago
- Category set to Site Handling, Site Sets & Routing
Updated by Gerrit Code Review over 5 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/c/Packages/TYPO3.CMS/+/60236
Updated by Gerrit Code Review over 5 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/c/Packages/TYPO3.CMS/+/60236
Updated by Gerrit Code Review over 5 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/c/Packages/TYPO3.CMS/+/60236
Updated by Gerrit Code Review over 5 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/c/Packages/TYPO3.CMS/+/60236
Updated by Gerrit Code Review over 5 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/c/Packages/TYPO3.CMS/+/60236
Updated by Gerrit Code Review over 5 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/c/Packages/TYPO3.CMS/+/60236
Updated by Michael Telgkamp about 5 years ago
Additional information: The original usecase is a call with added parameters for tracking a campaign - and all calls to this failed because the additional parameter caused an exception being thrown because the length of the parameter name was longer than 31 characters...
Updated by Oliver Hader about 5 years ago
What do I need to do in order to reproduce this behavior? Please provide relevant route configuration and URLs used in order to trigger the problem.
I was not able to reproduce the misbehavior in TYPO3 v10 with the information from above...
Updated by Michael Telgkamp about 5 years ago
To trigger the exception the parameter needs to be not defined for TYPO3 and longer than 31 characters or numbers only.
Updated by Gerrit Code Review about 5 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/c/Packages/TYPO3.CMS/+/62385
Updated by Gerrit Code Review about 5 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/c/Packages/TYPO3.CMS/+/62385
Updated by Gerrit Code Review about 5 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/c/Packages/TYPO3.CMS/+/62385
Updated by Gerrit Code Review about 5 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/c/Packages/TYPO3.CMS/+/62385
Updated by Gerrit Code Review about 5 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/c/Packages/TYPO3.CMS/+/62385
Updated by Gerrit Code Review about 5 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/c/Packages/TYPO3.CMS/+/62385
Updated by Gerrit Code Review almost 5 years ago
Patch set 1 for branch 9.5 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62411
Updated by Oliver Hader almost 5 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset d74d5d4d4382e6f36a8d830ba61304b78d856b7d.
Updated by Andreas Kienast almost 5 years ago
- Related to Bug #89802: EnhancerSiteRequestTest has superfluous call of var_dump() added
Updated by Gerrit Code Review almost 5 years ago
- Status changed from Resolved to Under Review
Patch set 2 for branch 9.5 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62411
Updated by Gerrit Code Review almost 5 years ago
Patch set 3 for branch 9.5 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62411
Updated by Gerrit Code Review almost 5 years ago
Patch set 4 for branch 9.5 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62411
Updated by Oliver Hader almost 5 years ago
- Status changed from Under Review to Resolved
Applied in changeset 41d29e6230f3cdf4f9c5f0d47762f9514c06ed60.
Updated by Benni Mack almost 5 years ago
- Status changed from Resolved to Closed
Updated by Oliver Hader almost 5 years ago
- Related to Bug #89647: Forms routing not working because of option 'deflatedParameters' added