Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add MediaProxy support to serve authenticated Matrix media #793

Open
wants to merge 16 commits into
base: develop
Choose a base branch
from
Open

Add MediaProxy support to serve authenticated Matrix media #793

wants to merge 16 commits into from

Conversation

tadzik
Copy link
Contributor

@tadzik tadzik commented Jan 3, 2025 •
edited
Loading

This slipped through the cracks, and unbreaks currently broken Matrix->Slack media bridging.

Depends on #775 to allow for newer matrix-appservice-bridge

TODO:

  • Figure out if we still need encryption support and how to handle it
    • Should just work as-is, though I wasn't able to test it
  • Non-images are uploaded as URLs, and are not being mirrored by Slack – it's as good time as any to add forever-media support to the upstream MediaProxy.

@tadzik tadzik requested a review from a team as a code owner January 3, 2025 14:20
@tadzik tadzik force-pushed the tadzik/media-proxy branch from 1bdf5aa to 09d7ce2 Compare January 3, 2025 14:34
@tadzik tadzik force-pushed the tadzik/media-proxy branch from 09d7ce2 to 586881f Compare January 3, 2025 14:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@tadzik