benawad / dogehouse

Taking voice conversations to the moon 🚀
https://dogehouse.tv/
MIT License
9.12k stars 1.48k forks source link

HTML tags #2851

Closed ghost closed 3 years ago

ghost commented 3 years ago

Describe the bug A bug making DogeHouse crash when you add an a tag to the chat text.

To Reproduce Steps to reproduce the behavior:

  1. Send a message in chat.
  2. Inspect element and add \<strong>hi\</strong>
  3. Delete the message
  4. See error

Expected behavior It should delete the message without any error.

What device are you on? I tried this all all devices.

jcsrb commented 3 years ago

using "Inspect element" is probably out of scope for protecting against what is the use-case of manually modifying the DOM?

ghost commented 3 years ago

using "Inspect element" is probably out of scope for protecting against what is the use-case of manually modifying the DOM?

Well its still bad

willdoescode commented 3 years ago

You manually added a element that was not supposed to be there, this is not a bug with dogehouse.

ghost commented 3 years ago

well NVM