aosus / infrastructure-meta

Issue tracker, status monitor and everything related to aosus's infrastructure.
0 stars 0 forks source link

Broken media from mautrix-discord bridge #5

Closed FarisZR closed 8 months ago

FarisZR commented 8 months ago

The discord CDN proxying trick doesn't work anymore, discord now requires signed URLs to use their CDN. There is this project, https://github.com/ShufflePerson/Discord_CDN which does what we need, a resolver for Discord CDN links using a normal account token.

The issue is I can't seem to get it to build in docker. https://github.com/ShufflePerson/Discord_CDN/issues/1

FarisZR commented 8 months ago

I have forked the project and added some patches for it to work in docker. I tested it locally and its working! PR coming up.

FarisZR commented 8 months ago

this still doesn't work, the reason is the MSC 3680 spec only allows 307/308 responses, 302 isn't accepted. https://github.com/Fizzadar/matrix-spec-proposals/blob/2b93616f5260014f256db96692fd0c46a31b2548/proposals/3860-media-download-redirect.md

FarisZR commented 8 months ago

fixed in https://github.com/aosus/Discord_CDN/commit/238b8fa4a43f1539c2237728bf76ebca34ca2d0f and deployed in https://github.com/aosus/cups-1/commit/34574b2f0eabf1dee1086a666d06a73eba325ec9