You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In iamb 0.0.10 (2e6376f) (installed via Debian apt), the URLs that iamb requests for images attached to messages are for the /_matrix/media/v3/ endpoints. Per https://spec.matrix.org/unstable/client-server-api/#content-repository, these are in the process of being deprecated. URLs of the form /_matrix/client/v1/media/ should be used, instead. Our server is running matrix-synapse-py3 1.20.2+bullseye1 (also installed via Debian apt).
To reproduce:
Connect to a Matrix server that uses newer image URLs.
Have someone send an image with a message attachment.
Try to download the image (:download in iamb).
Observe that the download fails.
The server's access logs should show that iamb requested a URL with an old-style path component.
The text was updated successfully, but these errors were encountered:
dstu
added a commit
to dstu/iamb
that referenced
this issue
Dec 9, 2024
It looks like this can be solved by updating the matrix-sdk dependency to version 0.8. I have taken a stab at this at https://github.com/dstu/iamb/tree/update_matrix_sdk_0.8. (That builds cleanly and appears to run fine, but I haven't given it a solid shakedown yet.)
Before opening a pull request, I'd like to have a better understanding of changes in the ruma and matrix-sdk crates, to make sure we aren't introducing new bugs. And it might be nice to have an integration test to validate the behavior of :download in the client before opening a pull request. But I'm happy to do so sooner if that seems reasonable.
In iamb 0.0.10 (2e6376f) (installed via Debian apt), the URLs that iamb requests for images attached to messages are for the
/_matrix/media/v3/
endpoints. Per https://spec.matrix.org/unstable/client-server-api/#content-repository, these are in the process of being deprecated. URLs of the form/_matrix/client/v1/media/
should be used, instead. Our server is running matrix-synapse-py3 1.20.2+bullseye1 (also installed via Debian apt).To reproduce:
:download
in iamb).The text was updated successfully, but these errors were encountered: