Skip to content

Conversation

@ckamm
Copy link
Contributor

@ckamm ckamm commented May 29, 2017

See owncloud/enterprise#1966

If the server and the client's database go out of sync, there could be
persistent 404 errors. This change ensures that the problem corrects
itself eventually by triggering a remote discovery of the file's
parent folders.

It does not address the root cause that might have lead to the
divergence.

See owncloud/enterprise#1966

If the server and the client's database go out of sync, there could be
persistent 404 errors. This change ensures that the problem corrects
itself eventually by triggering a remote discovery of the file's
parent folders.

It does not address the root cause that might have lead to the
divergence.
@ckamm ckamm self-assigned this May 29, 2017
@ckamm ckamm requested a review from guruz May 29, 2017 10:08
@ckamm
Copy link
Contributor Author

ckamm commented May 29, 2017

I don't think this is ideal: The issue shouldn't happen and hiding symptoms might make genuine bugs harder to track down, or get reported less.

@guruz guruz merged commit 577a271 into 2.3 May 29, 2017
@guruz guruz deleted the force-discovery-on-404 branch May 29, 2017 19:41
@SamuAlfageme SamuAlfageme mentioned this pull request Jul 24, 2017
84 tasks
@SamuAlfageme
Copy link
Contributor

@ckamm linking owncloud/core#28598 to have a reference here.

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

Successfully merging this pull request may close these issues.

4 participants