-
Notifications
You must be signed in to change notification settings - Fork 25
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
0.0%Error while parsing headersOther error? -1 #5
Comments
With
|
This is clearly a bug. The redirection is not resolved. Looks like we need more error handling. |
Cannot reproduce any more. Probably been fixed in a previous commit. Please verify. |
Works for me, too. But |
Just tried to update
What does this error mean? Is my zsync file malformed? Edit: Btw before using AppImageUpdate I tried using AppImageLauncher's update mechanism, which didn't work either (probably for the same reason). It just exits without any graphical error message and leaves a Edit2: When I download the target version into the same directory manually and run AppImageUpdate for the old version, it recognizes the target file as a seed file from which it can re-use 100%. It then loads the remaining parts (i.e. zero), verifies the checksum and exits with status code 0. |
In your case, the Re. AppImageLauncher, please open an issue over there, that makes managing issues much easier for us, and they can be worked on more quickly. Can you retry with |
@overheadhunter Bintray broke/disabled range requests, the server doesn't reply properly any more. Not our fault, neither your fault, they are to blame. @probonopd you have some contacts there, would you like to talk to them please? |
According to RFC 7233, Section 4.4 you can not rely on the server supporting range requests:
While this is sad and I love AppImages for being able to load just the delta, the update mechanism must be prepared to deal with "200" responses and just silently load the whole file. Maybe log a message suggesting that the update mechanism isn't as efficient due to the server's lacking support for range requests - so the AppImage's maintainer is informed about this and may choose a better server (or contact the hoster's customer support^^). |
Well, not good style, but if the standard even thinks it's okay... Okay, let's try to fix the problem in AppImageUpdate. I'd say it's good enough to "just download" the entire file instead of trying to be overly smart and abort after all ranges have been fetched. |
Using the latest continuous build on Deepin 15.4:
The text was updated successfully, but these errors were encountered: