Only show files in the file panel with a truthy url

[turt2live]

Description

As per https://gitlab.gnome.org/GNOME/fractal/-/issues/101

... or to summarize the issue, an event like the following will appear in the file panel:

  "content": {
    "body": "Ooh, Fractal has an icon in the Papirus icon pack",
    "url": null,
    "msgtype": "m.text",
    "formatted_body": null,
    "format": null
  },

Version information

• Platform: web (in-browser)
• Browser: Chrome 64
• OS: Windows 10
• URL: riot.im/develop

[uhoreg]

see also #4480 and #5076.

[t3chguy]

events are indexed with whether or not they contains_url so this'll be a Synapse bug :(

[t3chguy]

https://github.com/matrix-org/synapse/blob/752b7b32ed1c33651c0c64fbbb4289c3b62ac89b/synapse/storage/events.py#L1119-L1122

it'll also need some migration magic to fix this for all existing events in db

[t3chguy]

Although according to the above code null shouldn't be true for isinstance(event.content["url"], basestring)

[t3chguy]

This seems fine, so maybe its a fixed Synapse bug:

[t3chguy]

weird, even though its right in the db and the filter seems fine the event is in my FilePanel

[t3chguy]

ah turns out it is a Synapse bug where the is_url "formula" is inconsistent between files

[t3chguy]

though as per the spec:

it won't exclude non-truthy but instead non-string