anotherim/anotherim-desktop
2
0
Fork 0
Code Issues Pull requests Actions Packages Projects Releases Wiki Activity

Fix reporting failed HTTP upload

Browse source 
Status code can't be less than 200 and greater than or equal 300 at the same
time. Using || instead of && marks all uploads that do not have 2xx status code
as "failed".
This commit is contained in:
Wiktor Kwapisiewicz 2019-10-22 17:18:37 +02:00 committed by fiaxh
parent 247a368150
commit 0acbe4855d
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
plugins/http-files/src/file_sender.vala
View file

@ -102,7 +102,7 @@ public class HttpFileSender : FileSender, Object {
Soup.Session session = new Soup.Session(); Soup.Session session = new Soup.Session();
try { try {
yield session.send_async(message); yield session.send_async(message);
if (message.status_code < 200 && message.status_code >= 300) { if (message.status_code < 200 || message.status_code >= 300) {
throw new FileSendError.UPLOAD_FAILED("HTTP status code %s".printf(message.status_code.to_string())); throw new FileSendError.UPLOAD_FAILED("HTTP status code %s".printf(message.status_code.to_string()));
} }
} catch (Error e) { } catch (Error e) {