Skip to content

Refine sanitizer documentation#41434

Merged
julien-deramond merged 8 commits intomainfrom
docs/clarify-xss-risk
May 22, 2025
Merged

Refine sanitizer documentation#41434
julien-deramond merged 8 commits intomainfrom
docs/clarify-xss-risk

Conversation

@bardiharborow
Copy link
Copy Markdown
Member

@bardiharborow bardiharborow commented May 4, 2025

Disabling or modifying content sanitization can expose web authors to cross-site scripting (XSS) attacks. We also receive occasional reports claiming that the presence of the escape hatch is itself a vulnerability.

This PR expands the sanitizer documentation, highlighting both that using the escape hatch incorrectly can be unsafe and that it is intentionally unsafe.

Changed pages:

Disabling or modifying content sanitization can expose web authors to cross-site scripting (XSS) attacks. We also receive occasional reports claiming that the presence of the escape hatch is itself a vulnerability.

This commit expands the documentation, highlighting both that using the escape hatch incorrectly can be unsafe and that it is intentionally unsafe.
@bardiharborow bardiharborow requested a review from a team May 5, 2025 11:54
@julien-deramond julien-deramond requested a review from a team as a code owner May 5, 2025 18:39
Copy link
Copy Markdown
Member

@julien-deramond julien-deramond left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot for this PR, @bardiharborow — your changes really improved things 👌
I made a few minor tweaks in fc901e6 and added some comments as well.

Comment thread site/src/content/docs/getting-started/javascript.mdx Outdated
Comment thread site/src/content/docs/getting-started/javascript.mdx Outdated
Comment thread site/src/content/docs/getting-started/javascript.mdx Outdated
@julien-deramond julien-deramond moved this to Review in progress in v5.3.6 May 5, 2025
@julien-deramond julien-deramond requested a review from mdo May 5, 2025 18:46
@julien-deramond julien-deramond moved this to Review in progress in v5.3.7 May 5, 2025
@julien-deramond julien-deramond removed this from v5.3.6 May 5, 2025
@julien-deramond julien-deramond self-requested a review May 22, 2025 19:42
@julien-deramond julien-deramond moved this from Review in progress to Ready to merge in v5.3.7 May 22, 2025
@julien-deramond julien-deramond merged commit eeb1209 into main May 22, 2025
14 checks passed
@julien-deramond julien-deramond deleted the docs/clarify-xss-risk branch May 22, 2025 19:48
@github-project-automation github-project-automation Bot moved this from Ready to merge to Done in v5.3.7 May 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants