Bug #91389

viewhelper f:uri produces broken URLs when used with parameter noCache="1"

Added by Peter Linzenkirchner 12 days ago. Updated 7 days ago.

Status:
Needs Feedback
Priority:
Should have
Assignee:
-
Category:
Link Handling, Site Handling & Routing
Target version:
-
Start date:
2020-05-13
Due date:
% Done:

0%

TYPO3 Version:
9
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

This viewhelper:

<f:uri.action pageUid="{subscribeVerifyUid}" extensionName="fpnewsletter" pluginName="pi1" controller="Log" action="verify" arguments="{uid: uid, hash: hash}" noCache="1" absolute="1" />

results in a messed up URI:

https://mydomain.com/newsletter-verify-anmeldung/nc/?tx_fpnewsletter_pi1__action=verify&v47045b4d77d5bcd5075560abef27a6=Log&cHash=4f7788ef8d28686a7c9bd679bc5dd942

When i delete noCache="1" i get:

https://mydomain.com/newsletter-verify-anmeldung/?tx_fpnewsletter_pi1%5Baction%5D=verify&tx_fpnewsletter_pi1%5Bcontroller%5D=Log&cHash=29abd4570bb20da4a0bcad854780f141

which seems correct on first look but is messed up too: the arguments are missing.

I assume this is a side effect from the last update (9.5.17). Until 9.5.16 it worked.

History

#1 Updated by Peter Linzenkirchner 11 days ago

correction:
i made a mistake with the arguments - when i delete noCache="1" the viewhelper works correctly with all arguments. Sorry.

#2 Updated by Oliver Hader 11 days ago

  • Status changed from New to Needs Feedback

Thanks for your report and feedback. Does it mean this issue can be closed or are there further problems?
In case there are still problems, please post the relevant parts of your route enhancer configuration. Thx

#3 Updated by Oliver Hader 11 days ago

  • Category changed from Fluid to Link Handling, Site Handling & Routing

#4 Updated by Oliver Hader 11 days ago

  • Target version set to 9.5.17 & 10.4.2 regressions (temp)

#5 Updated by Peter Linzenkirchner 11 days ago

There is the problem left, that noCache="1" results in a messed up URL.

The route enhancer config:

routeEnhancers:
    NoCacheHandling:
        type: Simple
        routePath: '/{no_cache}'
        requirements:
            no_cache: '.*'
        aspects:
            no_cache:
                type: StaticValueMapper
                map:
                    nc: 1

#6 Updated by Oliver Hader 7 days ago

  • Target version deleted (9.5.17 & 10.4.2 regressions (temp))

Also available in: Atom PDF