matrix-org / matrix.to

A simple stateless privacy-protecting URL redirecting service for Matrix
http://matrix.to
Apache License 2.0
893 stars 189 forks source link

invalid matrix.to link recognized as valid but fails on mobile #179

Closed Bubu closed 2 years ago

Bubu commented 3 years ago

Describe the bug https://matrix.to/#cctg:bubu1.eu is recognized as a valid matrix.to link but it's invalid according to the spec. This leads to the link seeming to work on desktop/in the browser but then failing to work on mobile clients.

To Reproduce Steps to reproduce the behavior:

  1. open https://matrix.to/#cctg:bubu1.eu
  2. try to open it on mobile
  3. ... it won't load

Expected behavior Don't display the #cctg:bubu1.eu room when visiting https://matrix.to/#cctg:bubu1.eu

Screenshots

image

Desktop (please complete the following information):

Smartphone (please complete the following information):

ludwigbald commented 3 years ago

Same behavior for me, on Android 10 with Chrome. When opening the room in the browser, I get a preview. But when opening the link with Element Android, I get nothing.

There are two things that need to happen: matrix.to should recognize this link as wrong (#178 ) (maybe suggest the correct verison), and Element Android should implement an error message for invalid matrix.to links.

Bubu commented 3 years ago

implement an error message for invalid matrix.to links.

I does I think? --> I got a toast that said "Not a valid matrix.to link"

ludwigbald commented 3 years ago

I don't, though. I'll mention it over there too 👍

bwindels commented 2 years ago

I think there is a / missing, does https://matrix.to/#/#cctg:bubu1.eu work?

Bubu commented 2 years ago

Nice fix! :-) :tada:

Thanks @DanilaFe and @bwindels!