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

On bad connectivity, trying to upload a file will block. On cancelling the upload, the modal still appears (minutes later) #22998

Closed
ara4n opened this issue Aug 4, 2022 · 6 comments
Labels
A-Media O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect

Comments

@ara4n
Copy link
Member

ara4n commented Aug 4, 2022

Steps to reproduce

  1. Simulate a bad network
  2. In an e2ee room send some normal messages, which will probably stack up as we try to do e2ee with the server
  3. Send an image
  4. Watch an infinite fullscreen modal spinner; the thumbnail doesn't even appear (which is weird, as it shouldn't need to talk to the server to calculate a thumbnail)
  5. Click in the background of the modal to cancel it
  6. Move on with your life
  7. Be very confused when ~30s later the thumbnail upload still suddenly pops up... even if you're now in an entirely different room.

Outcome

What did you expect?

You should be able to queue up images to be sent even when on bad connectivity. They certainly shouldn't pop up unexpectedly later.

What happened instead?

Much flakiness.

Operating system

macOS 12.3.1 on M1

Application version

nightly

How did you install the app?

nightly

Homeserver

matrix.org

Will you send logs?

No

@ara4n ara4n added the T-Defect label Aug 4, 2022
@turt2live turt2live added A-Performance S-Minor Impairs non-critical functionality or suitable workarounds exist A-E2EE A-Media O-Uncommon Most users are unlikely to come across this or unexpected workflow labels Aug 5, 2022
@turt2live
Copy link
Member

(which is weird, as it shouldn't need to talk to the server to calculate a thumbnail)

This sounds like it might be the /media/config request or possibly it trying to encrypt before determining size/thumbnail/whatever

@t3chguy
Copy link
Member

t3chguy commented Nov 1, 2022

That is exactly what it is, it only happens once per session

@t3chguy
Copy link
Member

t3chguy commented Nov 1, 2022

Not specific to E2EE rooms either, the media info is used to early-reject uploads that are too large for your HS's config

@t3chguy t3chguy changed the title On bad connectivity, uploading an image in an E2EE room will block. On cancelling the upload, the modal still appears (minutes later) On bad connectivity, trying to upload a file will block. On cancelling the upload, the modal still appears (minutes later) Nov 1, 2022
@ara4n
Copy link
Member Author

ara4n commented May 7, 2023

(related: #7069)

@t3chguy
Copy link
Member

t3chguy commented May 9, 2023

Duplicate of #24667

@t3chguy t3chguy marked this as a duplicate of #24667 May 9, 2023
@t3chguy t3chguy closed this as completed May 9, 2023
@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale May 9, 2023
@gaelledel
Copy link

We should use the Retry banner that appears above the composer when sending fails.
This currently works for messages that failed to send. We should therefore display the same banner for anything else including photos, videos, attachments etc...
As a second phase, we will probably modify this and adopt the same behaviour/style as EX. This will come with the timeline rework using Compound.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Media O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect
Projects
None yet
Development

No branches or pull requests

4 participants