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

Video thumbnails incorrectly report "Error decrypting video" for network errors #28060

Open
Tracked by #245
richvdh opened this issue Sep 17, 2024 · 6 comments
Open
Tracked by #245
Labels
A-Media O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect

Comments

@richvdh
Copy link
Member

richvdh commented Sep 17, 2024

Suppose the media repository is having a bad day, and returning 500 or 404 errors; or that the video you are looking for has been removed/was never correctly uploaded.

In that case, the following is shown in the timeline:

image

This is not a decryption error; instead it is a networking error. It should be caught and reported as such.

Related: #27862, #28016

@richvdh richvdh added T-Defect S-Minor Impairs non-critical functionality or suitable workarounds exist A-Media O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience labels Sep 17, 2024
@richvdh
Copy link
Member Author

richvdh commented Sep 17, 2024

Console contains:

Unable to decrypt attachment:  DownloadError: NetworkError when attempting to fetch resource.

Other reports have the following in the logs:

2024-09-17T15:05:22.555Z W Unable to decrypt attachment:  MatrixError: [404] Not found (https://matrix-client.matrix.org/_matrix/media/v3/download/matrix.org/DAVspLKibesLGUjNopnQfTYa?allow_redirect=true)
M_NOT_FOUND: MatrixError: [404] Not found (https://matrix-client.matrix.org/_matrix/media/v3/download/matrix.org/DAVspLKibesLGUjNopnQfTYa?allow_redirect=true)
    at p (vector://vector/webapp/bundles/acaf93297a5a35ea8349/2586.js:2:446534)
    at d (vector://vector/webapp/bundles/acaf93297a5a35ea8349/2586.js:2:2737631)
    at async s.getFn (vector://vector/webapp/bundles/acaf93297a5a35ea8349/2586.js:2:2755377)
    at async ft.componentDidMount (vector://vector/webapp/bundles/acaf93297a5a35ea8349/2586.js:2:1942963)

@karolyi

This comment was marked as outdated.

@t3chguy

This comment was marked as off-topic.

@karolyi

This comment was marked as off-topic.

@richvdh

This comment was marked as off-topic.

@karolyi

This comment was marked as off-topic.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Media O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect
Projects
None yet
Development

No branches or pull requests

3 participants