- 🇬🇷Greece balis_m
Patch from #2 doesn' t apply to the current dev version. So, I rerolled it.
- Issue was unassigned.
- 🇸🇮Slovenia deaom
The patch does no longer apply to the latest version (3.1.0). Attaching the adjusted patch.
As there is a difference between current release and dev version (dev uses trait), attaching a patch for dev as well.
Marked as needs review. - 🇳🇱Netherlands Martijn de Wit 🇳🇱 The Netherlands
I don't know if it is a good idea to make an option to skip the sanitizer. (security etc... )
Why not make the options available: https://github.com/darylldoyle/svg-sanitizer?tab=readme-ov-file#options
There is already a Drupal module doing this:
https://www.drupal.org/project/svg_sanitizer →Maybe this module can work together with the above one or check the code they wrote.
- 🇮🇹Italy eglaw Cittadella
As @martijn-de-wit have stated – it's probably better (and more coherent) to achieve this kind of feature through correct usage of Drupal API, not through messing with the module itself and skipping important parts of code jus but watching some config value from a checkbox. The abovementioned way to move this kind of customization on API level is more clean and extensible.
I will drop this issue as Closed (won't fix), 'cuz the solution proposed is good.
- 🇫🇷France mably
That allowed tags and attributes thing could be interesting to have though.