Media-links now force download instead of just showing content inline in browser

[matrixbot]

This issue was originally created by @dennisse at matrix-org/dendrite#3139.

Background information

• Dendrite version or git SHA: 0.13.0 and above
• SQLite3 or Postgres?: N/A
• Running in Docker?: N/A
• go version: N/A
• Client used (if applicable): Any text-only client, and any non-matrix-client

Description

• What is the problem: Media-links now force download instead of just showing content inline in browser
• Who is affected:
  • Anyone using a text-only client
  • Anyone receiving a link to a matrix-attachement outside of a matrix-client (people share links to content uploaded to matrix outside of matrix).
  • This would also affect anyone on the other side of a bridge where attachements are sent as plain links, i.e. the matrix-IRC-bridges.
• How is this bug manifesting: When clicking a link to an attachement, a download is forced, instead of just showing the media inline in the browser. I used to be able to just click a link and view the attachement directly in my browser.
• When did this first appear: In Fix unsafe hotserving behaviour for multimedia uploads. #3113, and released in 0.13.0

Steps to reproduce

• Try to view https://matrix.dnns.no/_matrix/media/r0/download/dnns.no/cgFcEHtGykrmYbRjbzoxOUtl
• It will present as a download
• Expected behaviour would be to just show the image in the browser

The example above is from a server running matrix-synapse, not dendrite, but the same behaviour was introduced for dendrite in #3113.

This issue basically a duplicate of matrix-org/synapse#15885, but for dendrite. I don't actually run dendrite, but this new behaviour is really irritating to deal with as someone on the other side of a matrix-bridge where people share media.

Possible solutions

@joshqou mentions only returning disposition type inline for media in the original PR for synapse (matrix-org/synapse#15680), and returning attachement for other files. That would be a super solution, but this was unfortunately lost before the PR was merged.

Another less-than-stellar solution would be to provide a way to turn off returning attachement for all files.

In the mean time, I mention a possible workaround in matrix-org/synapse#15885 which should work for anyone hosting their own dendrite-server (behind a reverse proxy) as well.

[matrixbot]

This comment was originally posted by @joshqou at matrix-org/dendrite#3139 (comment).

As is mentioned in the original PR, supporting multiple disposition types would require a rewrite of basically all the media api tests. It would be a non-trivial amount of work and could potentially reintroduce the behaviour that was fixed if done improperly.

I’d suggest modifying your client to support opening native applications for uploads instead, rather than depending on a browser. You can pass most multimedia files directly to mpv or iina (Mac)

[matrixbot]

This comment was originally posted by @dennisse at matrix-org/dendrite#3139 (comment).

It feels extremely user-unfriendly to ask people to modify the behaviour of their systems for something as simple and ubiquitous as opening a link to view an image or a video.

Having to download an image to view it is really stupid. That is not how this is supposed to work. You would never browse a site that made you do that, or continue to open the gifs and images you receive.

[matrixbot]

This comment was originally posted by @joshqou at matrix-org/dendrite#3139 (comment).

If you aren’t familiar with the inner workings of your client then I’d suggest opening an issue about it in their tracker instead. The Matrix spec provides no guidelines as to how media should be returned so both the previous and current behaviour of synapse and dendrite are in-spec. It shouldn’t be up to homeservers to fix improper assumptions by clients

[matrixbot]

This comment was originally posted by @dennisse at matrix-org/dendrite#3139 (comment).

I'm not sure you're following the problem here? The problem isn't with any specific matrix-client. The problem is that it is no longer possible to share a link to a matrix-attachement outside a fancy matrix-client (I say fancy because there are text-only matrix-clients who also present a link to attachements, instead of showing it inline in the chat).

The "client" in this issue is any browser a user might open a link in.

People share links to matrix-attachements outside matrix. Like the link in Steps to reproduce above here. Those links get sent all over the place. I.e. when uploading something to matrix on a channel that is bridged to some other service. The user of the other service, i.e. IRC, have to click the link to view the media.

[matrixbot]

This comment was originally posted by @FlyveHest at matrix-org/dendrite#3139 (comment).

People share links to matrix-attachements outside matrix. Like the link in Steps to reproduce above here. Those links get sent all over the place. I.e. when uploading something to matrix on a channel that is bridged to some other service. The user of the other service, i.e. IRC, have to click the link to view the media.

This is exactly what has happened to me, I am running the official IRC appservice, and after updating to 1.87 i've had multiple users contact me about this very annoying behaviour change.

Clicking a https link to a picture, and having to download it to watch it is very cumbersome, and there is no way for me to change that behaviour as IRC clients are not Matrix clients and does not do anything other than present the link to the resource in its raw state.

The change may have come from a security perspective, but it would seem like no one considered the multitude of clients Matrix can possible run on (one of the very core features, if i'm not mistaken), and not just Matrix specific ones.

[matrixbot]

This comment was originally posted by @jjj333-p at matrix-org/dendrite#3139 (comment).

I thought this has been fixed, perhaps issue could be closed?

[matrixbot]

This comment was originally posted by @S7evinK at matrix-org/dendrite#3139 (comment).

Indeed, given we are now doing the same as Synapse - matrix-org/dendrite#3274