Skip to content

Test for unsafe input fields in web UI against XSS#6639

Open
plenaerts wants to merge 7 commits into
beetbox:masterfrom
plenaerts:master
Open

Test for unsafe input fields in web UI against XSS#6639
plenaerts wants to merge 7 commits into
beetbox:masterfrom
plenaerts:master

Conversation

@plenaerts
Copy link
Copy Markdown
Contributor

Description

Try to prevent GHSA-3gxm-wfjx-m847 from happening again.

This test verifies that the Underscore.js templates in index.html use
the escaping syntax (<%- %) instead of the non-escaping syntax (<%= %).

To Do

  • Changelog.

@plenaerts plenaerts requested a review from a team as a code owner May 14, 2026 17:43
@github-actions
Copy link
Copy Markdown

Thank you for the PR! The changelog has not been updated, so here is a friendly reminder to check if you need to add an entry.

@codecov
Copy link
Copy Markdown

codecov Bot commented May 14, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.46%. Comparing base (aa33b1c) to head (7a4c554).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #6639      +/-   ##
==========================================
+ Coverage   72.44%   72.46%   +0.01%     
==========================================
  Files         160      160              
  Lines       20690    20690              
  Branches     3272     3272              
==========================================
+ Hits        14989    14992       +3     
+ Misses       4976     4974       -2     
+ Partials      725      724       -1     

see 2 files with indirect coverage changes

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Comment thread test/plugins/test_web_xss.py Outdated
Comment thread test/plugins/test_web_xss.py Outdated
Comment thread test/plugins/test_web_xss.py Outdated
Comment thread test/plugins/test_web_xss.py Outdated
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.

2 participants