Does anyone know of a way to prevent JS from running inside of an SVG or stripping out JS from an SVG? In my use case I'd prefer to keep the image as an SVG and not convert it to a JPG. I've thought about stripping out script tags and on-attributes, but I would prefer to avoid a blacklist approach. Alternatively, I've looked at doing a whitelist approach of all attributes and tags, but the list is massive. Any ideas on how to approach this?
Does anyone know of a way to prevent JS from running inside of an SVG
If you embed an SVG as an image, it's guaranteed to not run any contained JS:
As a side effect this also prevents the SVG from loading any external resources. (See also: SVG as an Image)
If you feel fancy, an iframe sandbox would also prevent any script code from executing:
<iframe sandbox src="https://example.com/dangerous.svg"></iframe>
Note that if you're hosting user-provided SVGs, make sure users can't view them directly in the browser (by typing
https://yoursite.example/user-images/dangerous.svg in the URL) since that would trigger the XSS anyway. Instead you need to serve untrusted files as attachments. (See also: Is it safe to store and replay user-provided mime types?)
Yeah I was getting at untrusted users uploading SVGs in which case though I can display it in an image tag, by accessing the resource directly it'll still execute. I can further mitigate this by hosting on another domain but serving them as attachments is a great recommendation that solved the problem. Thanks!– winhowesJan 15, 2018 at 1:21
Use an existing library / software tool that can parse and convert SVG, and that lets you remove scripts. One important rule in security is to avoid reinventing the wheel if you can avoid it.
Which library to use is out of scope for this site, but you could try looking (and possibly asking) on http://stackoverflow.com/, or on https://softwarerecs.stackexchange.com/ , particularly under the tag svg.
As usual, read (and heed) a site's question guidelines (e.g. What is required for a question to contain “enough information”?) before asking.
Sometimes a tool is the right answer to a question, and in those cases it's OK to answer "use tool XYZ". Usual caveats apply, of course - disclose any affiliation you may have with the tool, and don't answer just to advertise a tool (whether your own or someone else's). Jan 12, 2018 at 14:35
May I suggest, BTW, that when you refer people to Software Recommendations, that you include a link to their question quality guidelines? SoftwareRecs is moderated very strictly, I believe we should warn users about that when we send them there. Jan 12, 2018 at 14:43
@S.L.Barth: Yes, sometimes a tool is the right answer, but it is rarely a specific tool. In this case (like in most cases) there's too little information to recommend a specific tool, so I recommended a category, and added help to find the right one.– sleskeJan 12, 2018 at 15:01
1@S.L.Barth: Good point about the guidelines, link added.– sleskeJan 12, 2018 at 15:01
scripttags. You also need to remove the
onattributes. There may also be other things I'm forgetting.