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

Support for authenticated media / profile pictures fail to load #2845

Open
SKBotNL opened this issue Nov 28, 2024 · 0 comments · May be fixed by #2856
Open

Support for authenticated media / profile pictures fail to load #2845

SKBotNL opened this issue Nov 28, 2024 · 0 comments · May be fixed by #2856
Assignees
Labels
T-Defect Something isn't working: bugs, crashes, hangs, vulnerabilities, or other reported problems

Comments

@SKBotNL
Copy link

SKBotNL commented Nov 28, 2024

Steps to reproduce

  1. Upload a fresh profile picture in Element (possibly optional)
  2. Join an Element Call
  3. See error:
    image

Since there doesn't seem to be an "extra information" text box I'll provide the extra information I know here

The browser version and Element X Android (probably, unverified) try to use /_matrix/media/v3/thumbnail/{serverName}/{mediaId} which is deprecated according to matrix-org/matrix-spec-proposals/pull/3916 and the specification. The desktop client (Element version 1.11.86, Linux) doesn't seem to have the same issue since it uses /_matrix/media/v1/thumbnail/{serverName}/{mediaId} (note the v1 instead of v3).

Outcome

What did you expect?

My profile picture to load

What happened instead?

My profile picture doesn't load

Operating system

Reproducible on Brave and Element X Android

Browser information

Brave Version 1.73.91 (Chromium: 131.0.6778.85) and Element X version 1.6.22

URL for webapp

No response

Will you send logs?

No

@SKBotNL SKBotNL added the T-Defect Something isn't working: bugs, crashes, hangs, vulnerabilities, or other reported problems label Nov 28, 2024
@Half-Shot Half-Shot self-assigned this Dec 2, 2024
@Half-Shot Half-Shot linked a pull request Dec 2, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T-Defect Something isn't working: bugs, crashes, hangs, vulnerabilities, or other reported problems
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants