[BUGFIX] felogin reset password links not clickable

Encoding a few extra character besides the ones according to RFC3986
makes password reset links working again in various mail clients which
do not comply to this RFC (and which do not have plans to fix this in
the near future).

Change-Id: I0b42bef6cb732c5fc6cc2d900407271cb606e301
Fixes: #23984
Releases: 6.2, 6.1, 6.0, 4.7, 4.5
Reviewed-on: https://review.typo3.org/17176
Reviewed-by: Markus Klein
Tested-by: Markus Klein
Reviewed-by: Jigal van Hemert
Tested-by: Jigal van Hemert

diff --git a/typo3/sysext/felogin/Classes/Controller/FrontendLoginController.php b/typo3/sysext/felogin/Classes/Controller/FrontendLoginController.php
index d39a72c..a71e9f3 100644 (file)
--- a/typo3/sysext/felogin/Classes/Controller/FrontendLoginController.php
+++ b/typo3/sysext/felogin/Classes/Controller/FrontendLoginController.php
@@ -411,8 +411,8 @@ class FrontendLoginController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlugin
                $isBaseURL = !empty($GLOBALS['TSFE']->baseUrl);
                $isFeloginBaseURL = !empty($this->conf['feloginBaseURL']);
                $link = $this->pi_getPageLink($GLOBALS['TSFE']->id, '', array(
-                       $this->prefixId . '[user]' => $user['uid'],
-                       $this->prefixId . '[forgothash]' => $randHash
+                       rawurlencode($this->prefixId . '[user]') => $user['uid'],
+                       rawurlencode($this->prefixId . '[forgothash]') => $randHash
                ));
                // Prefix link if necessary
                if ($isFeloginBaseURL) {