TYPO3 Core

Reproducing step 2 should be "Let felogin trigger a releoad redirect", of course.

For people with similar problems:

In the event I do this:

$myChangedRedirectUrl = $event->getRedirectUrl();

// here change $myChangedRedirectUrl

\header('Location: ' . $myChangedRedirectUrl);
throw new \TYPO3\CMS\Extbase\Mvc\Exception\StopActionException();

This is totally off-framework, but afaics the only way to achieve anything with this event. Feels totally ugly and hacky.

BeforeRedirectEvent does work, I just does not match with your expectations.

• adding new setters/getters for new modifiedRedirectUrl property might be fine
• and also fine using $event->getModifiedRedirectUrl() ?? $event->getRedirectUrl() when redirecting in https://github.com/TYPO3/typo3/blob/58558fee9ad4897016bd9a21a02a1c0a07595f6a/typo3/sysext/felogin/Classes/Controller/LoginController.php#L232

The original change in issue #88740 has been merged two years ago for TYPO3 v10.4.0.

I'm fine changing this. Reading through the passive aggressive rants in this issue took me longer than the actual fix.

Patch set 1 for branch 11.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/+/78059

Patch set 2 for branch 11.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/+/78059

Patch set 4 for branch 11.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/+/78059