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

One broken folder stops the entire sync #7199

Closed
HanaGemela opened this issue May 22, 2019 · 9 comments
Closed

One broken folder stops the entire sync #7199

HanaGemela opened this issue May 22, 2019 · 9 comments
Assignees
Labels
Milestone

Comments

@HanaGemela
Copy link
Contributor

HanaGemela commented May 22, 2019

Server: 10.2.0
Client: 2.6.0-daily20190513 (build 11776)
macOS: 10.14.4

Steps to recreate:

  1. Add a new account that has one broken folder on the server

Expected result: All items apart from the broken folder have been synced

Actual result: Nothing has been synced - see log below

Note: Currently working on how to recreate that broken folder on server. Once known, the issues will be linked together

image

image

@HanaGemela HanaGemela added this to the 2.6.1 milestone May 22, 2019
@michaelstingl
Copy link
Contributor

Looks like a candidate for my collection in #5859

@HanaGemela
Copy link
Contributor Author

Follow steps 1-5 in owncloud/core#35305 to create that problematic folder

@jnweiger
Copy link
Contributor

It should also be possible to poison one folder using mitmproxy with a python script that fakes a 500 response, whenever a specific folder is accessed.

@ogoffart
Copy link
Contributor

ogoffart commented Jun 4, 2019

What was the server error code. Looking at the error message it seems to be a 404. But 404 should not cause the sync to stop, neither should 500.
The only errors causing the sync to abort is a 503 if the server is in maintainnence, or a connection drop (connection timeout or similar)

Could you paste the log from the client?

@HanaGemela
Copy link
Contributor Author

@ogoffart I've just emailed you the logs

@ogoffart
Copy link
Contributor

ogoffart commented Jun 4, 2019

Thanks for the log. I realized the problem was during the discovery and not during the propagation.

Fix PR: #7219

@ckamm ckamm added the ReadyToTest QA, please validate the fix/enhancement label Jun 14, 2019
@ckamm ckamm modified the milestones: 2.6.1, 2.6.0 Jun 14, 2019
@HanaGemela
Copy link
Contributor Author

The folder is not synced but it is still shown in the selective sync list as synced

image

@HanaGemela HanaGemela removed the ReadyToTest QA, please validate the fix/enhancement label Jul 24, 2019
@HanaGemela HanaGemela modified the milestones: 2.6.0, 2.6.1 Jul 24, 2019
@HanaGemela
Copy link
Contributor Author

@michaelstingl please decide if it is worth the effort to fix the selective sync list

@michaelstingl
Copy link
Contributor

The remaining issue is only cosmetic, so I'd close it for now.

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

No branches or pull requests

5 participants