diff options
| author | Stefan Giehl <stefan@matomo.org> | 2019-04-03 00:15:49 +0300 |
|---|---|---|
| committer | diosmosis <diosmosis@users.noreply.github.com> | 2019-04-03 00:15:49 +0300 |
| commit | 98e53787f9d23a3bcb4fe0601fbfc6e23c4f3b03 (patch) | |
| tree | ffd2c3fc3fee565e8cd510586228ec54129e21e0 /core | |
| parent | 0df4c84a15de3dfb0ef4247c15a1dce97347976a (diff) | |
Strip HTML tags in API error messages (#14213)
* Strip HTML tags in API error messages
* check if root request is api request
* fix some tests
Diffstat (limited to 'core')
| -rw-r--r-- | core/API/ResponseBuilder.php | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/core/API/ResponseBuilder.php b/core/API/ResponseBuilder.php index 9845dee275..bbdd63c103 100644 --- a/core/API/ResponseBuilder.php +++ b/core/API/ResponseBuilder.php @@ -168,6 +168,10 @@ class ResponseBuilder { $message = ExceptionToTextProcessor::getWholeBacktrace($exception, $this->shouldPrintBacktrace); + if ($exception instanceof \Piwik\Exception\Exception && $exception->isHtmlMessage() && Request::isRootRequestApiRequest()) { + $message = strip_tags(str_replace('<br />', PHP_EOL, $message)); + } + return Renderer::formatValueXml($message); } |