Element-web: Escaping issue with (<50 character sequence (should render as text but instead it drops the rest of the message)

Created on 5 Nov 2020  Â·  3Comments  Â·  Source: vector-im/element-web

Description

Escaping issue with the sequence (<50 :

image

Steps to reproduce

This is a message that came through the Slack bridge, but it renders fine in Element Android -- it's just broken on Element web. You can see from the screenshot that no text is rendered after the "(" character -- the rest of the message is dropped, even though it is in the message source.

bug defect p2 major timeline

All 3 comments

The escaping issue is in the slack bridge. That's not valid HTML.

Yeah, we could switch to more of a tag-soup parser that will try & make sense of whatever nonsense you throw at it, but I don't think it would exactly be a priority and there's a certain argument to failing more obviously so we notice when bridges are misbehaving like this.

Another option would be to ignore the formatted body completely if it's invalid HTML and use the plain body.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

nvbln picture nvbln  Â·  3Comments

arthurlutz picture arthurlutz  Â·  3Comments

turt2live picture turt2live  Â·  3Comments

ara4n picture ara4n  Â·  3Comments

niedzielski picture niedzielski  Â·  3Comments