gitlab icon indicating copy to clipboard operation
gitlab copied to clipboard

Bot is sending illegal html attributes

Open krille-chan opened this issue 3 years ago • 1 comments

In the Matrix spec it is very hard specified which HTML tags and attributes are allowed in formatted messages. When using the native setInnerHtml method with a node validator in a browser, it complains a lot in the logs which afaik can't be muted:

Removing disallowed attribute <SPAN title="Issues relevant to the frontend team"> [main.dart.js:12596:47](https://famedly.gitlab.io/company/frontend/famedly-web/main.dart.js)
Removing disallowed attribute <SPAN title="This issue is intended to be the place for an open discussion about a specific topic."> [main.dart.js:12596:47](https://famedly.gitlab.io/company/frontend/famedly-web/main.dart.js)
Removing disallowed attribute <A data-mautrix-exclude-plaintext=""> [main.dart.js:12596:47](https://famedly.gitlab.io/company/frontend/famedly-web/main.dart.js)
Removing disallowed attribute <STRONG data-mautrix-exclude-plaintext=""> [main.dart.js:12596:47](https://famedly.gitlab.io/company/frontend/famedly-web/main.dart.js)
Removing disallowed attribute <A data-mautrix-exclude-plaintext=""> 2 [main.dart.js:12596:47](https://famedly.gitlab.io/company/frontend/famedly-web/main.dart.js)
Removing disallowed attribute <STRONG data-mautrix-exclude-plaintext=""> [main.dart.js:12596:47](https://famedly.gitlab.io/company/frontend/famedly-web/main.dart.js)
Removing disallowed attribute <A data-mautrix-exclude-plaintext=""> 2 [main.dart.js:12596:47](https://famedly.gitlab.io/company/frontend/famedly-web/main.dart.js)
Removing disallowed attribute <STRONG data-mautrix-exclude-plaintext=""> [main.dart.js:12596:47](https://famedly.gitlab.io/company/frontend/famedly-web/main.dart.js)
Removing disallowed attribute <A data-mautrix-exclude-plaintext=""> 2 [main.dart.js:12596:47](https://famedly.gitlab.io/company/frontend/famedly-web/main.dart.js)
Removing disallowed attribute <STRONG data-mautrix-exclude-plaintext=""> [main.dart.js:12596:47](https://famedly.gitlab.io/company/frontend/famedly-web/main.dart.js)
Removing disallowed attribute <A data-mautrix-exclude-plaintext=""> 2 [main.dart.js:12596:47](https://famedly.gitlab.io/company/frontend/famedly-web/main.dart.js)
Removing disallowed attribute <STRONG data-mautrix-exclude-plaintext=""> [main.dart.js:12596:47](https://famedly.gitlab.io/company/frontend/famedly-web/main.dart.js)
Removing disallowed attribute <A data-mautrix-exclude-plaintext=""> 4 [main.dart.js:12596:47](https://famedly.gitlab.io/company/frontend/famedly-web/main.dart.js)
Removing disallowed attribute <STRONG data-mautrix-exclude-plaintext=""> [main.dart.js:12596:47](https://famedly.gitlab.io/company/frontend/famedly-web/main.dart.js)
Removing disallowed attribute <A data-mautrix-exclude-plaintext=""> 2 [main.dart.js:12596:47](https://famedly.gitlab.io/company/frontend/famedly-web/main.dart.js)
Removing disallowed attribute <STRONG data-mautrix-exclude-plaintext=""> [main.dart.js:12596:47](https://famedly.gitlab.io/company/frontend/famedly-web/main.dart.js)
Removing disallowed attribute <A data-mautrix-exclude-plaintext=""> [main.dart.js:12596:47](https://famedly.gitlab.io/company/frontend/famedly-web/main.dart.js)
Removing disallowed attribute <SPAN title="Issues that are actionable / can be picked up for work"> [main.dart.js:12596:47](https://famedly.gitlab.io/company/frontend/famedly-web/main.dart.js)
Removing disallowed attribute <SPAN title="In progress"> [main.dart.js:12596:47](https://famedly.gitlab.io/company/frontend/famedly-web/main.dart.js)
Removing disallowed attribute <A data-mautrix-exclude-plaintext=""> [main.dart.js:12596:47](https://famedly.gitlab.io/company/frontend/famedly-web/main.dart.js)
Removing disallowed attribute <STRONG data-mautrix-exclude-plaintext=""> [main.dart.js:12596:47](https://famedly.gitlab.io/company/frontend/famedly-web/main.dart.js)
Removing disallowed attribute <A data-mautrix-exclude-plaintext=""> 2 [main.dart.js:12596:47](https://famedly.gitlab.io/company/frontend/famedly-web/main.dart.js)
Removing disallowed attribute <STRONG data-mautrix-exclude-plaintext=""> [main.dart.js:12596:47](https://famedly.gitlab.io/company/frontend/famedly-web/main.dart.js)
Removing disallowed attribute <A data-mautrix-exclude-plaintext=""> [main.dart.js:12596:47](https://famedly.gitlab.io/company/frontend/famedly-web/main.dart.js)
Removing disallowed attribute <SPAN title="Issues relevant to the frontend team"> [main.dart.js:12596:47](https://famedly.gitlab.io/company/frontend/famedly-web/main.dart.js)
Removing disallowed attribute <SPAN title="This issue is intended to be the place for an open discussion about a specific topic."> [main.dart.js:12596:47](https://famedly.gitlab.io/company/frontend/famedly-web/main.dart.js)
Removing disallowed attribute <A data-mautrix-exclude-plaintext=""> [main.dart.js:12596:47](https://famedly.gitlab.io/company/frontend/famedly-web/main.dart.js)
Removing disallowed attribute <STRONG data-mautrix-exclude-plaintext=""> [main.dart.js:12596:47](https://famedly.gitlab.io/company/frontend/famedly-web/main.dart.js)
Removing disallowed attribute <A data-mautrix-exclude-plaintext="">

Is there an alternative for using "title" or "data-*" attributes?

krille-chan avatar Apr 05 '22 06:04 krille-chan

The Matrix spec doesn't have any hard specification on which HTML tags and attributes are allowed, it only has a recommendation for what to allow when rendering.

tulir avatar Apr 05 '22 06:04 tulir