Skip to content

Mitigate localization security issue by disabling HTML rendering#4

Merged
ROBdk97 merged 1 commit intomainfrom
fix-DOM-text-reinterpreted-as-HTML
Apr 17, 2026
Merged

Mitigate localization security issue by disabling HTML rendering#4
ROBdk97 merged 1 commit intomainfrom
fix-DOM-text-reinterpreted-as-HTML

Conversation

@ROBdk97
Copy link
Copy Markdown
Owner

@ROBdk97 ROBdk97 commented Apr 17, 2026

This pull request addresses a security issue related to localization in the Property Inspector by removing HTML-based translation rendering and switching to plain text for help text. It also updates the changelog to document this patch.

Security and Localization Fixes:

  • Removed HTML-based translation rendering from the shared i18n helper and switched localized help text to plain text in ControlPanel.html to prevent potential security issues. (PluginCore/PropertyInspector/Keys/ControlPanel.html, [1] [2]

Documentation Updates:

  • Added a changelog entry for v1.2.2.0 describing the localization security fix. (CHANGELOG.md, CHANGELOG.mdR18-R23)

Mitigate a localization security issue by disabling HTML-based translation rendering. Switched data-i18n-html usages to data-i18n-text in ControlPanel.html, removed support for data-i18n-html/innerHTML in the i18n source (sc-common.js) and updated the compiled components (sc-components.js). Updated locale files to remove embedded HTML (e.g. removed <b> tags) and added a changelog entry for v1.2.2.0. Also includes generated/packed plugin outputs.
@ROBdk97 ROBdk97 self-assigned this Apr 17, 2026
@ROBdk97 ROBdk97 added the bug Something isn't working label Apr 17, 2026
@ROBdk97 ROBdk97 merged commit a5d4415 into main Apr 17, 2026
7 checks passed
@ROBdk97 ROBdk97 deleted the fix-DOM-text-reinterpreted-as-HTML branch April 17, 2026 14:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant