Merge pull request #212 from DioxusLabs/jk/remove-attr-dangerous

fix: add a check for dangerousinnerhtml in interpreter
This commit is contained in:
Jonathan Kelley authored and GitHub committed 2022-02-05 21:32:30 -05:00
commit 92ec0a448f
1 file changed
+7 -5
+7 -5
View File
@@ -141,15 +141,17 @@ export class Interpreter {
} }
RemoveAttribute(root, name) { RemoveAttribute(root, name) {
const node = this.nodes[root]; const node = this.nodes[root];
node.removeAttribute(name);
if (name === "value") { if (name === "value") {
node.value = ""; node.value = "";
} } else if (name === "checked") {
if (name === "checked") {
node.checked = false; node.checked = false;
} } else if (name === "selected") {
if (name === "selected") {
node.selected = false; node.selected = false;
} else if (name == "dangerous_inner_html") {
node.innerHTML = "";
} else {
node.removeAttribute(name);
} }
} }
handleEdits(edits) { handleEdits(edits) {