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

Transfers are not retried when they fail #751

Open
spantaleev opened this issue Dec 17, 2018 · 18 comments
Open

Transfers are not retried when they fail #751

spantaleev opened this issue Dec 17, 2018 · 18 comments

Comments

@spantaleev
Copy link

In an area where changing between networks happens often (LTE / Pocket Wi-Fi / other public Wi-Fi, possibly with walled garden login), Nextcloud transfers are unreliable.

I have a bunch of transfers that are stuck (forever?) with 2 different kinds of error:

  • The certificate for this server is invalid
  • Error code 100

Expected behaviour

Failed transfers are retried again later. If not automatically, then at least manually (by tapping on the transfer).

Actual behaviour

When a transfer fails, it's not retried at all and remains in an error state forever.

Steps to reproduce

Unknown.

iOS version

iOS 12.1

App version

Nextcloud for iOS 2.22.7.4

Server configuration

Operating system: CentOS 7.5

Database: PostgreSQL 11.1

Nextcloud version: 15.0.0 running within the official Nextcloud Apache image.

@marinofaggiana
Copy link
Member

Hi @spantaleev thanks for you report ... it's difficult for me repeat this issue, how I can reproduce it ? I have try with switch Wi-Fi / 4G but do not sprint the error.

@spantaleev
Copy link
Author

@marinofaggiana, I think it's more to do not with changing between Wi-Fi and 4G, but rather with ending up up on a Wi-Fi network which uses a walled-garden login (forwarding traffic to some login page, which obviously fails SSL certificate checks, etc.)

@sebastiaanveld
Copy link

sebastiaanveld commented Dec 19, 2018

@marinofaggiana I just ran into the same issue. I was in airplane-mode when I shot some movies (.mov) which queued up for uploading in the Activity screen message status showing "Transfer aborted'. I just duplicated:

  • Have Airplane mode enabled
  • Shoot a movie
  • Open the NC App
  • Disable Airplane mode/Enabled 4G/WIFI
  • NC App Activity shows (but not always) "Auto upload https://mync.domain.ext/remote.php/webdav/Photos Folder could not be created"
  • NC App Activity shows (but not always) "Upload xxx.mov Transfer aborted"
    (Translated Activity messages from Dutch).

Environment:
NC IOS App 2.22.8.11
NC 15.0.0

fileuploadaborted

@mattf
Copy link

mattf commented Dec 19, 2018

fyi, for error code 413 (payload too large), i find that nc ios app 2.22.8.11 (from testflight) will retry the transfers

@marinofaggiana
Copy link
Member

Hi everyone try the 2.22.8 (13), I don't know if it repair this issue but I have found a little bug

marinofaggiana pushed a commit that referenced this issue Dec 22, 2018
@spantaleev
Copy link
Author

How can one test that version? It's some build released on TestFlight?

@instantlinux
Copy link

I am able to reproduce this as well. I was moving nextcloud to a new host, forgot to update the buffer-size params, and now have about 100 photos in limbo. This bug should be easy to reproduce: install nextcloud behind a nginx reverse-proxy, set its proxy-body-size / client-body-buffer-size values below the size of typical photos, try to do an upload, then readjust the proxy to allow large-file uploads. The iOS app won't give you any option to retry the failed Error code 413 uploads.

Nextcloud for iOS 2.22.7.4
Nextcloud server 15.0.0

See also: nextcloud/desktop #925.

@Xansala
Copy link

Xansala commented Jan 22, 2019

The issue still remains with iOS 2.22.8.20 and it persists also with an update to server version 15.0.2 - there is no possibility to trigger manually a retry nor does the app seem to trigger it automatically (these fills are hanging there for weeks already). I have the same issue on my wife's iPhone too (both at latest iOS). Here is a screenshot from my phone how it looks like:

Edit: changed picture size

@DrRSatzteil
Copy link

DrRSatzteil commented Jan 22, 2019

I also get a variety of weird upload issues (locked ressources, server not responding, failed uploads aren't retried) especially when I'm trying to upload the whole camera roll from my phone. One major issue that I'm observing since the last versions is that photo uploads are either not retried at all or retried over and over again after the upload was completed (the file is already on the server but the app keeps on uploading it again and again). The funny thing is that the app keeps on doing this even when you delete the file from the phone (it still seems to be cached somewhere).

But there is something interesting I just found out today (maybe it helps to track down the issue): If I'm looking at the "Transfers" screen uploads keep on repeating over and over again and the queue does not get any shorter. However when I'm navigating to the folder where the uploads should be placed (I'm using subfolders in the photo directory) the uploads are working ok (still some weird errors every now and then but I'm able to upload all the files and failed uploads are retried and work eventually). This whole behaviour seems a bit weird but you may try if this "fixes" your issues as well.

@marinofaggiana
Copy link
Member

Hi the new 2.22.9 is available for testing in TestFlight with several news for the transfer, please try this new version for make test on upload.

@DrRSatzteil
Copy link

DrRSatzteil commented Jan 22, 2019

Where can I find the invite for Testflight?

EDIT: Sorry, found it: #513

@marinofaggiana
Copy link
Member

@DrRSatzteil
Copy link

Thanks!

I just restored 100 Photos from the "recently deleted" folder an the app instantly began to upload. All the files where uploaded successfully without any errors or unneccesary retries!

The App is still not super responsive while the upload is running (scrolling in the transfers list is kind of choppy) but it gets the job done. Looks pretty good!

@marinofaggiana
Copy link
Member

@DrRSatzteil whats model of iPhone or iPad use you ?

@DrRSatzteil
Copy link

I used an iPhone 7 32 GB (MN8X2ZD/A). Nexcloud Server is version 15.0.2.

@DrRSatzteil
Copy link

DrRSatzteil commented Jan 22, 2019

To get a scenario similar to the one I had earlier today I restored even more pictures and started to reupload about 600 Photos. So far (uploaded about 150 pictures) without any of the issues I had this morning with 2.22.8.

Edit: Upload went very smoothly for all 600 files

@user32000
Copy link

I've got two stuck uploads in iOS app version 2.23.1.4. The Transfers screen shows both files, but below that says "0 elements to upload". The app is set to store documents "On my iPhone". When the two files were created, the iOS device had a good Internet connection, but could not access the Nextcloud server (which sits on a private network). Returning to the private network later on, these files are now stuck, cannot be deleted in the iOS app, and there seems to be no way to trigger a new upload attempt.

Files I subsequently create in app while connected to the private network transfer immediately and as expected.

@Xansala
Copy link

Xansala commented Apr 21, 2019

Just wanted to leave a comment that with the latest iOS App the issue got resolved. Could upload more than 7000 pictues/videos smoothly.

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

No branches or pull requests

8 participants