diff options
| author | Richard van der Hoff <richard@matrix.org> | 2019-11-05 17:22:58 +0000 |
|---|---|---|
| committer | Richard van der Hoff <richard@matrix.org> | 2019-11-05 17:22:58 +0000 |
| commit | 81d49cbb07a4dc5a673e31a8a626af6e8a18f801 (patch) | |
| tree | feb0caffef241b54fc3fd90e0819c07d415326fb /synapse | |
| parent | Apply suggestions from code review (diff) | |
| download | synapse-81d49cbb07a4dc5a673e31a8a626af6e8a18f801.tar.xz | |
Fix exception when OpenGraph tag values are ints
Diffstat (limited to 'synapse')
| -rw-r--r-- | synapse/rest/media/v1/preview_url_resource.py | 3 |
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 4d4b3c1462..ec9c4619c9 100644 --- a/synapse/rest/media/v1/preview_url_resource.py +++ b/synapse/rest/media/v1/preview_url_resource.py @@ -274,7 +274,8 @@ class PreviewUrlResource(DirectServeResource): # filter out any stupidly long values keys_to_remove = [] for k, v in og.items(): - if len(k) > OG_TAG_NAME_MAXLEN or len(v) > OG_TAG_VALUE_MAXLEN: + # values can be numeric as well as strings, hence the cast to str + if len(k) > OG_TAG_NAME_MAXLEN or len(str(v)) > OG_TAG_VALUE_MAXLEN: logger.warning( "Pruning overlong tag %s from OG data", k[:OG_TAG_NAME_MAXLEN] ) |