Merge pull request #68 from araise-dev/fix/186-vuln-stored-xss

Fix/186 vuln stored xss

Author: tuxes3

.github/workflows/ci.yaml

@@ -49,7 +49,7 @@ jobs:
                   SYMFONY_REQUIRE: ${{ matrix.symfony }}
               uses: ramsey/composer-install@v2
             - name: Run test suite on PHP ${{ matrix.php }} and Symfony ${{ matrix.symfony }}
-              run: vendor/bin/simple-phpunit
+              run: vendor/bin/phpunit
             - name: Run ECS
               run: vendor/bin/ecs
             - name: Run PHPStan

Makefile

@@ -41,6 +41,6 @@ styles:
 
 ## PHP Unit
 phpunit:
-	vendor/bin/simple-phpunit
+	vendor/bin/phpunit
 
 

composer.json

@@ -25,6 +25,8 @@
         "symfony/stimulus-bundle": "^2.16"
     },
     "require-dev": {
+        "araise/core-bundle": "dev-develop as 1.1",
+        "araise/search-bundle": "dev-develop as 3.1",
         "symfony/phpunit-bridge": "^6.4|^7.0",
         "symfony/config": "^6.4|^7.0",
         "symfony/dependency-injection": "^6.4|^7.0",
@@ -40,7 +42,10 @@
         "symfony/webpack-encore-bundle": "^1.14|^2.1",
         "symfony/security-core": "^6.4|^7.0",
         "symfony/security-bundle": "^6.4|^7.0",
-        "phpstan/phpstan": "^1.5"
+        "phpstan/phpstan": "^1.5",
+        "slevomat/coding-standard": "8.22.1",
+        "phpunit/phpunit": "^10",
+        "symfony/test-pack": "^1.0"
     },
     "autoload": {
         "psr-4": {

phpstan.neon

@@ -3,5 +3,3 @@ parameters:
     paths:
         - src
         - tests
-    bootstrapFiles:
-        - vendor/bin/.phpunit/phpunit-9.6-0/vendor/autoload.php

src/Resources/views/tailwind_2/_table.html.twig

@@ -221,7 +221,8 @@
                                 {% endif %}
                             {% endif %}
                         {% endapply %}
-                        {{ araise_table_column_render(column, row) }}
+                        {% set columnValue = araise_table_column_render(column, row) %}
+                        {{ wwd_is_html_safe(column) ? columnValue|raw : columnValue }}
                         {% if column.option('link_the_column_content') and columnLink|default(false) %}</a>{% endif %}
                     </td>
                 {% endfor %}
@@ -312,7 +313,8 @@
                             class="px-3 py-2 {{ footerColumn.option('attributes')['class'] ?? '' }}"
                             {{ attr|map((value, attr) => "#{attr}=\"#{value}\"")|join(' ')|raw }}
                         >
-                            {{ araise_table_column_render(footerColumn, table.footerData) }}
+                            {% set columnValue = araise_table_column_render(footerColumn, table.footerData) %}
+                            {{ wwd_is_html_safe(footerColumn) ? columnValue|raw : columnValue }}
                         </td>
                     {% endfor %}
 

src/Twig/TableRenderExtension.php

@@ -27,18 +27,20 @@ public function __construct(
 
     public function getFunctions(): array
     {
-        $options = [
+
+        $options = $noSafeOptions = [
             'needs_context' => true,
             'is_safe' => ['html'],
             'is_safe_callback' => true,
             'blockName' => 'blockName',
         ];
+        $noSafeOptions['is_safe'] = [];
 
         return [
             new TwigFunction('araise_table_render', fn ($context, Table $table) => $this->renderTable($context, $table), $options),
             new TwigFunction('araise_table_only_render', fn ($context, Table $table) => $this->renderTable($context, $table, 'table_table'), $options),
             new TwigFunction('araise_table_action_render', fn ($context, Action $action, $entity) => $this->renderTableAction($context, $action, $entity), $options),
-            new TwigFunction('araise_table_column_render', fn ($context, Column $column, $entity) => $this->renderTableColumn($context, $column, $entity), $options),
+            new TwigFunction('araise_table_column_render', fn ($context, Column $column, $entity) => $this->renderTableColumn($context, $column, $entity), $noSafeOptions),
         ];
     }