1
97%
master: 97%

Ran 17 Nov 2020 01:19AM UTC

Files 110

Run time 13s

Badge

Embed ▾

Committed 16 Nov 2020 09:48PM UTC coverage: 96.72% (+4.8%) from 91.885%

Job # 1429.1

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

Run Details

2683 of 2774 relevant lines covered (96.72%)

51.46 hits per line

TYPO3 / Fluid / 1429 / 1
97%
master: 97%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 1429.1

TYPO3 / Fluid / 1429 / 1 97% master: 97%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 1429.1

TYPO3 / Fluid / 1429 / 1
97%
master: 97%

README BADGES
x