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

The Trezor Suite update progress bar is stuck on Linux. #14416

Open
Ondra-Zik-SL opened this issue Sep 18, 2024 · 8 comments · May be fixed by #15448
Open

The Trezor Suite update progress bar is stuck on Linux. #14416

Ondra-Zik-SL opened this issue Sep 18, 2024 · 8 comments · May be fixed by #15448
Assignees
Labels
bug Something isn't working as expected Linux Linux related

Comments

@Ondra-Zik-SL
Copy link

Describe the bug
The update progress bar is stuck on Linux, showing 0.0 B / 0.0 B the entire time the update is being downloaded. After a while, depending on your connection, the update is completed, and you can restart Trezor Suite.

Info:

  • Suite Version: 24.8.3 updating to 24.9.2
  • OS: Ubuntu 24.04, Endeavour OS(rolling release)

How to reproduce

  1. Open Trezor Suite AppImage version 24.8.3.
  2. Start the update process and accept the update to version 24.9.2.
  3. Observe the stuck progress bar.

Expected behavior
The progress bar should reflect the correct status of the downloaded data.

Screenshots

Screencast.from.2024-09-18.14-22-15.mp4
@Ondra-Zik-SL Ondra-Zik-SL added the bug Something isn't working as expected label Sep 18, 2024
@github-project-automation github-project-automation bot moved this to 🎯 To do in Issues Suite Sep 18, 2024
@MiroslavProchazka MiroslavProchazka added Linux Linux related labels Sep 18, 2024
@MiroslavProchazka
Copy link
Contributor

cc @peter-sanderson this may be connected to the recent changes in auto-update in Electron

@STew790
Copy link
Contributor

STew790 commented Oct 2, 2024

I just tried the autoupdate and it is still an issue, unfortunately.

Info:

  • Suite version: desktop 24.10.1 (3017e58)
  • Browser: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) TrezorSuite/24.10.1 Chrome/126.0.6478.234 Electron/31.6.0 Safari/537.36
  • OS: Linux x86_64
  • Screen: 1920x1080
  • Device: Trezor T3T1 2.8.3 regular (revision 7f373ae71eca855dd41b4a0fdcc7cadaa13a8281)
  • Transport: BridgeTransport 2.0.33

@peter-sanderson
Copy link
Contributor

I think this issue is not related to any recent changes we made and it was there before.

I will definitely look into it, but it would be probably trickier...

@peter-sanderson
Copy link
Contributor

peter-sanderson commented Nov 18, 2024

It is very interesting, that in case I start with tor and then turn it off, it start to work and progress is shown gradually.

@peter-sanderson
Copy link
Contributor

In some issues they mention that the problem is missing Content-Length header. But we have it.

curl -I 'https://data.trezor.io/suite/releases/desktop/canary/Trezor-Suite-24.11.1-linux-x86_64.AppImage'

HTTP/2 200 
content-type: binary/octet-stream
content-length: 170411812
date: Mon, 18 Nov 2024 11:33:56 GMT
last-modified: Wed, 13 Nov 2024 14:17:07 GMT
etag: "f417f181ea9828277281aa2a04581baa-21"
server: AmazonS3
x-cache: Hit from cloudfront
via: 1.1 9b9ab8e6e595847652a9158c684a8926.cloudfront.net (CloudFront)
x-amz-cf-pop: PRG50-C1
x-amz-cf-id: IMr-FIZqmMOV3pEB_3FFpEKza2Ara6BWK8fQIq0owc2-j3Ag5m39SQ==
age: 5910
x-frame-options: SAMEORIGIN
strict-transport-security: max-age=31536000

@peter-sanderson
Copy link
Contributor

peter-sanderson commented Nov 18, 2024

Whet I force it (I changed the source code in node_modules of the Electron Updater) to use full download it works. 🤔 🤔 🤔

(It logs: Cannot download differentially, fallback to full download)

This is the real reason why it works when switching Tor (it raises exception that triggers this flow)

@peter-sanderson peter-sanderson linked a pull request Nov 19, 2024 that will close this issue
@komret komret moved this from 🎯 To do to 🚫 Blocked in Issues Suite Dec 3, 2024
@peter-sanderson
Copy link
Contributor

I have created a new issue in Electron Builder repo: electron-userland/electron-builder#8722 to grab some more attention.

Originally I thought that replying and taking over of existing issue will be enough.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working as expected Linux Linux related
Projects
Status: 🚫 Blocked
Development

Successfully merging a pull request may close this issue.

4 participants