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.

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

Authenticated media support #349

Open
reynoldsme opened this issue Sep 4, 2024 · 5 comments
Open

Authenticated media support #349

reynoldsme opened this issue Sep 4, 2024 · 5 comments

Comments

@reynoldsme
Copy link
Contributor

Matrix v1.11 introduces MSC3916 - Authenticated Media which when implemented and enabled on a homeserver, requires clients to send the users access token with any requests for media files which were uploaded after the setting was enabled.

This is still currently an optional setting, but quick adoption is likely and the matrix.org homeserver itself is scheduled to enable it on September 4th, 2024.

Currently iamb will return the following message when attempting to retrieve new media (media from before the setting was enabled is fine, as expected) from a Synapse server with enable_authenticated_media: True set: [Image error: Matrix(Http(Api(Server(ClientApi(Error { status_code: 404, body: Standard { kind: NotFound, message: "Not found" } })))))]

The Client implementation guidance section of the v1.11 release announcement may be helpful.

@reynoldsme
Copy link
Contributor Author

Oh, I see. This should happen automatically by updating the matrix-sdk dependency matrix-org/matrix-rust-sdk#3598 from 0.7.1 to something newer, but that either has not been released and/or the crates.io entry has not been updated yet. https://crates.io/crates/matrix-sdk

@ulyssa ulyssa pinned this issue Sep 4, 2024
@reynoldsme
Copy link
Contributor Author

I tried changing the version property of the matrix-sdk dependency to a git reference the latest commit of main on the matrix-rust-sdk in Cargo.toml, but I found there were a few breaking changes. Someone with a better understanding of the codebase and the matrix rust sdk may want to have look, it could be the needed changes are actually minimal.

@reynoldsme
Copy link
Contributor Author

Looks like matrix-org/matrix-rust-sdk#3742 is the issue to watch for any planned release of matrx-rust-sdk.

@reynoldsme
Copy link
Contributor Author

matrix-sdk v0.8.0 is now released and is up on crates.io! https://crates.io/crates/matrix-sdk

@mordquist
Copy link
Contributor

There is an updated version in Issue 378. @dstu, I will run your fork and report any issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants