matrix-docker-ansible-deploy icon indicating copy to clipboard operation
matrix-docker-ansible-deploy copied to clipboard

Published 20 hours ago • verified publisher icon spantaleev

Update `matrix-appservice-irc` version

Open thomwiggers opened this issue 1 year ago • 13 comments

The update to 3.0 brings with it that it now uses MediaProxy.

thomwiggers avatar Sep 09 '24 11:09 thomwiggers

Setting up media proxy is a bit above my level

thomwiggers avatar Sep 09 '24 12:09 thomwiggers

I have rebased this, but would appreciate some help getting the authenticated media set up correctly

thomwiggers avatar Dec 04 '24 10:12 thomwiggers

It seems that the media proxy is indeed part of the bridge, but that you need to set up e.g. traefik to route traffic to it. This is well above my paygrade.

thomwiggers avatar May 07 '25 20:05 thomwiggers

@thomwiggers I had a shot at the traefik changes, not really tested though: https://github.com/JadedBlueEyes/matrix-docker-ansible-deploy/tree/patch-11

JadedBlueEyes avatar Sep 17 '25 13:09 JadedBlueEyes

Thanks, that does look plausible... I hope I will have some time to test it.

thomwiggers avatar Sep 17 '25 14:09 thomwiggers

The following gets posted to IRC

https://matrix.example.com/irc/v1/media/download/AXhzMyKXPbuQNpDBTy0kF9LvE0dTD7gjU9AbvfQaqoTBvqISAPERZVGQLJiz0IS9MrwQaNG33xyAY-bLjVrKSsBCeZheuENAAHdnZ3JzLm5sL3docFF4R0hhYXB1dmpXWW1rV092RmVVSA

But following it just gets a 404 :(

I haven't had time to dig into the logs yet.

thomwiggers avatar Sep 19 '25 12:09 thomwiggers

The Traefik GUI dashboard is likely to show more hints

JadedBlueEyes avatar Sep 22 '25 06:09 JadedBlueEyes

The Traefik GUI says the rule is Host(`example.com`) && PathPrefix(`/irc/`) so it appears the hostname is not properly specified (matrix is on a subdomain)

thomwiggers avatar Sep 24 '25 15:09 thomwiggers

Using matrix_server_fqn_matrix instead of matrix_host brings me to:

Cannot GET /irc/v1/media/download/AXhzMyKXPbuQNpDBTy0kF9LvE0dTD7gjU9AbvfQaqoTBvqISAPERZVGQLJiz0IS9MrwQaNG33xyAY-bLjVrKSsBCeZheuENAAHdnZ3JzLm5sL3docFF4R0hhYXB1dmpXWW1rV092RmVVSA

which appears to result from the bridge, because:

sep 24 17:34:48 archeron matrix-appservice-irc[1084584]: INFO 15:34:48:214 [MediaProxy] GET /irc/v1/media/download/AduY_44ZkBJckUbjG2zsp9ZDKKyjFpnEJuC07BiWMnBjmm2QHUKgrL5JoUrRsjOcyEdV-P7R3cuLM6fdygaSL8lCeZoGljJQAHdnZ3JzLm5sL3RES2xpZWxPZHJYRGpjaHVmaXFXR3NvUQ 172.26.0.2 Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/26.0 Safari/605.1.15

thomwiggers avatar Sep 24 '25 15:09 thomwiggers

Going off https://github.com/matrix-org/matrix-appservice-irc/issues/1828 it may be that /irc needs to get stripped off the URL

thomwiggers avatar Sep 24 '25 15:09 thomwiggers

From what I can see CI is just failing on some typos and a capital in a variable name (_certResolver)

JadedBlueEyes avatar Oct 13 '25 21:10 JadedBlueEyes

yeah I simply haven't had time to look at that + some variable cleanup yet.

thomwiggers avatar Oct 14 '25 07:10 thomwiggers

This has been working for me for a while now. With the last couple of fixes, I think this is ready to get reviewed and merged.

thomwiggers avatar Nov 19 '25 08:11 thomwiggers