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

[External Storage] Google Docs files are not synced in the client cause server reports empty content lenght #26797

Closed
SamuAlfageme opened this issue Dec 7, 2016 · 9 comments

Comments

@SamuAlfageme
Copy link

SamuAlfageme commented Dec 7, 2016

As "Google Docs files are free", they are causing an error in the desktop client when syncing since their exported versions do have size.

screen shot 2016-12-07 at 13 48 47

Steps to reproduce

  1. Create files with Google Drive interface (Docs, Spreadsheets, Presentations, ...)
  2. Configure Google Drive as External Storage option
  3. Sync with the desktop client

Expected behaviour

The files can be synced by the client and are the same that can be downloaded via the web interface (the results of the export with GDrive)

Actual behaviour

The files are included in the Not Synced tab with the action: Broken webserver returning empty content length for non-empty file on resume and therefore, ignored.

screen shot 2016-12-07 at 13 24 53

Server configuration

  • Operating system: CentOS 6.6
  • Web server: Apache
  • Database: MySQL
  • PHP version: 7.0
  • ownCloud version: 9.1.3 RC1

@PVince81

@guruz
Copy link
Contributor

guruz commented Dec 7, 2016

The message was introduced here: owncloud/client@7fc7925

I wonder if you get this for all files all the time?
Is this really a resume?

@cdamken
Copy link
Contributor

cdamken commented Mar 27, 2017

@guruz

I wonder if you get this for all files all the time?

Only some files have the problem, I can see in my case only the odt files.

Is this really a resume?

Not necessarily.

Here are my logs:

https://cloud.owncloud.com/owncloud/index.php/apps/files/?dir=/Shared/owncloud/support/github-issues/core/26797

Filename:
damken-cloud-26797.log

@PVince81 PVince81 modified the milestones: 10.0.1, 10.0 Apr 26, 2017
@PVince81 PVince81 modified the milestones: triage, 10.0.1 May 17, 2017
@david-boles
Copy link

I seem to be getting a similar issue, except that my error is "The downloaded file does not match the checksum, it will be resumed.". This is really annoying, any fix?

@Ablablab
Copy link

Ablablab commented Dec 24, 2017

+1
Server configuration
Operating system: Qnap 4.3.3.0404
Web server: Apache
Database: MariaDB
ownCloud version: 10.0.4.4

@PVince81
Copy link
Contributor

PVince81 commented Jan 8, 2018

This is likely because the Google docs files are magic internal files in Google Drive. To be able to sync them down, the GDrive external storage implementation in ownCloud needs to ask Google Drive to export the file. The exported file likely isn't resumable because it is likely that it is binary different every time it is downloaded again.

A possible fix would be to somehow have the external storage code refuse to fseek for those files which should cancel resuming and restart download from scratch.

The content-length of said exported files is also not predictable which might be why the field is not set. I thought we set it to -1 or -2 at some point.

@PVince81
Copy link
Contributor

PVince81 commented Jan 8, 2018

As for the clients, I'd suggest that the resume information gets discarded at the time the error is shown so that the next sync run redownloads the full file.

@ownclouders
Copy link
Contributor

Hey, this issue has been closed because the label status/STALE is set and there were no updates for 7 days. Feel free to reopen this issue if you deem it appropriate.

(This is an automated comment from GitMate.io.)

@SamuAlfageme
Copy link
Author

SamuAlfageme commented Feb 16, 2018

Moving to owncloud/files_external_gdrive#17 since files_external_gdrive has split from core now.

ping @PVince81

@felixboehm felixboehm removed this from the triage milestone Apr 10, 2018
@lock
Copy link

lock bot commented Jul 30, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Jul 30, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

8 participants