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

Installer downloaded even though it doesn't exist #7390

Closed
HanaGemela opened this issue Aug 14, 2019 · 2 comments · Fixed by #9153
Closed

Installer downloaded even though it doesn't exist #7390

HanaGemela opened this issue Aug 14, 2019 · 2 comments · Fixed by #9153
Labels
good first issue p2-high Escalation, on top of current planning, release blocker type:bug
Milestone

Comments

@HanaGemela
Copy link
Contributor

HanaGemela commented Aug 14, 2019

Windows 10
Client beta1
Online updater: client-qa-server-0.4

Steps to recreate:

  1. Set a wrong download url (https://download.owncloud.com/desktop/daily/ownCloud-11.6.0.12480.12264-daily20190807.msi) on the online updater
  2. Click 'Restart&Update' button in General tab in the client (new version has to be available to see this button)
  3. Check the 'Update failed' dialog message

Actual result: The message says that the installer has been downloaded and it really is present in AppData\Roaming\ownCloud
Expected result: File has not been downloaded as it doesn't exist

TODO:

  • Allow https redirects
  • Download to a temporary file and only rename to the actual name if the download was successful
@ckamm
Copy link
Contributor

ckamm commented Aug 19, 2019

I don't understand this one. There already is error handling for catching bad downloads. Will need to try to reproduce.

@michaelstingl michaelstingl modified the milestones: 2.6.1, 2.6.2 Nov 26, 2019
@TheOneRing TheOneRing added p2-high Escalation, on top of current planning, release blocker and removed p3-medium Normal priority labels Mar 17, 2021
@TheOneRing TheOneRing modified the milestones: 2.6.2, 2.8.0 Mar 17, 2021
@TheOneRing
Copy link
Member

Ensure the download was successful and follow https redirects, right now we don't even follow redirects to our attic server.
On a redirect we write the http response to the target file....

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue p2-high Escalation, on top of current planning, release blocker type:bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants