tyler.durden
/
icingaweb2
mirror of https://github.com/Icinga/icingaweb2.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

PluginOutput: Transform newlines first and purify HTML afterwards 

fixes #4686
This commit is contained in:
Johannes Meyer 2022-03-01 12:11:09 +01:00
parent dd7f418b48
commit 48e0f1bd1f
2 changed files with 27 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
modules/monitoring/application/views/helpers/PluginOutput.php
View File

@ -88,16 +88,18 @@ class Zend_View_Helper_PluginOutput extends Zend_View_Helper_Abstract
if (empty($output)) {
return '';
}
if ($command !== null) {
$output = $this->hookRenderer->render($command, $output, ! $raw);
}
if (preg_match('~<\w+(?>\s\w+=[^>]*)?>~', $output)) {
// HTML
$output = preg_replace(
$output = HtmlPurifier::process(preg_replace(
self::$htmlPatterns,
self::$htmlReplacements,
HtmlPurifier::process($output)
);
$output
));
$isHtml = true;
} else {
// Plaintext
@ -109,6 +111,7 @@ class Zend_View_Helper_PluginOutput extends Zend_View_Helper_Abstract
);
$isHtml = false;
}
$output = trim($output);
// Add zero-width space after commas which are not followed by a whitespace character
// in oder to help browsers to break words in plugin output
@ -121,6 +124,7 @@ class Zend_View_Helper_PluginOutput extends Zend_View_Helper_Abstract
$output = '<div class="plugin-output preformatted">' . $output . '</div>';
}
}
return $output;
}

20
modules/monitoring/test/php/application/views/helpers/PluginOutputTest.php
View File

@ -152,4 +152,24 @@ class PluginOutputTest extends BaseTestCase
);
}
}
public function testNewlineProcessingInHtmlOutput()
{
$this->checkHtmlOutput(
'This is plugin output\n\n<ul>\n <li>with a HTML list</li>\n</ul>\n\n'
. 'and more text that\nis split onto multiple\n\nlines',
<<<HTML
This is plugin output
<ul>
<li>with a HTML list</li>
</ul>
and more text that
is split onto multiple
lines
HTML
);
}
}