Sanitizer removes too much of data from SVG

Question

Sanitizer removes too much of data from SVG

FreddieChopin opened this issue 8 years ago · comments

When the attached file is uploaded via this plugin, the arrow on the vertical axis is rotated 90degrees - instead of pointing up it points to the right.

Original:
scheduling.zip

Sanitized:
scheduling-sanitized.zip

Wrong:

Correct:

If's hard to get any useful diff, as the plugin also changes all whitespace in the file...

Freddie Chopin · Answer 1 · Sun Nov 27 2016 05:52:00 GMT+0800 (China Standard Time)

Just to be clear, I'm using sanitizer via Safe SVG plugin in wordpress.

Daryll Doyle · Answer 2 · Mon Nov 28 2016 18:44:43 GMT+0800 (China Standard Time)

Thanks for the report Freddie, I've narrowed this down to the orient attribute being stripped from the arrowheads.

I'll take a closer look at this tonight and hopefully get a fix up for you!

Daryll Doyle · Answer 3 · Fri Dec 02 2016 07:18:00 GMT+0800 (China Standard Time)

7a1e995 fixes this one. A WordPress plugin update will follow shortly!

Freddie Chopin · Answer 4 · Sat Dec 03 2016 19:14:56 GMT+0800 (China Standard Time)

Thanks, the arrow is no longer changed!