[BUGFIX] Remove HTML from exception messages 09/38609/2
authorMathias Brodala <mbrodala@pagemachine.de>
Fri, 10 Apr 2015 06:49:56 +0000 (08:49 +0200)
committerGeorg Ringer <georg.ringer@gmail.com>
Mon, 20 Apr 2015 18:58:02 +0000 (20:58 +0200)
Exception messages are independent from a specific context and thus
should not contain HTML code.

Resolves: #66344
Releases: master
Change-Id: I8bfab2d1da3fc3baf344d833eaaec8b267732baf
Reviewed-on: http://review.typo3.org/38609
Reviewed-by: Andreas Fernandez <andreas.fernandez@aspedia.de>
Tested-by: Andreas Fernandez <andreas.fernandez@aspedia.de>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
typo3/sysext/backend/Classes/Controller/LoginController.php
typo3/sysext/backend/Classes/Tree/View/AbstractTreeView.php
typo3/sysext/core/Classes/Authentication/AbstractUserAuthentication.php
typo3/sysext/extensionmanager/Classes/Utility/Connection/TerUtility.php

index cf6362a..cb1fe3e 100644 (file)
@@ -343,7 +343,7 @@ class LoginController {
                                         * 26/4 2005: This does not work anymore, because the saving of challenge values
                                         * in $_SESSION means the system will act as if the password was wrong.
                                         */
-                                       throw new \RuntimeException('Login-error: Yeah, that\'s a classic. No cookies, no TYPO3.<br /><br />' .
+                                       throw new \RuntimeException('Login-error: Yeah, that\'s a classic. No cookies, no TYPO3. ' .
                                                'Please accept cookies from TYPO3 - otherwise you\'ll not be able to use the system.', 1294586846);
                                } else {
                                        // try it once again - that might be needed for auto login
index 3eb8054..f6191af 100644 (file)
@@ -756,7 +756,7 @@ abstract class AbstractTreeView {
                        $crazyRecursionLimiter--;
                        $newID = $row['uid'];
                        if ($newID == 0) {
-                               throw new \RuntimeException('Endless recursion detected: TYPO3 has detected an error in the database. Please fix it manually (e.g. using phpMyAdmin) and change the UID of ' . $this->table . ':0 to a new value.<br /><br />See <a href="http://forge.typo3.org/issues/16150" target="_blank">forge.typo3.org/issues/16150</a> to get more information about a possible cause.', 1294586383);
+                               throw new \RuntimeException('Endless recursion detected: TYPO3 has detected an error in the database. Please fix it manually (e.g. using phpMyAdmin) and change the UID of ' . $this->table . ':0 to a new value. See http://forge.typo3.org/issues/16150 to get more information about a possible cause.', 1294586383);
                        }
                        // Reserve space.
                        $this->tree[] = array();
index eb1bdc5..2878a46 100644 (file)
@@ -609,9 +609,9 @@ abstract class AbstractUserAuthentication {
                        if ($this->formfield_status && $loginData['uident'] && $loginData['uname']) {
                                $httpHost = GeneralUtility::getIndpEnv('TYPO3_HOST_ONLY');
                                if (!$this->getMethodEnabled && ($httpHost != $authInfo['refInfo']['host'] && !$GLOBALS['TYPO3_CONF_VARS']['SYS']['doNotCheckReferer'])) {
-                                       throw new \RuntimeException('TYPO3 Fatal Error: Error: This host address ("' . $httpHost . '") and the referer host ("' . $authInfo['refInfo']['host'] . '") mismatches!<br />
-                                               It\'s possible that the environment variable HTTP_REFERER is not passed to the script because of a proxy.<br />
-                                               The site administrator can disable this check in the "All Configuration" section of the Install Tool (flag: TYPO3_CONF_VARS[SYS][doNotCheckReferer]).', 1270853930);
+                                       throw new \RuntimeException('TYPO3 Fatal Error: Error: This host address ("' . $httpHost . '") and the referer host ("' . $authInfo['refInfo']['host'] . '") mismatches! ' .
+                                               'It is possible that the environment variable HTTP_REFERER is not passed to the script because of a proxy. ' .
+                                               'The site administrator can disable this check in the "All Configuration" section of the Install Tool (flag: TYPO3_CONF_VARS[SYS][doNotCheckReferer]).', 1270853930);
                                }
                                // Delete old user session if any
                                $this->logoff();
index 84a9026..c2f5414 100644 (file)
@@ -53,7 +53,7 @@ class TerUtility {
                        // Fetch and return:
                        $extensionData = $this->decodeExchangeData($t3x);
                } else {
-                       throw new ExtensionManagerException('Error: MD5 hash of downloaded file not as expected:<br />' . $md5 . ' != ' . $expectedMd5, 1334426098);
+                       throw new ExtensionManagerException('Error: MD5 hash of downloaded file not as expected: ' . $md5 . ' != ' . $expectedMd5, 1334426098);
                }
                return $extensionData;
        }