3
98%
master: 97%

Ran 17 Nov 2020 03:00AM UTC

Files 98

Run time 24s

Badge

Embed ▾

Committed 16 Nov 2020 09:54PM UTC coverage: 98.498% (-0.2%) from 98.737%

Job # 1432.3

Build Type

push

travis-ci

Committed by

NamelessCoder

Commit Message

[SECURITY] Introduce selective argument escaping

Addresses three XSS vulnerabilities:

* The "then" and "else" arguments of condition ViewHelpers
  were not escaped. They are now escaped based on the
  escapeChildren toggle of the ViewHelper, which is ON by
  default in subclasses of AbstractConditionViewHelper.
* Content arguments in ViewHelpers which disable
  escapeOutput were not escaped, but values passed as
  child node were escaped. Both cases are now treated
  the same and escaping is based on escapeChildren state.
* TagBased ViewHelpers allowed attribute names containing
  HTML if passed in "additionalAttributes" which made XSS
  possible by crafting array keys with HTML. Attribute names
  are now subjected to the same escaping as attribute values.

Also fixes a couple of undesirable behaviors as well, e.g. avoids
double escaping of output in some combinations of escapeOutput=true
and quoted arguments.

# Conflicts:
#	src/Core/ViewHelper/AbstractConditionViewHelper.php

# Conflicts:
#	src/Core/ViewHelper/TagBuilder.php
#	tests/Unit/Core/Parser/TemplateParserTest.php

Run Details

2623 of 2663 relevant lines covered (98.5%)

52.73 hits per line

TYPO3 / Fluid / 1432 / 3
98%
master: 97%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 1432.3

TYPO3 / Fluid / 1432 / 3 98% master: 97%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 1432.3

TYPO3 / Fluid / 1432 / 3
98%
master: 97%

README BADGES
x