Skip to content

fix: centralize render escaping (v1.2.3)#17

Merged
masonlet merged 6 commits into
mainfrom
fix/render-escaping
Jul 12, 2026
Merged

fix: centralize render escaping (v1.2.3)#17
masonlet merged 6 commits into
mainfrom
fix/render-escaping

Conversation

@masonlet

Copy link
Copy Markdown
Member

Moves HTML escaping from parseQueryParams to the render layer, so all text is escaped exactly once at the point it enters markup:

  • renderSvg now escapes title, was escaped during parsing
  • createLegend now escapes language names, was unescaped
  • rename resolveColour to resolveThemeColour
  • parseQueryParams now returns chartTitle unescaped

@masonlet
masonlet merged commit 8cae9d1 into main Jul 12, 2026
2 checks passed
@masonlet
masonlet deleted the fix/render-escaping branch July 12, 2026 22:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant