Skip to content

Commit 7cfe425

Browse files
Merge pull request #7 from martinwittmann/xss-fix
Escaping the css classes added here to prevent xss and other security issues
2 parents bf2c0bc + 599e6b4 commit 7cfe425

1 file changed

Lines changed: 5 additions & 0 deletions

File tree

src/BemTwigExtension.php

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55
use Drupal\Core\Template\Attribute;
66
use Twig\Extension\AbstractExtension;
77
use Twig\TwigFunction;
8+
use Drupal\Component\Utility\Html;
89

910
/**
1011
* Class DefaultService.
@@ -129,6 +130,10 @@ public function bem($context, $base_class, $modifiers = [], $blockname = '', $ex
129130
}
130131
// Add class attribute.
131132
if (!empty($classes)) {
133+
// Escape the css classes added to prevent security issues.
134+
$classes = array_map(function($css_class) {
135+
return Html::cleanCssIdentifier($css_class);
136+
}, $classes);
132137
$attributes->setAttribute('class', $classes);
133138
}
134139
return $attributes;

0 commit comments

Comments
 (0)