Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Can't open attached images in a new tab anymore #15959

Closed
nukeop opened this issue Jul 19, 2023 · 3 comments
Closed

Can't open attached images in a new tab anymore #15959

nukeop opened this issue Jul 19, 2023 · 3 comments
Labels
A-Media-Repository Uploading, downloading images and video, thumbnailing O-Occasional Affects or can be seen by some users regularly or most users rarely S-Tolerable Minor significance, cosmetic issues, low or no impact to users. T-Enhancement New features, changes in functionality, improvements in performance, or user-facing enhancements. X-Regression Something broke which worked on a previous release

Comments

@nukeop
Copy link

nukeop commented Jul 19, 2023

Description

I used to be able to open media links of this format in a new tab:
https://server.com/_matrix/media/v3/download/server.com/media-id

Now, the default behavior is to download them, instead of opening them. I am running the latest Synapse.

Steps to reproduce

Using Element, attach an image. Middle-click the posted image.

Homeserver

My own homeserver originally, and it affects any server running the latest synapse

Synapse Version

v1.88.0

Installation Method

Docker (matrixdotorg/synapse)

Database

PostgreSQL, single server

Workers

Single process

Platform

N/A

Configuration

No response

Relevant log output

N/A

Anything else that would be useful to know?

No response

@reivilibre
Copy link
Contributor

This was more or less an intentional choice (but perhaps one we should change our mind on), the change appeared in #15680.

The idea is that this is for security: not all documents are really safe to present inline to a web browser.

I can see us changing our minds by e.g. having a small list of media types which we allow to be opened inline, e.g. text/plain, image/png, image/jpeg, etc.

cc @Half-Shot who was talking about this earlier on since this change currently provides an impediment to the way that IRC bridges use the media store as a pastebin for long pieces of text.

@reivilibre reivilibre added A-Media-Repository Uploading, downloading images and video, thumbnailing S-Tolerable Minor significance, cosmetic issues, low or no impact to users. X-Regression Something broke which worked on a previous release O-Occasional Affects or can be seen by some users regularly or most users rarely T-Enhancement New features, changes in functionality, improvements in performance, or user-facing enhancements. labels Jul 21, 2023
@itsrachelfish
Copy link

Duplicate of #15885? Either way, please fix this! 🙏

@reivilibre
Copy link
Contributor

indeed, duplicate of #15885 :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A-Media-Repository Uploading, downloading images and video, thumbnailing O-Occasional Affects or can be seen by some users regularly or most users rarely S-Tolerable Minor significance, cosmetic issues, low or no impact to users. T-Enhancement New features, changes in functionality, improvements in performance, or user-facing enhancements. X-Regression Something broke which worked on a previous release
Projects
None yet
Development

No branches or pull requests

3 participants