Bug #104398
closedFelogin - ignoring noredirect param
100%
Description
in Line 249 it is checked for argument 'noredirect' - but in
the f:form.hidden has only a name attribute without extbase context (name="noredirect" instead of name="tx_felogin_login[noredirect]")
so the 'noredirect' argument never gets checked
following test works:
|| (isset($this->request->getParsedBody()['noredirect']) && $this->request->getParsedBody()['noredirect'] === '1')
Updated by Gerrit Code Review 7 months ago
- Status changed from Accepted to Under Review
Patch set 1 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/85313
Updated by Gerrit Code Review 7 months ago
Patch set 2 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/85313
Updated by Gerrit Code Review 5 months ago
Patch set 1 for branch 12.4 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/86144
Updated by Torben Hansen 5 months ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 0dda9b9a852fb9126a4c55e61ebe889a35591417.
Updated by Christian Kuhn 5 months ago
- Related to Task #105010: Protect felogin controller isRedirectDisabled() added