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

Only let authed local users request remote content (for the first time a given piece of remote content is requested) #2133

Open
ara4n opened this issue Apr 17, 2017 · 3 comments
Labels
T-Enhancement New features, changes in functionality, improvements in performance, or user-facing enhancements.

Comments

@ara4n
Copy link
Member

ara4n commented Apr 17, 2017

I think i'm right in saying that curently any arbitrary user on the 'net can query a media URL on an arbitrary HS to get it to cache arbitrary content from Matrix. Surely this should be limited (for the first request) to users local to that HS

@ara4n ara4n changed the title Only let authed local users request remote content Only let authed local users request remote content (for the first time a given piece of remote content is requested) Apr 17, 2017
@ara4n
Copy link
Member Author

ara4n commented Apr 17, 2017

To clarify: why would you ever let remote users cause your homeserver to cache remote content? only local users should be able to do this. given the first person to query content (be it a bridge or a real user) will be authed, this seems like a reasonable requirement to avoid abuse.

@richvdh
Copy link
Member

richvdh commented Dec 3, 2019

related: matrix-org/matrix-spec#870

@ShadowJonathan
Copy link
Contributor

Slightly related; Discord allows people to copy-paste content URLs to then paste in other chats, or even on other websites. This has created some sort of social culture around it which discord tolerates, because it means images arent getting re-uploaded every time, which saves data on their servers.

This is somewhat the same on the matrix federation, because someone could post a MXC URI in another room, and i want to set that as my avatar, and then someone sees that avatar, and would want to send that to their room to show off, its hard to track who is "authenticated" to view the picture/content, then.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
T-Enhancement New features, changes in functionality, improvements in performance, or user-facing enhancements.
Projects
None yet
Development

No branches or pull requests

4 participants