Skip to content

Don't inject dynamic values in SVG #16

@Loirooriol

Description

@Loirooriol

The code shouldn't be vulnerable to SVG injection because the only external data that I insert are colors, and I sanitize them. However, this approach is fragile, scales poorly if I want to insert other values (like font styles in #13), and is bad practice. I should use DOM instead.

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't workingfixed

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions