Skip to content

Commit cdd03db

Browse files
committed
Merge branch 'release/8.4.1'
2 parents e53a370 + 3d37598 commit cdd03db

File tree

4 files changed

+12
-9
lines changed

4 files changed

+12
-9
lines changed

Documentation/Changelog/Readme.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44

55
| Version | Release Date | Description |
66
|------------ |--------------|-----------------------------------------------------------------------------------------------------------------------|
7+
| 8.4.1 | 2021-08-11 | Task: Also Sanitize CSV and XLS export in Pi2 (not only the export in the backend module) |
78
| 8.4.0 | 2021-08-11 | Task: Sanitize CSV and XLS export against excel hacks (see https://typo3.org/security/advisory/typo3-psa-2021-002) |
89
| | | Task: Add automatic test via github actions |
910
| 8.3.3 | 2021-07-27 | Bugfix: Fix some more typehint problems |

Resources/Private/Templates/Output/ExportCsv.html

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -22,15 +22,17 @@
2222

2323
<f:if condition="{vh:condition.isArray(val: '{answer.value}')}">
2424
<f:else>
25-
"<vh:string.removeQuote>{answer.value}</vh:string.removeQuote>";
25+
"<vh:string.removeQuote><vh:string.sanitizeCsvCell>{answer.value}</vh:string.sanitizeCsvCell></vh:string.removeQuote>";
2626
</f:else>
2727
<f:then>
2828
"<vh:string.removeQuote>
29-
<f:for each="{answer.value}" as="singleValue">
30-
<f:if condition="{singleValue}">
31-
{singleValue},
32-
</f:if>
33-
</f:for>
29+
<vh:string.sanitizeCsvCell>
30+
<f:for each="{answer.value}" as="singleValue">
31+
<f:if condition="{singleValue}">
32+
{singleValue},
33+
</f:if>
34+
</f:for>
35+
</vh:string.sanitizeCsvCell>
3436
</vh:string.removeQuote>";
3537
</f:then>
3638
</f:if>

Resources/Private/Templates/Output/ExportXls.html

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -32,12 +32,12 @@
3232
<f:then>
3333
<f:for each="{answer.value}" as="singleValue">
3434
<f:if condition="{singleValue}">
35-
{singleValue},
35+
<vh:string.sanitizeCsvCell>{singleValue}</vh:string.sanitizeCsvCell>,
3636
</f:if>
3737
</f:for>
3838
</f:then>
3939
<f:else>
40-
{answer.value}
40+
<vh:string.sanitizeCsvCell>{answer.value}</vh:string.sanitizeCsvCell>
4141
</f:else>
4242
</f:if>
4343
</f:if>

ext_emconf.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
and easy to use mailform extension with a lots of features
77
(spam prevention, marketing information, optin, ajax submit, diagram analysis, etc...)',
88
'category' => 'plugin',
9-
'version' => '8.4.0',
9+
'version' => '8.4.1',
1010
'state' => 'stable',
1111
'author' => 'Powermail Development Team',
1212
'author_email' => 'service@in2code.de',

0 commit comments

Comments
 (0)