Skip to content
This repository has been archived by the owner on May 7, 2019. It is now read-only.

Unburden zotero's servers in case #11

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

oneyb
Copy link

@oneyb oneyb commented Jun 23, 2018

made it quicker to retry installation in case something goes wrong and remove the burden from the zotero server

Updated the copyright year

Copy link
Owner

@smathot smathot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this. However, wouldn't this break when the temporary file does exist, but is corrupted, for example because the previous download attempt aborted halfway?

@oneyb
Copy link
Author

oneyb commented Jun 28, 2018 via email

@oneyb
Copy link
Author

oneyb commented Jun 28, 2018

That's what I mean. I tested both downloads (which one should automate).

Zotero doesn't explicitly support this:
https://forums.zotero.org/discussion/70280/checksum-to-verify-download-integrity

I think they should...
Trust is good, testing is better.
(Vertrauen ist gut, Kontrolle ist besser.)

Copy link
Owner

@smathot smathot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This indeed goes some way to checking the integrity. However, there are a few issues:

  • It would take me extra time because I'd have to explicitly look up the checksums for each update
  • If the file is corrupted, it isn't downloaded again, but rather the installation is aborted. This is especially problematic when the script is automatically executed in the context of an Ubuntu repository, because the user won't have any idea of what went wrong.

The last issue is solvable, but the first probably isn't. So, while I appreciate the idea, I'm afraid that I'm not inclined to merge these changes.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants