Matrix HTML sanitizer missing table tag allowlist

Fixes on record

Merged to main. A stamp counts the issues a fix is linked to.

• #45631 fix(matrix): preserve markdown table structure

  merged to main

  files: gateway/platforms/matrix.py, tests/gateway/test_matrix.py

  Show the diff

  • gateway/platforms/matrix.py

    @@ -209,7 +209,7 @@ class _MatrixHtmlSanitizer(HTMLParser):
         _ALLOWED_TAGS = {
             "a", "b", "blockquote", "br", "code", "del", "em", "h1", "h2", "h3",
             "h4", "h5", "h6", "hr", "i", "li", "ol", "p", "pre", "s", "strike",
    -        "strong", "ul",
    +        "strong", "table", "tbody", "td", "th", "thead", "tr", "ul",
         }
         _VOID_TAGS = {"br", "hr"}
     

  • tests/gateway/test_matrix.py

    @@ -1090,6 +1090,24 @@ def test_matrix_markdown_preserves_code_fences(self):
             assert "<code" in result
             assert "print" in result
     
    +    def test_matrix_markdown_preserves_table_structure(self):
    +        table = "\n".join(
    +            [
    +                "| Item | Quantity |",
    +                "| --- | --- |",
    +                "| Apples | 4 |",
    +                "| Bread | 1 |",
    +            ]
    +        )
    +
    +        result = self.adapter._markdown_to_html(table)
    +
    +        assert "<table>" in result
    +        assert "<thead>" in result
    +        assert "<tbody>" in result
    +        assert "<th>Item</th>" in result
    +        assert "<td>Apples</td>" in result
    +
     
     # ---------------------------------------------------------------------------
     # Helper: display name extraction