summary refs log tree commit diff
diff options
context:
space:
mode:
authorErik Johnston <erik@matrix.org>2016-08-05 14:45:11 +0100
committerErik Johnston <erik@matrix.org>2016-08-05 14:45:11 +0100
commit5bcccfde6c8e32b0186a072074c13fbeb51c0d04 (patch)
tree242d1616bf2af5bd31d4d6efa66aea68f8b04f27
parentMerge pull request #987 from matrix-org/erikj/fix_backfill_auth (diff)
downloadsynapse-5bcccfde6c8e32b0186a072074c13fbeb51c0d04.tar.xz
Don't include html comments in description
-rw-r--r--synapse/rest/media/v1/preview_url_resource.py3
1 files changed, 2 insertions, 1 deletions
diff --git a/synapse/rest/media/v1/preview_url_resource.py b/synapse/rest/media/v1/preview_url_resource.py
index 4060593f7f..bdd0e60c5b 100644
--- a/synapse/rest/media/v1/preview_url_resource.py
+++ b/synapse/rest/media/v1/preview_url_resource.py
@@ -345,7 +345,8 @@ class PreviewUrlResource(Resource):
                 # lines)
                 text_nodes = (
                     re.sub(r'\s+', '\n', el.text).strip()
-                    for el in cloned_tree.iter() if el.text
+                    for el in cloned_tree.iter()
+                    if el.text and isinstance(el.tag, basestring)  # Removes comments
                 )
                 og['og:description'] = summarize_paragraphs(text_nodes)